From 258679c2402d95105817b5c29ef9722cda2a0988 Mon Sep 17 00:00:00 2001 From: Martin Traverse Date: Fri, 26 Jul 2024 23:11:42 +0100 Subject: [PATCH] Update tests for proto3 to let implicit members be optional but not nullable --- tests/cli.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/cli.js b/tests/cli.js index 5f78a25c7..3208edfc1 100644 --- a/tests/cli.js +++ b/tests/cli.js @@ -227,7 +227,7 @@ tape.test("with --null-semantics, optional fields are handled correctly in proto test.ok(jsCode.includes("@member {number|null} c"), "Member for c should be nullable") test.ok(jsCode.includes("OptionalFields.prototype.c = null;"), "Initializer for c should be null") - test.ok(jsCode.includes("@property {number} d OptionalFields d"), "Property for d should not be nullable") + test.ok(jsCode.includes("@property {number|undefined} [d] OptionalFields d"), "Property for d should be optional but not nullable") test.ok(jsCode.includes("@member {number} d"), "Member for d should not be nullable") test.ok(jsCode.includes("OptionalFields.prototype.d = 0;"), "Initializer for d should be zero")