aboutsummaryrefslogtreecommitdiff
path: root/test
diff options
context:
space:
mode:
authorRoel Spilker <r.spilker@gmail.com>2013-11-19 21:02:59 +0100
committerRoel Spilker <r.spilker@gmail.com>2013-11-19 21:02:59 +0100
commit78b2d6919e35887940f9f11b6ae1731245739b83 (patch)
tree0dc305883c19a862a38669374c5614e26480d4b8 /test
parent5deb185591904d275cb06eea85c0d739587fc738 (diff)
parent83b7e77b0cce6cd5993b17f36164271accdd281c (diff)
downloadlombok-78b2d6919e35887940f9f11b6ae1731245739b83.tar.gz
lombok-78b2d6919e35887940f9f11b6ae1731245739b83.tar.bz2
lombok-78b2d6919e35887940f9f11b6ae1731245739b83.zip
Merge branch 'master' of github.com:rzwitserloot/lombok
Conflicts: src/delombok/lombok/delombok/DelombokApp.java
Diffstat (limited to 'test')
-rw-r--r--test/core/src/lombok/CompilerMessageMatcher.java14
-rw-r--r--test/core/src/lombok/RunTestsViaEcj.java2
-rw-r--r--test/transform/resource/after-delombok/BuilderWithAccessors.java65
-rw-r--r--test/transform/resource/after-delombok/ConstructorsWithAccessors.java15
-rw-r--r--test/transform/resource/after-delombok/GetterSetterJavadoc.java12
-rw-r--r--test/transform/resource/after-delombok/NonNullOnParameterAbstract.java10
-rw-r--r--test/transform/resource/after-delombok/NonNullPlain.java9
-rw-r--r--test/transform/resource/after-delombok/NonNullWithSneakyThrows.java12
-rw-r--r--test/transform/resource/after-delombok/TestOperators.java48
-rw-r--r--test/transform/resource/after-ecj/BuilderWithAccessors.java47
-rw-r--r--test/transform/resource/after-ecj/ConstructorsWithAccessors.java13
-rw-r--r--test/transform/resource/after-ecj/InjectField.java4
-rw-r--r--test/transform/resource/after-ecj/NonNullOnParameterAbstract.java13
-rw-r--r--test/transform/resource/after-ecj/NonNullWithSneakyThrows.java18
-rw-r--r--test/transform/resource/after-ecj/TestOperators.java51
-rw-r--r--test/transform/resource/before/BuilderWithAccessors.java7
-rw-r--r--test/transform/resource/before/Constructors.java2
-rw-r--r--test/transform/resource/before/ConstructorsWithAccessors.java6
-rw-r--r--test/transform/resource/before/GetterOnMethod.java4
-rw-r--r--test/transform/resource/before/GetterOnMethodErrors.java2
-rw-r--r--test/transform/resource/before/GetterOnMethodErrors2.java14
-rw-r--r--test/transform/resource/before/GetterSetterJavadoc.java12
-rw-r--r--test/transform/resource/before/NonNullOnParameterAbstract.java7
-rw-r--r--test/transform/resource/before/NonNullWithSneakyThrows.java5
-rw-r--r--test/transform/resource/before/SetterOnMethodOnParam.java4
-rw-r--r--test/transform/resource/before/TestOperators.java48
-rw-r--r--test/transform/resource/messages-delombok/Accessors.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/BuilderInvalidUse.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/ConflictingStaticConstructorNames.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/EqualsAndHashCodeWithExistingMethods.java.messages6
-rw-r--r--test/transform/resource/messages-delombok/EqualsAndHashCodeWithSomeExistingMethods.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/FieldDefaultsNoop.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/GetterAlreadyExists.java.messages16
-rw-r--r--test/transform/resource/messages-delombok/GetterBoolean.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/GetterLazyInvalid.java.messages12
-rw-r--r--test/transform/resource/messages-delombok/GetterOnMethodErrors.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/GetterOnMethodErrors2.java.messages8
-rw-r--r--test/transform/resource/messages-delombok/LoggerSlf4jAlreadyExists.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/LoggerSlf4jOnNonStaticInnerClass.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/LoggerSlf4jOnNonType.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/LoggerSlf4jTypes.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/NonNullOnParameter.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/NonNullOnParameterAbstract.java.messages1
-rw-r--r--test/transform/resource/messages-delombok/NonNullPlain.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/SetterAlreadyExists.java.messages14
-rw-r--r--test/transform/resource/messages-delombok/SetterOnMethod.java.messages14
-rw-r--r--test/transform/resource/messages-delombok/SetterOnParam.java.messages14
-rw-r--r--test/transform/resource/messages-delombok/SetterOnParamAndOnMethod.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/SimpleTypeResolution.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/SynchronizedNameNoSuchField.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/SynchronizedNameStaticToInstanceRef.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/ValErrors.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/ValInBasicFor.java.messages2
-rw-r--r--test/transform/resource/messages-delombok/WitherAlreadyExists.java.messages14
-rw-r--r--test/transform/resource/messages-delombok/WitherOnStatic.java.messages4
-rw-r--r--test/transform/resource/messages-delombok/WitherWithDollar.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/Accessors.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/BuilderInvalidUse.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/ConflictingStaticConstructorNames.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/EqualsAndHashCodeWithExistingMethods.java.messages6
-rw-r--r--test/transform/resource/messages-ecj/EqualsAndHashCodeWithSomeExistingMethods.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/FieldDefaultsNoop.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/GetterAlreadyExists.java.messages16
-rw-r--r--test/transform/resource/messages-ecj/GetterBoolean.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/GetterLazyInvalid.java.messages12
-rw-r--r--test/transform/resource/messages-ecj/GetterOnMethodErrors.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/GetterOnMethodErrors2.java.messages8
-rw-r--r--test/transform/resource/messages-ecj/LoggerSlf4jAlreadyExists.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/LoggerSlf4jOnNonStaticInnerClass.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/LoggerSlf4jOnNonType.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/LoggerSlf4jTypes.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/NonNullOnParameter.java.messages6
-rw-r--r--test/transform/resource/messages-ecj/NonNullOnParameterAbstract.java.messages1
-rw-r--r--test/transform/resource/messages-ecj/NonNullPlain.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/SetterAlreadyExists.java.messages14
-rw-r--r--test/transform/resource/messages-ecj/SetterOnMethod.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/SetterOnParam.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/SimpleTypeResolution.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/SynchronizedName.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/SynchronizedNameNoSuchField.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/SynchronizedNameStaticToInstanceRef.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/ValErrors.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/ValInBasicFor.java.messages6
-rw-r--r--test/transform/resource/messages-ecj/ValRawType.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/ValueExperimental.java.messages2
-rw-r--r--test/transform/resource/messages-ecj/WitherAlreadyExists.java.messages14
-rw-r--r--test/transform/resource/messages-ecj/WitherOnStatic.java.messages4
-rw-r--r--test/transform/resource/messages-ecj/WitherWithDollar.java.messages2
-rw-r--r--test/transform/resource/messages-idempotent/NonNullOnParameter.java.messages2
-rw-r--r--test/transform/resource/messages-idempotent/NonNullOnParameterAbstract.java.messages1
-rw-r--r--test/transform/resource/messages-idempotent/NonNullPlain.java.messages6
-rw-r--r--test/transform/resource/messages-idempotent/SimpleTypeResolution.java.messages2
-rw-r--r--test/transform/resource/messages-idempotent/SynchronizedNameStaticToInstanceRef.java.messages2
-rw-r--r--test/transform/resource/messages-idempotent/ValErrors.java.messages6
-rw-r--r--test/transform/resource/messages-idempotent/ValInBasicFor.java.messages2
95 files changed, 557 insertions, 204 deletions
diff --git a/test/core/src/lombok/CompilerMessageMatcher.java b/test/core/src/lombok/CompilerMessageMatcher.java
index af12e199..db288749 100644
--- a/test/core/src/lombok/CompilerMessageMatcher.java
+++ b/test/core/src/lombok/CompilerMessageMatcher.java
@@ -38,30 +38,26 @@ public class CompilerMessageMatcher {
/** Line Number (starting at 1) */
private final long line;
- /** Position is either column number, OR position in file starting from the first byte. */
- private final long position;
private final Collection<String> messageParts;
- public CompilerMessageMatcher(long line, long position, String message) {
+ public CompilerMessageMatcher(long line, String message) {
this.line = line;
- this.position = position;
this.messageParts = Arrays.asList(message.split("\\s+"));
}
public static CompilerMessageMatcher asCompilerMessageMatcher(CompilerMessage message) {
- return new CompilerMessageMatcher(message.getLine(), message.getColumnOrPosition(), message.getMessage());
+ return new CompilerMessageMatcher(message.getLine(), message.getMessage());
}
@Override public String toString() {
StringBuilder parts = new StringBuilder();
for (String part : messageParts) parts.append(part).append(" ");
if (parts.length() > 0) parts.setLength(parts.length() - 1);
- return String.format("%d:%d %s", line, position, parts);
+ return String.format("%d %s", line, parts);
}
public boolean matches(CompilerMessage message) {
if (message.getLine() != this.line) return false;
- if (message.getColumnOrPosition() != this.position) return false;
for (String token : messageParts) {
if (!message.getMessage().contains(token)) return false;
}
@@ -78,12 +74,12 @@ public class CompilerMessageMatcher {
return out;
}
- private static final Pattern PATTERN = Pattern.compile("^(\\d+):(\\d+) (.*)$");
+ private static final Pattern PATTERN = Pattern.compile("^(\\d+) (.*)$");
private static CompilerMessageMatcher read(String line) {
line = line.trim();
if (line.isEmpty()) return null;
Matcher m = PATTERN.matcher(line);
if (!m.matches()) throw new IllegalArgumentException("Typo in test file: " + line);
- return new CompilerMessageMatcher(Integer.parseInt(m.group(1)), Integer.parseInt(m.group(2)), m.group(3));
+ return new CompilerMessageMatcher(Integer.parseInt(m.group(1)), m.group(2));
}
}
diff --git a/test/core/src/lombok/RunTestsViaEcj.java b/test/core/src/lombok/RunTestsViaEcj.java
index f7294f1f..586c124a 100644
--- a/test/core/src/lombok/RunTestsViaEcj.java
+++ b/test/core/src/lombok/RunTestsViaEcj.java
@@ -112,7 +112,7 @@ public class RunTestsViaEcj extends AbstractRunTests {
CategorizedProblem[] problems = compilationResult.getAllProblems();
if (problems != null) for (CategorizedProblem p : problems) {
- messages.add(new CompilerMessage(p.getSourceLineNumber(), p.getSourceStart(), p.getSourceStart(), p.isError(), p.getMessage()));
+ messages.add(new CompilerMessage(p.getSourceLineNumber(), p.getSourceStart(), p.isError(), p.getMessage()));
}
CompilationUnitDeclaration cud = compilationUnit_.get();
diff --git a/test/transform/resource/after-delombok/BuilderWithAccessors.java b/test/transform/resource/after-delombok/BuilderWithAccessors.java
new file mode 100644
index 00000000..503c33be
--- /dev/null
+++ b/test/transform/resource/after-delombok/BuilderWithAccessors.java
@@ -0,0 +1,65 @@
+class BuilderWithAccessors {
+ private final int plower;
+ private final int pUpper;
+ private int _foo;
+ private int __bar;
+ @java.lang.SuppressWarnings("all")
+ BuilderWithAccessors(final int plower, final int upper, final int foo, final int _bar) {
+ this.plower = plower;
+ this.pUpper = upper;
+ this._foo = foo;
+ this.__bar = _bar;
+ }
+ @java.lang.SuppressWarnings("all")
+ public static class BuilderWithAccessorsBuilder {
+ private int plower;
+ private int upper;
+ private int foo;
+ private int _bar;
+
+ @java.lang.SuppressWarnings("all")
+ BuilderWithAccessorsBuilder() {
+
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public BuilderWithAccessorsBuilder plower(final int plower) {
+ this.plower = plower;
+ return this;
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public BuilderWithAccessorsBuilder upper(final int upper) {
+ this.upper = upper;
+ return this;
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public BuilderWithAccessorsBuilder foo(final int foo) {
+ this.foo = foo;
+ return this;
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public BuilderWithAccessorsBuilder _bar(final int _bar) {
+ this._bar = _bar;
+ return this;
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public BuilderWithAccessors build() {
+ return new BuilderWithAccessors(plower, upper, foo, _bar);
+ }
+
+ @java.lang.Override
+ @java.lang.SuppressWarnings("all")
+ public java.lang.String toString() {
+ return "BuilderWithAccessors.BuilderWithAccessorsBuilder(plower=" + this.plower + ", upper=" + this.upper + ", foo=" + this.foo + ", _bar=" + this._bar + ")";
+ }
+ }
+
+ @java.lang.SuppressWarnings("all")
+ public static BuilderWithAccessorsBuilder builder() {
+ return new BuilderWithAccessorsBuilder();
+ }
+} \ No newline at end of file
diff --git a/test/transform/resource/after-delombok/ConstructorsWithAccessors.java b/test/transform/resource/after-delombok/ConstructorsWithAccessors.java
new file mode 100644
index 00000000..9937c7ee
--- /dev/null
+++ b/test/transform/resource/after-delombok/ConstructorsWithAccessors.java
@@ -0,0 +1,15 @@
+class ConstructorsWithAccessors {
+ int plower;
+ int pUpper;
+ int _huh;
+ int __huh2;
+
+ @java.beans.ConstructorProperties({"plower", "upper", "huh", "_huh2"})
+ @java.lang.SuppressWarnings("all")
+ public ConstructorsWithAccessors(final int plower, final int upper, final int huh, final int _huh2) {
+ this.plower = plower;
+ this.pUpper = upper;
+ this._huh = huh;
+ this.__huh2 = _huh2;
+ }
+}
diff --git a/test/transform/resource/after-delombok/GetterSetterJavadoc.java b/test/transform/resource/after-delombok/GetterSetterJavadoc.java
index 7bf92d1f..af9cbb5e 100644
--- a/test/transform/resource/after-delombok/GetterSetterJavadoc.java
+++ b/test/transform/resource/after-delombok/GetterSetterJavadoc.jav