Use snapshotId instead of separate tests (#2207)

Author: Vampire

spock-specs/src/test/groovy/org/spockframework/smoke/ast/AstSpec.groovy

@@ -344,30 +344,10 @@ class Ext <T extends Serializable, V extends Cloneable> {
     snapshotter.assertThat(result.source).matchesSnapshot()
   }
 
-  @Requires({ GroovyRuntimeUtil.groovy4orNewer })
-  def "Primitive types are used in AST transformation Groovy >= 4"() {
-    when:
-    def result = compiler.transpileWithImports('''
-class TestSpec extends Specification {
-  def 'test'() {
-    expect:
-    true
-    when:
-    true
-    then:
-    thrown(RuntimeException)
-  }
-}
-''',
-        EnumSet.of(Show.METHODS),
-        CompilePhase.OUTPUT)
-
-    then:
-    textSnapshotter.assertThat(result.source).matchesSnapshot()
-  }
+  def "Primitive types are used in AST transformation"() {
+    given:
+    def snapshotId = GroovyRuntimeUtil.groovy4orNewer ? "groovy4" : ""
 
-  @Requires({ GroovyRuntimeUtil.groovy3orOlder })
-  def "Primitive types are used in AST transformation Groovy <= 3"() {
     when:
     def result = compiler.transpileWithImports('''
 class TestSpec extends Specification {
@@ -385,6 +365,6 @@ class TestSpec extends Specification {
         CompilePhase.OUTPUT)
 
     then:
-    textSnapshotter.assertThat(result.source).matchesSnapshot()
+    textSnapshotter.assertThat(result.source).matchesSnapshot(snapshotId)
   }
 }