Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Migrate TestSniperPrinter to JUnit5 #3921

Merged
merged 2 commits into from
May 14, 2021
Merged

test: Migrate TestSniperPrinter to JUnit5 #3921

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7513ba3
Migrate TestSnipePriter to JUnit5
slarse May 12, 2021
c4da2e6
Remove accidentally added test case
slarse May 12, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 18 additions & 25 deletions src/test/java/spoon/test/prettyprinter/TestSniperPrinter.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,12 @@
*/
package spoon.test.prettyprinter;

import org.junit.Rule;
import org.junit.Test;
import org.junit.rules.TemporaryFolder;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.io.TempDir;
import spoon.Launcher;
import spoon.SpoonException;
import spoon.refactoring.Refactoring;
import spoon.reflect.CtModel;
import spoon.reflect.code.CtBlock;
import spoon.reflect.code.CtConstructorCall;
import spoon.reflect.code.CtCodeSnippetExpression;
import spoon.reflect.code.CtExpression;
Expand Down Expand Up @@ -60,51 +58,47 @@
import java.util.List;
import java.util.function.BiConsumer;
import java.util.function.Consumer;
import java.util.function.Function;

import static org.hamcrest.CoreMatchers.allOf;
import static org.hamcrest.CoreMatchers.containsString;
import static org.hamcrest.CoreMatchers.anyOf;
import static org.hamcrest.MatcherAssert.assertThat;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertThrows;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertThrows;
import static org.junit.jupiter.api.Assertions.assertTrue;
import static org.junit.jupiter.api.Assertions.fail;

public class TestSniperPrinter {

@Rule
public TemporaryFolder folder = new TemporaryFolder();

@Test
public void testClassRename1() throws Exception {
public void testClassRename1(@TempDir File tempDir) throws Exception {
// contract: one can sniper out of the box after Refactoring.changeTypeName
testClassRename(type -> {
testClassRename(tempDir, type -> {
Refactoring.changeTypeName(type, "Bar");
});
}

@Test
public void testClassRename2() throws Exception {
public void testClassRename2(@TempDir File tempDir) throws Exception {
// contract: one can sniper after setSimpleName
// with the necessary tweaks
testClassRename(type -> {
testClassRename(tempDir, type -> {
type.setSimpleName("Bar");
type.getFactory().CompilationUnit().addType(type);
});

}

public void testClassRename(Consumer<CtType<?>> renameTransfo) throws Exception {
public void testClassRename(File tempdir, Consumer<CtType<?>> renameTransfo) throws Exception {
// contract: sniper supports class rename
String testClass = ToBeChanged.class.getName();
Launcher launcher = new Launcher();
launcher.addInputResource(getResourcePath(testClass));
launcher.getEnvironment().setPrettyPrinterCreator(() -> {
return new SniperJavaPrettyPrinter(launcher.getEnvironment());
});
launcher.setBinaryOutputDirectory(folder.newFolder());
launcher.setBinaryOutputDirectory(tempdir);
launcher.buildModel();
Factory f = launcher.getFactory();

Expand All @@ -117,7 +111,7 @@ public void testClassRename(Consumer<CtType<?>> renameTransfo) throws Exception


String contentOfPrettyPrintedClassFromDisk = getContentOfPrettyPrintedClassFromDisk(type);
assertTrue(contentOfPrettyPrintedClassFromDisk, contentOfPrettyPrintedClassFromDisk.contains("EOLs*/ Bar<T, K>"));
assertTrue(contentOfPrettyPrintedClassFromDisk.contains("EOLs*/ Bar<T, K>"), contentOfPrettyPrintedClassFromDisk);

}

Expand Down Expand Up @@ -559,7 +553,7 @@ public void testAddedImportStatementPlacedOnSeparateLineInFileWithoutPackageStat

Consumer<CtType<?>> addArrayListImport = type -> {
Factory factory = type.getFactory();
assertTrue("there should be no package statement in this test file", type.getPackage().isUnnamedPackage());
assertTrue(type.getPackage().isUnnamedPackage(), "there should be no package statement in this test file");
CtCompilationUnit cu = factory.CompilationUnit().getOrCreate(type);
CtTypeReference<?> arrayListRef = factory.Type().get(java.util.ArrayList.class).getReference();
cu.getImports().add(factory.createImport(arrayListRef));
Expand All @@ -580,7 +574,7 @@ public void testAddedImportStatementPlacedOnSeparateLineInFileWithPackageStateme

Consumer<CtType<?>> addArrayListImport = type -> {
Factory factory = type.getFactory();
assertFalse("there should be a package statement in this test file", type.getPackage().isUnnamedPackage());
assertFalse(type.getPackage().isUnnamedPackage(), "there should be a package statement in this test file");
CtCompilationUnit cu = factory.CompilationUnit().getOrCreate(type);
CtTypeReference<?> arrayListRef = factory.Type().get(java.util.ArrayList.class).getReference();
cu.getImports().add(factory.createImport(arrayListRef));
Expand Down Expand Up @@ -870,8 +864,8 @@ public void testToStringWithSniperPrinter(String inputSourcePath) throws Excepti
assertTrue(result.length() > 0);
}

assertTrue("ToString() on element (" + el.getClass().getName() + ") = \"" + el + "\" is not in original content",
originalContent.contains(result.replace("\t", "")));
assertTrue(originalContent.contains(result.replace("\t", "")),
"ToString() on element (" + el.getClass().getName() + ") = \"" + el + "\" is not in original content");
} catch (UnsupportedOperationException | SpoonException e) {
//Printer should not throw exception on printable element. (Unless there is a bug in the printer...)
fail("ToString() on Element (" + el.getClass().getName() + "): at " + el.getPath() + " lead to an exception: " + e);
Expand All @@ -883,5 +877,4 @@ public void testToStringWithSniperPrinter(String inputSourcePath) throws Excepti
public void testToStringWithSniperOnElementScan() throws Exception {
testToStringWithSniperPrinter("src/test/java/spoon/test/prettyprinter/testclasses/ElementScan.java");
}

}