Merge pull request #142 from nebula-plugins/deprecate-integrationspec

Deprecate IntegrationSpec
nebula-plugins · Feb 13, 2024 · b3de807 · b3de807
2 parents 9bb901b + d91b067
commit b3de807
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/main/groovy/nebula/test/BaseIntegrationSpec.groovy b/src/main/groovy/nebula/test/BaseIntegrationSpec.groovy
@@ -24,6 +24,12 @@ import spock.lang.Specification
  * {@link Specification} implementation of the {@link IntegrationBase}.
  */
 @CompileStatic
+/**
+ * BaseIntegrationSpec is not recommended as it is not compatible with Gradle's instrumentation mechanisms
+ * ex. https://github.com/gradle/gradle/issues/27956 and https://github.com/gradle/gradle/issues/27639
+ *
+ * This will be removed in the next nebula-test major version
+ */
 abstract class BaseIntegrationSpec extends Specification implements IntegrationBase {
     @Rule
     TestName testName = new TestName()

diff --git a/src/main/groovy/nebula/test/IntegrationSpec.groovy b/src/main/groovy/nebula/test/IntegrationSpec.groovy
@@ -23,6 +23,13 @@ import groovy.transform.CompileStatic
  * @author Marcin Erdmann
  */
 @CompileStatic
+/**
+ * IntegrationSpec is not recommended as it is not compatible with Gradle's instrumentation mechanisms
+ * ex. https://github.com/gradle/gradle/issues/27956 and https://github.com/gradle/gradle/issues/27639
+*
+ * This will be removed in the next nebula-test major version
+ */
+@Deprecated
 abstract class IntegrationSpec extends BaseIntegrationSpec implements Integration {
     def setup() {
         Integration.super.initialize(getClass(), testName.methodName)