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

Unified Diff: mojo/public/js/tests/validation_unittest.js

Issue 2796253002: Associated Message Validation (Closed)
Patch Set: Validate payloadInterfaceIds before getting it. Use [0] for dimensions for validateArrayPointer. Ca… 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 | « mojo/public/js/interface_types.js ('k') | mojo/public/js/validator.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/public/js/tests/validation_unittest.js
diff --git a/mojo/public/js/tests/validation_unittest.js b/mojo/public/js/tests/validation_unittest.js
index 2a07315436e84113f095b189b8e8d80c04a59c8b..abefe8e878c61b13b5eae0dc7e0872185f596c7f 100644
--- a/mojo/public/js/tests/validation_unittest.js
+++ b/mojo/public/js/tests/validation_unittest.js
@@ -6,6 +6,7 @@ define([
"console",
"file",
"gin/test/expect",
+ "mojo/public/interfaces/bindings/tests/validation_test_associated_interfaces.mojom",
"mojo/public/interfaces/bindings/tests/validation_test_interfaces.mojom",
"mojo/public/js/bindings",
"mojo/public/js/buffer",
@@ -16,6 +17,7 @@ define([
], function(console,
file,
expect,
+ testAssociatedInterface,
testInterface,
bindings,
buffer,
@@ -270,6 +272,12 @@ define([
testInterface.BoundsCheckTestInterface.validateResponse]);
}
+ function testAssociatedConformanceMessageValidation() {
+ testMessageValidation("associated_conformance_", [
+ testAssociatedInterface.AssociatedConformanceTestInterface
+ .validateRequest]);
+ }
+
function testIntegratedMessageValidation(testFilesPattern, endpoint) {
var testFiles = getMessageTestFiles(testFilesPattern);
expect(testFiles.length).toBeGreaterThan(0);
@@ -321,6 +329,7 @@ define([
}
expect(checkTestMessageParser()).toBeNull();
+ testAssociatedConformanceMessageValidation();
testConformanceMessageValidation();
testBoundsCheckMessageValidation();
testResponseConformanceMessageValidation();
« no previous file with comments | « mojo/public/js/interface_types.js ('k') | mojo/public/js/validator.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698