Remove @isolated annotations and update misleading parallel test comments

- Removed @isolated from CommandObjectsSpec and CommandObjectNoDataSpec as it
  only affects Spock's internal parallelism, not Gradle fork-based test isolation
- Updated comments across 29 files to clarify that fixes address test isolation
  (shared static state between tests in the same fork), not parallel execution
- Clarified that tests run sequentially within each fork but static state persists

Author: jamesfredley

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/connections/MongoConnectionSourcesSpec.groovy

@@ -60,7 +60,7 @@ class MongoConnectionSourcesSpec extends AutoStartedMongoSpec {
     }
 
     void setup() {
-        // Ensure tenant property is cleared before each test for isolation with parallel test execution
+        // Ensure tenant property is cleared before each test for test isolation
         System.clearProperty(SystemPropertyTenantResolver.PROPERTY_NAME)
     }
 

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/connections/MultiTenancySpec.groovy

@@ -58,7 +58,7 @@ class MultiTenancySpec extends AutoStartedMongoSpec {
     }
 
     void setup() {
-        // Ensure tenant property is cleared before each test for isolation with parallel test execution
+        // Ensure tenant property is cleared before each test for test isolation
         System.clearProperty(SystemPropertyTenantResolver.PROPERTY_NAME)
     }
 

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/connections/SchemaBasedMultiTenancySpec.groovy

@@ -52,7 +52,7 @@ class SchemaBasedMultiTenancySpec extends AutoStartedMongoSpec {
     }
 
     void setup() {
-        // Ensure tenant property is cleared before each test for isolation with parallel test execution
+        // Ensure tenant property is cleared before each test for test isolation
         System.clearProperty(SystemPropertyTenantResolver.PROPERTY_NAME)
     }
 

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/connections/SingleTenancySpec.groovy

@@ -67,7 +67,7 @@ class SingleTenancySpec extends AutoStartedMongoSpec {
     }
 
     void setup() {
-        // Ensure tenant property is cleared before each test for isolation with parallel test execution
+        // Ensure tenant property is cleared before each test for test isolation
         System.clearProperty(SystemPropertyTenantResolver.PROPERTY_NAME)
     }
 

grails-data-mongodb/core/src/test/groovy/org/grails/datastore/gorm/mongo/multitenancy/MongoStaticApiMultiTenancySpec.groovy

@@ -52,7 +52,7 @@ class MongoStaticApiMultiTenancySpec extends AutoStartedMongoSpec {
     }
 
     void setup() {
-        // Ensure tenant property is cleared before each test for isolation with parallel test execution
+        // Ensure tenant property is cleared before each test for test isolation
         System.clearProperty(SystemPropertyTenantResolver.PROPERTY_NAME)
     }
 

grails-domain-class/src/test/groovy/grails/persistence/DomainClassTraitSpec.groovy

@@ -39,7 +39,7 @@ class DomainClassTraitSpec extends Specification {
 
     void cleanupSpec() {
         // Use cleanupSpec() instead of cleanup() to clear Holders only after all tests
-        // in this class complete, avoiding interference with parallel tests in other classes
+        // in this class complete, ensuring proper test isolation
         Holders.clear()
     }
 

grails-gsp/plugin/src/test/groovy/org/grails/web/mapping/RestfulReverseUrlRenderingTests.groovy

@@ -31,12 +31,12 @@ import spock.lang.Specification
 class RestfulReverseUrlRenderingTests extends Specification implements UrlMappingsUnitTest<RestfulReverseUrlMappings> {
 
     def setup() {
-        // Clear any existing URL mappings artefacts to prevent pollution from parallel test execution
+        // Clear any existing URL mappings artefacts to ensure test isolation
         clearUrlMappingsArtefacts()
     }
 
     def cleanup() {
-        // Clear URL mappings artefacts added by this test to prevent pollution of other tests
+        // Clear URL mappings artefacts added by this test to ensure test isolation
         clearUrlMappingsArtefacts()
     }
 

grails-gsp/plugin/src/test/groovy/org/grails/web/taglib/AbstractGrailsTagTests.groovy

@@ -376,7 +376,7 @@ abstract class AbstractGrailsTagTests {
 
         onDestroy()
 
-        // Clear URL mappings artefacts to prevent pollution of parallel tests
+        // Clear URL mappings artefacts to ensure test isolation
         if (ga instanceof DefaultGrailsApplication) {
             ((DefaultGrailsApplication) ga).@artefactInfo.remove(UrlMappingsArtefactHandler.TYPE)
         }

grails-gsp/plugin/src/test/groovy/org/grails/web/taglib/FormTagLibTests.groovy

@@ -52,7 +52,7 @@ class FormTagLibTests extends Specification implements TagLibUnitTest<FormTagLib
     def setup() {
         tagLib.requestDataValueProcessor = new MockRequestDataValueProcessor()
 
-        // Clear any existing URL mappings artefacts to prevent pollution from parallel test execution
+        // Clear any existing URL mappings artefacts to ensure test isolation
         clearUrlMappingsArtefacts()
         grailsApplication.addArtefact(UrlMappingsArtefactHandler.TYPE, FormTagLibUrlMappings)
 

grails-mimetypes/src/main/groovy/org/grails/web/mime/HttpServletRequestExtension.groovy

@@ -76,7 +76,7 @@ class HttpServletRequestExtension {
     static MimeType[] getMimeTypes(HttpServletRequest request) {
         MimeType[] result = (MimeType[]) request.getAttribute(GrailsApplicationAttributes.REQUEST_FORMATS)
         if (!result) {
-            // First try to get context from GrailsWebRequest (more reliable in tests with parallel execution)
+            // First try to get context from GrailsWebRequest (more reliable for test isolation)
             // This avoids issues where the servlet context attributes may be polluted by other tests
             ApplicationContext context = null
             GrailsWebRequest webRequest = GrailsWebRequest.lookup()