remove empty method from TestSuite

Author: wirthi

graal-js/src/com.oracle.truffle.js.test.external/src/com/oracle/truffle/js/test/external/nashorn/TestNashorn.java

@@ -91,10 +91,6 @@ public TestRunnable createTestRunnable(TestFile file) {
         return new TestNashornRunnable(this, file);
     }
 
-    @Override
-    protected void setupTestFile(TestFile testFile) {
-    }
-
     @Override
     protected File getTestsConfigFile() {
         return Paths.get(getConfig().getSuiteConfigLoc(), TESTS_CONFIG_FILE).toFile();

graal-js/src/com.oracle.truffle.js.test.external/src/com/oracle/truffle/js/test/external/suite/TestSuite.java

@@ -151,8 +151,6 @@ public Engine getSharedEngine() {
 
     protected abstract TestRunnable createTestRunnable(TestFile file);
 
-    protected abstract void setupTestFile(TestFile testFile);
-
     protected abstract File getTestsConfigFile();
 
     protected abstract File getUnexpectedlyFailedTestsFile();
@@ -348,7 +346,6 @@ private TestFile createTestFile(File file) {
         TestFile testFile = new TestFile(relativizeTestPath(file));
         testFile.setStatus(TestFile.Status.PASS);
         testFile.setRunInIsolation(false);
-        setupTestFile(testFile);
         // possibly merge with config
         final String filePath = testFile.getFilePath();
         TestFile configuredTestFile = testsConfig.get(filePath);

graal-js/src/com.oracle.truffle.js.test.external/src/com/oracle/truffle/js/test/external/test262/Test262.java

@@ -140,10 +140,6 @@ public TestRunnable createTestRunnable(TestFile file) {
         return new Test262Runnable(this, file);
     }
 
-    @Override
-    protected void setupTestFile(TestFile testFile) {
-    }
-
     @Override
     protected File getTestsConfigFile() {
         return Paths.get(getConfig().getSuiteConfigLoc(), TESTS_CONFIG_FILE).toFile();

graal-js/src/com.oracle.truffle.js.test.external/src/com/oracle/truffle/js/test/external/testv8/TestV8.java

@@ -128,10 +128,6 @@ public TestRunnable createTestRunnable(TestFile file) {
         return new TestV8Runnable(this, file);
     }
 
-    @Override
-    protected void setupTestFile(TestFile testFile) {
-    }
-
     @Override
     protected File getTestsConfigFile() {
         return Paths.get(getConfig().getSuiteConfigLoc(), TESTS_CONFIG_FILE).toFile();

graal-js/src/com.oracle.truffle.js.test/src/com/oracle/truffle/js/test/nashorn/HereDocumentTest.java

@@ -52,23 +52,21 @@
 
 public class HereDocumentTest {
     private static String testIntl(String sourceText) {
-        try (Context context = JSTest.newContextBuilder().option(JSContextOptions.NASHORN_COMPATIBILITY_MODE_NAME, "true").option(JSContextOptions.SCRIPTING_NAME, "true").allowAllAccess(
-                        true).build()) {
-            Value result = context.eval(Source.newBuilder(JavaScriptLanguage.ID, sourceText, "javaadapter-test").buildLiteral());
+        try (Context context = JSTest.newContextBuilder().option(JSContextOptions.NASHORN_COMPATIBILITY_MODE_NAME, "true").option(JSContextOptions.SCRIPTING_NAME, "true").build()) {
+            Value result = context.eval(Source.newBuilder(JavaScriptLanguage.ID, sourceText, "here-document-test").buildLiteral());
             Assert.assertTrue(result.isString());
             return result.asString();
         }
     }
 
     @Test
-    public void javaAdapterTest() {
+    public void hereDocumentTest() {
         String sourceCode = "var str = <<EOD\n" +
                         "line1\n" +
                         "  line2\n" +
                         "    line3\n" +
                         "EOD\n" +
                         "str;";
-
         Assert.assertEquals("line1\n  line2\n    line3", testIntl(sourceCode));
     }
 }