Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(411)

Unified Diff: third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp

Issue 2781303002: [ES6 modules] Introduce ScriptModuleHash (Closed)
Patch Set: rebased Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/bindings/core/v8/ScriptModule.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp b/third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp
index f9ed1ab3fd8f429e8604faae290e724348c12b67..841ee4104e82d22d931027edd5e5b2d69a6aed38 100644
--- a/third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/ScriptModuleTest.cpp
@@ -86,6 +86,50 @@ TEST(ScriptModuleTest, compileFail) {
ASSERT_TRUE(module.isNull());
}
+TEST(ScriptModuleTest, equalAndHash) {
+ V8TestingScope scope;
+
+ ScriptModule moduleNull;
+ ScriptModule moduleA =
+ ScriptModule::compile(scope.isolate(), "export const a = 'a';", "a.js");
+ ASSERT_FALSE(moduleA.isNull());
+ ScriptModule moduleB =
+ ScriptModule::compile(scope.isolate(), "export const b = 'b';", "b.js");
+ ASSERT_FALSE(moduleB.isNull());
+ Vector<char> moduleDeletedBuffer(sizeof(ScriptModule));
+ ScriptModule& moduleDeleted =
+ *reinterpret_cast<ScriptModule*>(moduleDeletedBuffer.data());
+ HashTraits<ScriptModule>::constructDeletedValue(moduleDeleted, true);
+
+ EXPECT_EQ(moduleNull, moduleNull);
+ EXPECT_EQ(moduleA, moduleA);
+ EXPECT_EQ(moduleB, moduleB);
+ EXPECT_EQ(moduleDeleted, moduleDeleted);
+
+ EXPECT_NE(moduleNull, moduleA);
+ EXPECT_NE(moduleNull, moduleB);
+ EXPECT_NE(moduleNull, moduleDeleted);
+
+ EXPECT_NE(moduleA, moduleNull);
+ EXPECT_NE(moduleA, moduleB);
+ EXPECT_NE(moduleA, moduleDeleted);
+
+ EXPECT_NE(moduleB, moduleNull);
+ EXPECT_NE(moduleB, moduleA);
+ EXPECT_NE(moduleB, moduleDeleted);
+
+ EXPECT_NE(moduleDeleted, moduleNull);
+ EXPECT_NE(moduleDeleted, moduleA);
+ EXPECT_NE(moduleDeleted, moduleB);
+
+ EXPECT_NE(DefaultHash<ScriptModule>::Hash::hash(moduleA),
+ DefaultHash<ScriptModule>::Hash::hash(moduleB));
+ EXPECT_NE(DefaultHash<ScriptModule>::Hash::hash(moduleNull),
+ DefaultHash<ScriptModule>::Hash::hash(moduleA));
+ EXPECT_NE(DefaultHash<ScriptModule>::Hash::hash(moduleNull),
+ DefaultHash<ScriptModule>::Hash::hash(moduleB));
+}
+
TEST(ScriptModuleTest, moduleRequests) {
V8TestingScope scope;
ScriptModule module = ScriptModule::compile(
« no previous file with comments | « third_party/WebKit/Source/bindings/core/v8/ScriptModule.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698