Skip to content

Commit

Permalink
Update protobuf test to use jssuite from forked files (#636)
Browse files Browse the repository at this point in the history
* Update protobuf test to use jssuite from forked files

* Update bazel version

* Update protobuf test to use jssuite from forked files

* Update protobuf test to use jssuite from forked files and remove usage of assertHTMLEquals

* Remove legacy protobuf test

* Add back legacy test

* Add a blank line at the end of legacy example test

* Update legacy test to use forked jsunit and asserts

* Update legacy test to use forked jsunit and asserts

* Add back require line and remove additional msg.field()
  • Loading branch information
mollyibot authored Jan 7, 2025
1 parent 9b79a03 commit 9f58706
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
6 changes: 3 additions & 3 deletions closure/protobuf/test/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ closure_js_test(
],
deps = [
":foo",
"@com_google_javascript_closure_library//closure/goog/testing:testsuite",
"//closure/testing/library:testsuite",
],
)

Expand Down Expand Up @@ -95,7 +95,7 @@ closure_js_test(
deps = [
":baz",
":foo",
"@com_google_javascript_closure_library//closure/goog/testing:testsuite",
"//closure/testing/library:testsuite",
],
)

Expand Down Expand Up @@ -131,6 +131,6 @@ closure_js_test(
],
deps = [
":bat",
"@com_google_javascript_closure_library//closure/goog/testing:testsuite",
"//closure/testing/library:testsuite",
],
)
4 changes: 2 additions & 2 deletions closure/protobuf/test/legacy/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ closure_js_test(
srcs = ["example_test.js"],
deps = [
":example_lib",
"@com_google_javascript_closure_library//closure/goog/testing:asserts",
"@com_google_javascript_closure_library//closure/goog/testing:jsunit",
"//closure/testing/library:asserts",
"//closure/testing/library:jsunit",
],
)
3 changes: 1 addition & 2 deletions closure/protobuf/test/legacy/example_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,5 @@ goog.require('io.bazel.rules.closure.protobuf.Example');

function testExample() {
var msg = new io.bazel.rules.closure.protobuf.Example('value');
msg.field();
assertHTMLEquals('value', msg.field());
assertEquals('value', msg.field());
}

0 comments on commit 9f58706

Please sign in to comment.