Skip to content

Commit

Permalink
Autoformat all the *Test.java files under javascript/jscomp
Browse files Browse the repository at this point in the history
This will certainly have hurt the readability of some test cases where
our preferred `lines()` call with a trailing comment to force
one-argument-per-line is not already in use.

But, this will eliminate the very annoying case where autoformatting
keeps forcing formatting changes that obscure the meaningful ones.

PiperOrigin-RevId: 544147892
  • Loading branch information
varomodt authored and copybara-github committed Jun 28, 2023
1 parent e4ce27f commit 0102544
Show file tree
Hide file tree
Showing 170 changed files with 5,362 additions and 6,982 deletions.
1 change: 1 addition & 0 deletions test/com/google/javascript/jscomp/AstValidatorTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -1507,6 +1507,7 @@ public void testFeatureValidation_asyncGeneratorFunctions() {
public void testFeatureValidation_objectLiteralsWithSpread() {
testFeatureValidation("var obj = {...something};", Feature.OBJECT_LITERALS_WITH_SPREAD);
}

@Test
public void testValidFeatureInScript() {
// Since we're building the AST by hand, there won't be any types on it.
Expand Down
3 changes: 1 addition & 2 deletions test/com/google/javascript/jscomp/ChangeVerifierTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,7 @@ public void testChangeVerification() {

verifier.checkRecordedChanges(mainScript);

mainScript.addChildToFront(
IR.function(IR.name("A"), IR.paramList(), IR.block()));
mainScript.addChildToFront(IR.function(IR.name("A"), IR.paramList(), IR.block()));
compiler.reportChangeToChangeScope(mainScript);

IllegalStateException e =
Expand Down
Loading

0 comments on commit 0102544

Please sign in to comment.