Remove methods deprecated by GH-447

Author: ascopes

java-compiler-testing/src/main/java/io/github/ascopes/jct/assertions/JctCompilationAssert.java

@@ -250,22 +250,6 @@ public OutputContainerGroupAssert outputGroup(Location location) {
     return new OutputContainerGroupAssert(group);
   }
 
-  /**
-   * Get assertions on the path containing class outputs, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the class outputs.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #classOutputPackages()} or {@link #classOutputModules()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public OutputContainerGroupAssert classOutput() {
-    return outputGroup(StandardLocation.CLASS_OUTPUT);
-  }
-
   /**
    * Get assertions on the path containing class package outputs, if it exists.
    *
@@ -294,22 +278,6 @@ public ModuleContainerGroupAssert classOutputModules() {
     return outputGroup(StandardLocation.CLASS_OUTPUT).modules();
   }
 
-  /**
-   * Get assertions on the path containing generated source outputs, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the source outputs.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #sourceOutputPackages()} or {@link #sourceOutputModules()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public OutputContainerGroupAssert sourceOutput() {
-    return outputGroup(StandardLocation.SOURCE_OUTPUT);
-  }
-
   /**
    * Get assertions on the path containing generated source package outputs, if it exists.
    *
@@ -338,22 +306,6 @@ public ModuleContainerGroupAssert sourceOutputModules() {
     return outputGroup(StandardLocation.SOURCE_OUTPUT).modules();
   }
 
-  /**
-   * Get assertions on the path containing the class path, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the class path.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #classPathPackages()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public PackageContainerGroupAssert classPath() {
-    return packageGroup(StandardLocation.CLASS_PATH);
-  }
-
   /**
    * Get assertions on the path containing the class path, if it exists.
    *
@@ -368,22 +320,6 @@ public PackageContainerGroupAssert classPathPackages() {
     return packageGroup(StandardLocation.CLASS_PATH);
   }
 
-  /**
-   * Get assertions on the path containing the source path, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the source path.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #sourcePathPackages()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public PackageContainerGroupAssert sourcePath() {
-    return packageGroup(StandardLocation.SOURCE_PATH);
-  }
-
   /**
    * Get assertions on the path containing the source path, if it exists.
    *
@@ -398,22 +334,6 @@ public PackageContainerGroupAssert sourcePathPackages() {
     return packageGroup(StandardLocation.SOURCE_PATH);
   }
 
-  /**
-   * Get assertions on the path containing the source path, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the source path.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #moduleSourcePathModules()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public ModuleContainerGroupAssert moduleSourcePath() {
-    return moduleGroup(StandardLocation.MODULE_SOURCE_PATH);
-  }
-
   /**
    * Get assertions on the path containing the source path, if it exists.
    *
@@ -428,22 +348,6 @@ public ModuleContainerGroupAssert moduleSourcePathModules() {
     return moduleGroup(StandardLocation.MODULE_SOURCE_PATH);
   }
 
-  /**
-   * Get assertions on the path containing the module path, if it exists.
-   *
-   * <p>If not configured, the value being asserted on will be {@code null} in value.
-   *
-   * @return the assertions to perform on the module path.
-   * @throws AssertionError if the compilation was null, or no group for the location
-   *                        was found.
-   * @deprecated use {@link #modulePathModules()} instead.
-   */
-  @Deprecated(since = "0.6.4", forRemoval = true)
-  @SuppressWarnings("DeprecatedIsStillUsed")
-  public ModuleContainerGroupAssert modulePath() {
-    return moduleGroup(StandardLocation.MODULE_PATH);
-  }
-
   /**
    * Get assertions on the path containing the module path, if it exists.
    *

java-compiler-testing/src/test/java/io/github/ascopes/jct/tests/unit/assertions/JctCompilationAssertTest.java

@@ -765,27 +765,6 @@ void outputGroupReturnsContainerGroupAssertions() {
     }
   }
 
-  @DisplayName(".classOutput() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void classOutputPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(OutputContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.classOutput()).thenCallRealMethod();
-    when(assertions.outputGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.classOutput();
-
-    // Then
-    verify(assertions).classOutput();
-    verify(assertions).outputGroup(StandardLocation.CLASS_OUTPUT);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
-
   @DisplayName(".classOutputPackages() performs the expected operations")
   @Test
   void classOutputPackagesPerformsTheExpectedOperations() {
@@ -834,27 +813,6 @@ void classOutputModulesPerformsTheExpectedOperations() {
     assertThat(actualAssertions).isSameAs(expectedAssertions);
   }
 
-  @DisplayName(".sourceOutput() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void sourceOutputPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(OutputContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.sourceOutput()).thenCallRealMethod();
-    when(assertions.outputGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.sourceOutput();
-
-    // Then
-    verify(assertions).sourceOutput();
-    verify(assertions).outputGroup(StandardLocation.SOURCE_OUTPUT);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
-
   @DisplayName(".sourceOutputPackages() performs the expected operations")
   @Test
   void sourceOutputPackagesPerformsTheExpectedOperations() {
@@ -903,27 +861,6 @@ void sourceOutputModulesPerformsTheExpectedOperations() {
     assertThat(actualAssertions).isSameAs(expectedAssertions);
   }
 
-  @DisplayName(".classPath() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void classPathPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(PackageContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.classPath()).thenCallRealMethod();
-    when(assertions.packageGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.classPath();
-
-    // Then
-    verify(assertions).classPath();
-    verify(assertions).packageGroup(StandardLocation.CLASS_PATH);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
-
   @DisplayName(".classPathPackages() performs the expected operations")
   @Test
   void classPathPackagesPerformsTheExpectedOperations() {
@@ -944,27 +881,6 @@ void classPathPackagesPerformsTheExpectedOperations() {
     assertThat(actualAssertions).isSameAs(expectedAssertions);
   }
 
-  @DisplayName(".sourcePath() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void sourcePathPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(PackageContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.sourcePath()).thenCallRealMethod();
-    when(assertions.packageGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.sourcePath();
-
-    // Then
-    verify(assertions).sourcePath();
-    verify(assertions).packageGroup(StandardLocation.SOURCE_PATH);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
-
   @DisplayName(".sourcePathPackages() performs the expected operations")
   @Test
   void sourcePathPackagesPerformsTheExpectedOperations() {
@@ -985,27 +901,6 @@ void sourcePathPackagesPerformsTheExpectedOperations() {
     assertThat(actualAssertions).isSameAs(expectedAssertions);
   }
 
-  @DisplayName(".moduleSourcePath() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void moduleSourcePathPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(ModuleContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.moduleSourcePath()).thenCallRealMethod();
-    when(assertions.moduleGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.moduleSourcePath();
-
-    // Then
-    verify(assertions).moduleSourcePath();
-    verify(assertions).moduleGroup(StandardLocation.MODULE_SOURCE_PATH);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
-
   @DisplayName(".moduleSourcePathModules() performs the expected operations")
   @Test
   void moduleSourcePathModulesPerformsTheExpectedOperations() {
@@ -1025,27 +920,6 @@ void moduleSourcePathModulesPerformsTheExpectedOperations() {
     verifyNoMoreInteractions(assertions);
     assertThat(actualAssertions).isSameAs(expectedAssertions);
   }
-  
-  @DisplayName(".modulePath() performs the expected operations")
-  @SuppressWarnings("removal")
-  @Test
-  void modulePathPerformsTheExpectedOperations() {
-    // Given
-    var expectedAssertions = mock(ModuleContainerGroupAssert.class);
-
-    var assertions = mock(JctCompilationAssert.class);
-    when(assertions.modulePath()).thenCallRealMethod();
-    when(assertions.moduleGroup(any())).thenReturn(expectedAssertions);
-
-    // When
-    var actualAssertions = assertions.modulePath();
-
-    // Then
-    verify(assertions).modulePath();
-    verify(assertions).moduleGroup(StandardLocation.MODULE_PATH);
-    verifyNoMoreInteractions(assertions);
-    assertThat(actualAssertions).isSameAs(expectedAssertions);
-  }
 
   @DisplayName(".modulePathModules() performs the expected operations")
   @Test