Suggestions for AssertJComparableTemplates

PicnicSupermarket · Sep 8, 2022 · 19f3da2 · 19f3da2
1 parent c53784f
commit 19f3da2
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 26 deletions.
diff --git a/...ib/src/main/java/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplates.java b/...ib/src/main/java/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplates.java
@@ -12,8 +12,7 @@
 final class AssertJComparableTemplates {
   private AssertJComparableTemplates() {}
 
-  static final class AbstractComparableAssertActualIsEqualByComparingToExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsEqualByComparingTo<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isEqualTo(0);
@@ -26,8 +25,7 @@ AbstractIntegerAssert<?> before(T actual, T expected) {
     }
   }
 
-  static final class AbstractComparableAssertActualIsNotEqualByComparingToExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsNotEqualByComparingTo<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isNotEqualTo(0);
@@ -40,8 +38,7 @@ AbstractIntegerAssert<?> before(T actual, T expected) {
     }
   }
 
-  static final class AbstractComparableAssertActualIsLessThanExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsLessThan<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isNegative();
@@ -54,8 +51,7 @@ AbstractIntegerAssert<?> before(T actual, T expected) {
     }
   }
 
-  static final class AbstractComparableAssertActualIsLessThanOrEqualToExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsLessThanOrEqualTo<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isNotPositive();
@@ -68,8 +64,7 @@ AbstractIntegerAssert<?> before(T actual, T expected) {
     }
   }
 
-  static final class AbstractComparableAssertActualIsGreaterThanExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsGreaterThan<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isPositive();
@@ -82,8 +77,7 @@ AbstractIntegerAssert<?> before(T actual, T expected) {
     }
   }
 
-  static final class AbstractComparableAssertActualIsGreaterThanOrEqualToExpected<
-      T extends Comparable<? super T>> {
+  static final class AssertThatIsGreaterThanOrEqualTo<T extends Comparable<? super T>> {
     @BeforeTemplate
     AbstractIntegerAssert<?> before(T actual, T expected) {
       return assertThat(actual.compareTo(expected)).isNotNegative();

diff --git a/...sources/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplatesTestInput.java b/...sources/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplatesTestInput.java
@@ -3,31 +3,31 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import java.math.BigDecimal;
-import org.assertj.core.api.AbstractAssert;
+import org.assertj.core.api.AbstractComparableAssert;
 import tech.picnic.errorprone.refaster.test.RefasterTemplateTestCase;
 
 final class AssertJComparableTemplatesTest implements RefasterTemplateTestCase {
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsEqualByComparingToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsEqualByComparingTo() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isEqualTo(0);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsNotEqualByComparingToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsNotEqualByComparingTo() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isNotEqualTo(0);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsLessThanExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsLessThan() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isNegative();
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsLessThanOrEqualToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsLessThanOrEqualTo() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isNotPositive();
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsGreaterThanExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsGreaterThan() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isPositive();
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsGreaterThanOrEqualToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsGreaterThanOrEqualTo() {
     return assertThat(BigDecimal.ZERO.compareTo(BigDecimal.ONE)).isNotNegative();
   }
 }
diff --git a/...ources/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplatesTestOutput.java b/...ources/tech/picnic/errorprone/refastertemplates/AssertJComparableTemplatesTestOutput.java
@@ -3,31 +3,31 @@
 import static org.assertj.core.api.Assertions.assertThat;
 
 import java.math.BigDecimal;
-import org.assertj.core.api.AbstractAssert;
+import org.assertj.core.api.AbstractComparableAssert;
 import tech.picnic.errorprone.refaster.test.RefasterTemplateTestCase;
 
 final class AssertJComparableTemplatesTest implements RefasterTemplateTestCase {
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsEqualByComparingToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsEqualByComparingTo() {
     return assertThat(BigDecimal.ZERO).isEqualByComparingTo(BigDecimal.ONE);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsNotEqualByComparingToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsNotEqualByComparingTo() {
     return assertThat(BigDecimal.ZERO).isNotEqualByComparingTo(BigDecimal.ONE);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsLessThanExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsLessThan() {
     return assertThat(BigDecimal.ZERO).isLessThan(BigDecimal.ONE);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsLessThanOrEqualToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsLessThanOrEqualTo() {
     return assertThat(BigDecimal.ZERO).isLessThanOrEqualTo(BigDecimal.ONE);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsGreaterThanExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsGreaterThan() {
     return assertThat(BigDecimal.ZERO).isGreaterThan(BigDecimal.ONE);
   }
 
-  AbstractAssert<?, ?> testAbstractComparableAssertActualIsGreaterThanOrEqualToExpected() {
+  AbstractComparableAssert<?, ?> testAssertThatIsGreaterThanOrEqualTo() {
     return assertThat(BigDecimal.ZERO).isGreaterThanOrEqualTo(BigDecimal.ONE);
   }
 }