Merge pull request #93 from processing/restore_tests

benfry · Apr 27, 2020 · 69a92b0 · 69a92b0
2 parents 1a90b30 + 7e17706
commit 69a92b0
Show file tree

Hide file tree

Showing 16 changed files with 21 additions and 105 deletions.
diff --git a/...ex/util/CachedRuntimePathFactoryTest.java → ...de/java/CachedRuntimePathFactoryTest.java b/...ex/util/CachedRuntimePathFactoryTest.java → ...de/java/CachedRuntimePathFactoryTest.java
@@ -18,15 +18,12 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 

diff --git a/...til/CodeFolderRuntimePathFactoryTest.java → ...ava/CodeFolderRuntimePathFactoryTest.java b/...til/CodeFolderRuntimePathFactoryTest.java → ...ava/CodeFolderRuntimePathFactoryTest.java
@@ -18,14 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
-import processing.mode.java.pdex.util.RuntimePathFactoryTestUtil;
 
 import java.io.File;
 import java.util.List;

diff --git a/...il/CoreLibraryRuntimePathFactoryTest.java → ...va/CoreLibraryRuntimePathFactoryTest.java b/...il/CoreLibraryRuntimePathFactoryTest.java → ...va/CoreLibraryRuntimePathFactoryTest.java
@@ -18,14 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
-import processing.mode.java.pdex.util.RuntimePathFactoryTestUtil;
 
 import java.util.List;
 

diff --git a/...g/mode/java/pdex/ImportStatementTest.java → ...essing/mode/java/ImportStatementTest.java b/...g/mode/java/pdex/ImportStatementTest.java → ...essing/mode/java/ImportStatementTest.java
@@ -1,4 +1,4 @@
-package processing.mode.java.pdex;
+package processing.mode.java;
 
 import org.junit.Assert;
 import org.junit.Before;

diff --git a/...ex/util/JavaFxRuntimePathFactoryTest.java → ...de/java/JavaFxRuntimePathFactoryTest.java b/...ex/util/JavaFxRuntimePathFactoryTest.java → ...de/java/JavaFxRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/...pdex/util/JavaRuntimePathFactoryTest.java → ...mode/java/JavaRuntimePathFactoryTest.java b/...pdex/util/JavaRuntimePathFactoryTest.java → ...mode/java/JavaRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/.../LibrarySearchRuntimePathFactoryTest.java → .../LibrarySearchRuntimePathFactoryTest.java b/.../LibrarySearchRuntimePathFactoryTest.java → .../LibrarySearchRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/.../LibrarySketchRuntimePathFactoryTest.java → .../LibrarySketchRuntimePathFactoryTest.java b/.../LibrarySketchRuntimePathFactoryTest.java → .../LibrarySketchRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/...til/ModeSearchRuntimePathFactoryTest.java → ...ava/ModeSearchRuntimePathFactoryTest.java b/...til/ModeSearchRuntimePathFactoryTest.java → ...ava/ModeSearchRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/...til/ModeSketchRuntimePathFactoryTest.java → ...ava/ModeSketchRuntimePathFactoryTest.java b/...til/ModeSketchRuntimePathFactoryTest.java → ...ava/ModeSketchRuntimePathFactoryTest.java
@@ -18,13 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.util.List;
 

diff --git a/java/test/processing/mode/java/ParserTests.java b/java/test/processing/mode/java/ParserTests.java
@@ -19,7 +19,6 @@
 import org.junit.Test;
 
 import processing.app.SketchException;
-import processing.mode.java.pdex.JdtCompilerUtil;
 import processing.mode.java.preproc.PreprocessorResult;
 import processing.mode.java.preproc.issue.PdePreprocessIssueException;
 
@@ -68,21 +67,6 @@ static void expectRunnerException(final String id,
     }
   }
 
-  static void expectCompilerException(final String id) {
-    try {
-      final String program = preprocess(id, res(id + ".pde"));
-      boolean succeeded = compile(id, program);
-      if (succeeded) {
-        fail("Expected to fail.");
-      }
-    } catch (Exception e) {
-      if (!e.equals(e.getCause()) && e.getCause() != null)
-        fail(e.getCause().toString());
-      else
-        fail(e.toString());
-    }
-  }
-
   static void expectGood(final String id) {
     expectGood(id, true);
   }
@@ -94,13 +78,6 @@ static void expectGood(final String id, boolean ignoreWhitespace) {
   static void expectGood(final String id, boolean ignoreWhitespace, Optional<String> packageName) {
     try {
       final String program = preprocess(id, res(id + ".pde"), packageName);
-      boolean successful = compile(id, program);
-      if (!successful) {
-        System.err.println("----------------------------");
-        System.err.println(program);
-        System.err.println("----------------------------");
-        fail("Compilation failed.");
-      }
 
       final File expectedFile = res(id + ".expected");
       if (expectedFile.exists()) {
@@ -218,11 +195,6 @@ public void bug1064() {
     expectGood("bug1064");
   }
 
-  @Test
-  public void bug1145() {
-    expectCompilerException("bug1145");
-  }
-
   @Test
   public void bug1362() {
     expectGood("bug1362");
@@ -393,31 +365,4 @@ public void testPdfWrite() {
     expectGood("pdfwrite");
   }
 
-  private static boolean compile(String id, String program) {
-    // Create compilable AST to get syntax problems
-    CompilationUnit compilableCU = JdtCompilerUtil.makeAST(
-        ASTParser.newParser(AST.JLS11),
-        program.toCharArray(),
-        JdtCompilerUtil.COMPILER_OPTIONS
-    );
-
-    // Get syntax problems from compilable AST
-    Optional<IProblem> problem = Arrays.stream(compilableCU.getProblems())
-        .filter(IProblem::isError)
-        .findFirst();
-
-    if (problem.isPresent()) {
-      IProblem problemFound = problem.get();
-
-      System.err.println("Compilation issue: "
-          + problemFound.getMessage()
-          + "(" + problemFound.getSourceLineNumber() + ")"
-      );
-
-      return false;
-    } else {
-      return true;
-    }
-  }
-
 }
diff --git a/...de/java/pdex/util/ProblemFactoryTest.java → ...cessing/mode/java/ProblemFactoryTest.java b/...de/java/pdex/util/ProblemFactoryTest.java → ...cessing/mode/java/ProblemFactoryTest.java
@@ -1,4 +1,4 @@
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Assert;
 import org.junit.Before;

diff --git a/...ava/pdex/util/RuntimePathBuilderTest.java → ...ing/mode/java/RuntimePathBuilderTest.java b/...ava/pdex/util/RuntimePathBuilderTest.java → ...ing/mode/java/RuntimePathBuilderTest.java
@@ -18,15 +18,11 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.junit.Before;
 import org.junit.Test;
 import processing.app.Sketch;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
-import processing.mode.java.pdex.PreprocessedSketch;
-import processing.mode.java.pdex.util.RuntimePathFactoryTestUtil;
 
 import java.io.File;
 import java.util.Arrays;
@@ -42,7 +38,7 @@ public class RuntimePathBuilderTest {
   private JavaMode testMode;
   private List<ImportStatement> testImports;
   private Sketch testSketch;
-  private PreprocessedSketch.Builder result;
+  private PreprocSketch.Builder result;
 
   @Before
   public void setUp() throws Exception {
@@ -51,7 +47,7 @@ public void setUp() throws Exception {
     testImports = RuntimePathFactoryTestUtil.createTestImports();
     testSketch = RuntimePathFactoryTestUtil.createTestSketch();
 
-    result = new PreprocessedSketch.Builder();
+    result = new PreprocSketch.Builder();
     result.programImports.addAll(testImports);
     result.sketch = testSketch;
 

diff --git a/...pdex/util/RuntimePathFactoryTestUtil.java → ...mode/java/RuntimePathFactoryTestUtil.java b/...pdex/util/RuntimePathFactoryTestUtil.java → ...mode/java/RuntimePathFactoryTestUtil.java
@@ -18,14 +18,12 @@
 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 */
 
-package processing.mode.java.pdex.util;
+package processing.mode.java;
 
 import org.mockito.Mockito;
 import processing.app.Library;
 import processing.app.Sketch;
 import processing.app.SketchException;
-import processing.mode.java.JavaMode;
-import processing.mode.java.pdex.ImportStatement;
 
 import java.io.File;
 import java.io.IOException;

diff --git a/...ode/java/preproc/util/SyntaxUtilTest.java → .../processing/mode/java/SourceUtilTest.java b/...ode/java/preproc/util/SyntaxUtilTest.java → .../processing/mode/java/SourceUtilTest.java
@@ -1,23 +1,22 @@
-package processing.mode.java.preproc.util;
+package processing.mode.java;
 
 import org.junit.Assert;
 import org.junit.Test;
-import processing.mode.java.preproc.code.SyntaxUtil;
 
 
-public class SyntaxUtilTest {
+public class SourceUtilTest {
 
   @Test
   public void getCountPresent() {
     String input = "test1,test2\n,test3";
-    int count = processing.mode.java.preproc.code.SyntaxUtil.getCount(input, ",");
+    int count = SourceUtil.getCount(input, ",");
     Assert.assertEquals(2, count);
   }
 
   @Test
   public void getCountNotPresent() {
     String input = "test1 test2 test3";
-    int count = SyntaxUtil.getCount(input, ",");
+    int count = SourceUtil.getCount(input, ",");
     Assert.assertEquals(0, count);
   }
 

diff --git a/...proc/code/PrintWriterWithEditGenTest.java → ...a/preproc/PrintWriterWithEditGenTest.java b/...proc/code/PrintWriterWithEditGenTest.java → ...a/preproc/PrintWriterWithEditGenTest.java
@@ -1,12 +1,11 @@
-package processing.mode.java.preproc.code;
+package processing.mode.java.preproc;
 
 import org.antlr.v4.runtime.TokenStreamRewriter;
 import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 import org.mockito.Mockito;
-import processing.mode.java.pdex.TextTransform;
-import processing.mode.java.preproc.PdeParseTreeListener;
+import processing.mode.java.TextTransform;
 
 import java.util.List;