improving comment of why single test group is challenging

Author: AgustinBettati

.github/workflows/e2e-testing.yaml

@@ -61,11 +61,13 @@ jobs:
           MONGODB_ATLAS_ORG_ID: ${{ secrets.CLOUD_DEV_ORG_ID }}
           MONGODB_ATLAS_BASE_URL: https://cloud-dev.mongodb.com/
           MONGODB_ATLAS_SECRET_PROFILE: cfn-cloud-dev-github-action
-        # run idividual test allowing for parallel execution. Due to usage of t.Setenv() in test code, t.Parallel() is not possible
         run: |
           cd cfn-resources/test/e2e/cluster
           go test -timeout 90m -v -run '^TestClusterCFN$' .
 
+  # Run idividual test in separate test group for parallel execution. 
+  # Due to usage of t.Setenv() in test code t.Parallel() is not possible.
+  # Having both tests run in same `go test` execution is not possible due to scripts used for private registry publishing modifying same files 
   cluster-pause:
     needs: change-detection
     if: ${{ needs.change-detection.outputs.cluster == 'true' }}
@@ -93,7 +95,6 @@ jobs:
           MONGODB_ATLAS_ORG_ID: ${{ secrets.CLOUD_DEV_ORG_ID }}
           MONGODB_ATLAS_BASE_URL: https://cloud-dev.mongodb.com/
           MONGODB_ATLAS_SECRET_PROFILE: cfn-cloud-dev-github-action
-        # run idividual test allowing for parallel execution. Due to usage of t.Setenv() in test code, t.Parallel() is not possible
         run: |
           cd cfn-resources/test/e2e/cluster
           go test -timeout 90m -v -run '^TestClusterPauseCFN$' .    

cfn-resources/test/e2e/cluster/cluster_pause_test.go

@@ -140,9 +140,8 @@ func testCreatePauseStack(t *testing.T, c *pauseTestContext) {
 
 	cluster := readPauseClusterFromAtlas(t, c)
 
-	a := assert.New(t)
-	a.Equal(cluster.GetId(), clusterID)
-	a.False(cluster.GetPaused())
+	assert.Equal(t, cluster.GetId(), clusterID)
+	assert.False(t, cluster.GetPaused())
 }
 
 func testUpdatePauseState(t *testing.T, c *pauseTestContext, pause bool) {
@@ -163,20 +162,14 @@ func testUpdatePauseState(t *testing.T, c *pauseTestContext, pause bool) {
 
 	cluster := readPauseClusterFromAtlas(t, c)
 
-	a := assert.New(t)
-	if pause {
-		a.True(cluster.GetPaused())
-	} else {
-		a.False(cluster.GetPaused())
-	}
+	assert.Equal(t, pause, cluster.GetPaused())
 }
 
 func testDeletePauseStack(t *testing.T, c *pauseTestContext) {
 	t.Helper()
 	utility.DeleteStack(t, c.cfnClient, pauseStackName)
-	a := assert.New(t)
 	_, resp, _ := c.atlasClient20231115014.ClustersApi.GetCluster(ctx.Background(), c.clusterTmplObj.ProjectID, c.clusterTmplObj.Name).Execute()
-	a.Equal(404, resp.StatusCode)
+	assert.Equal(t, 404, resp.StatusCode)
 }
 
 func cleanupPauseResources(t *testing.T, c *pauseTestContext) {