Implement review comments

spockframework · Dec 17, 2023 · a7642b5 · a7642b5
1 parent 443fc01
commit a7642b5
Show file tree

Hide file tree

Showing 3 changed files with 163 additions and 164 deletions.
diff --git a/spock-specs/src/test/groovy/org/spockframework/smoke/Interceptors.groovy b/spock-specs/src/test/groovy/org/spockframework/smoke/Interceptors.groovy
@@ -1,17 +1,16 @@
 /*
  * Copyright 2023 the original author or authors.
  *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
  *
- *     https://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
  */
 
 package org.spockframework.smoke
@@ -183,7 +182,7 @@ class SubSpec extends SuperSpec {
       specInfo.specsBottomToTop.each { spec ->
         spec.addInitializerInterceptor {
           assertIterationContext(it)
-          proceed(it, 'initializer', "$it.spec.name.$it.feature.name / $spec.name")
+          proceed(it, 'initializer', "$it.spec.name.$it.feature.name[#$it.iteration.iterationIndex] / $spec.name")
         }
       }
       specInfo.allInitializerMethods*.addInterceptor {
@@ -192,7 +191,7 @@ class SubSpec extends SuperSpec {
       }
       allFeatures*.addInitializerInterceptor {
         assertIterationContext(it)
-        proceed(it, 'feature scoped initializer', "$it.spec.name.$it.feature.name")
+        proceed(it, 'feature scoped initializer', "$it.spec.name.$it.feature.name[#$it.iteration.iterationIndex]")
       }
       allFeatures*.addIterationInterceptor {
         assertIterationContext(it)
@@ -206,7 +205,7 @@ class SubSpec extends SuperSpec {
       }
       allFeatures*.addSetupInterceptor {
         assertIterationContext(it)
-        proceed(it, 'feature scoped setup', "$it.spec.name.$it.feature.name")
+        proceed(it, 'feature scoped setup', "$it.spec.name.$it.feature.name[#$it.iteration.iterationIndex]")
       }
       specInfo.allSetupMethods*.addInterceptor {
         assertIterationMethodContext(it)
@@ -224,7 +223,7 @@ class SubSpec extends SuperSpec {
       }
       allFeatures*.addCleanupInterceptor {
         assertIterationContext(it)
-        proceed(it, 'feature scoped cleanup', "$it.spec.name.$it.feature.name")
+        proceed(it, 'feature scoped cleanup', "$it.spec.name.$it.feature.name[#$it.iteration.iterationIndex]")
       }
       specInfo.allCleanupMethods*.addInterceptor {
         assertIterationMethodContext(it)