Skip to content

Commit

Permalink
Modified DoctestsTest to include optimization level
Browse files Browse the repository at this point in the history
The current behavior is a bit misleading, because the test case name
includes only the javascript source file, but not the optimization
level. So, you get three tests with the exact same names, for levels
-1, 0, and 9. This change simply adds the optimization level in the
name of the test.
  • Loading branch information
andreabergia authored and gbrail committed Oct 31, 2023
1 parent dd9c638 commit 1457790
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion testsrc/org/mozilla/javascript/tests/DoctestsTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ public static String loadFile(File f) throws IOException {
return new String(buf);
}

@Parameters(name = "{0}")
@Parameters(name = "{0} opt:{2}")
public static Collection<Object[]> doctestValues() throws IOException {
File[] doctests = getDoctestFiles();
List<Object[]> result = new ArrayList<Object[]>();
Expand Down

0 comments on commit 1457790

Please sign in to comment.