chore(grants): cleanup

hashicorp · Feb 12, 2025 · 892fb32 · 892fb32
1 parent e8bc074
commit 892fb32
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 45 deletions.
diff --git a/internal/iam/repository_role_grant_ext_test.go b/internal/iam/repository_role_grant_ext_test.go
@@ -724,7 +724,6 @@ func TestGrantsForUser_DirectAssociation(t *testing.T) {
 			_, err := repo.GrantsForUser(ctx, userId, iam.WithTestCacheMultiGrantTuples(multiGrantTuplesCache))
 			require.NoError(t, err)
 
-			// log.Println("multiGrantTuplesCache", pretty.Sprint(*multiGrantTuplesCache))
 			assert.ElementsMatch(t, *multiGrantTuplesCache, expMultiGrantTuples[userId])
 		}
 	})
@@ -1038,14 +1037,7 @@ func TestGrantsForUser_DirectAssociation(t *testing.T) {
 					},
 				},
 			}
-			/*
-				log.Println("org1", directGrantOrg1.PublicId)
-				log.Println("proj1a", directGrantProj1a.PublicId)
-				log.Println("proj1b", directGrantProj1b.PublicId)
-				log.Println("org2", directGrantOrg2.PublicId)
-				log.Println("proj2a", directGrantProj2a.PublicId)
-				log.Println("proj2b", directGrantProj2b.PublicId)
-			*/
+
 			assert.Len(t, directGrants, len(expDirectGrants))
 			for k, v := range directGrants {
 				assert.ElementsMatch(t, v, expDirectGrants[k])
@@ -1395,7 +1387,6 @@ func TestGrantsForUser_Group(t *testing.T) {
 			_, err := repo.GrantsForUser(ctx, userId, iam.WithTestCacheMultiGrantTuples(multiGrantTuplesCache))
 			require.NoError(t, err)
 
-			// log.Println("multiGrantTuplesCache", pretty.Sprint(*multiGrantTuplesCache))
 			assert.ElementsMatch(t, *multiGrantTuplesCache, expMultiGrantTuples[userId])
 		}
 	})
@@ -1709,14 +1700,7 @@ func TestGrantsForUser_Group(t *testing.T) {
 					},
 				},
 			}
-			/*
-				log.Println("org1", directGrantOrg1.PublicId)
-				log.Println("proj1a", directGrantProj1a.PublicId)
-				log.Println("proj1b", directGrantProj1b.PublicId)
-				log.Println("org2", directGrantOrg2.PublicId)
-				log.Println("proj2a", directGrantProj2a.PublicId)
-				log.Println("proj2b", directGrantProj2b.PublicId)
-			*/
+
 			assert.Len(t, directGrants, len(expDirectGrants))
 			for k, v := range directGrants {
 				assert.ElementsMatch(t, v, expDirectGrants[k])
@@ -2088,7 +2072,6 @@ func TestGrantsForUser_ManagedGroup(t *testing.T) {
 			_, err := repo.GrantsForUser(ctx, userId, iam.WithTestCacheMultiGrantTuples(multiGrantTuplesCache))
 			require.NoError(t, err)
 
-			// log.Println("multiGrantTuplesCache", pretty.Sprint(*multiGrantTuplesCache))
 			assert.ElementsMatch(t, *multiGrantTuplesCache, expMultiGrantTuples[userId])
 		}
 	})
@@ -2402,14 +2385,7 @@ func TestGrantsForUser_ManagedGroup(t *testing.T) {
 					},
 				},
 			}
-			/*
-				log.Println("org1", directGrantOrg1.PublicId)
-				log.Println("proj1a", directGrantProj1a.PublicId)
-				log.Println("proj1b", directGrantProj1b.PublicId)
-				log.Println("org2", directGrantOrg2.PublicId)
-				log.Println("proj2a", directGrantProj2a.PublicId)
-				log.Println("proj2b", directGrantProj2b.PublicId)
-			*/
+
 			assert.Len(t, directGrants, len(expDirectGrants))
 			for k, v := range directGrants {
 				assert.ElementsMatch(t, v, expDirectGrants[k])

diff --git a/internal/iam/repository_role_grant_test.go b/internal/iam/repository_role_grant_test.go
@@ -757,7 +757,6 @@ func TestGrantsForUser(t *testing.T) {
 		_, err := repo.GrantsForUser(ctx, user.PublicId, WithTestCacheMultiGrantTuples(multiGrantTuplesCache))
 		require.NoError(t, err)
 
-		// log.Println("multiGrantTuplesCache", pretty.Sprint(*multiGrantTuplesCache))
 		assert.ElementsMatch(t, *multiGrantTuplesCache, expMultiGrantTuples)
 	})
 
@@ -1070,14 +1069,7 @@ func TestGrantsForUser(t *testing.T) {
 					},
 				},
 			}
-			/*
-				log.Println("org1", directGrantOrg1.PublicId)
-				log.Println("proj1a", directGrantProj1a.PublicId)
-				log.Println("proj1b", directGrantProj1b.PublicId)
-				log.Println("org2", directGrantOrg2.PublicId)
-				log.Println("proj2a", directGrantProj2a.PublicId)
-				log.Println("proj2b", directGrantProj2b.PublicId)
-			*/
+
 			assert.Len(t, directGrants, len(expDirectGrants))
 			for k, v := range directGrants {
 				assert.ElementsMatch(t, v, expDirectGrants[k])
@@ -1671,7 +1663,6 @@ func TestGrantsForUser_Group(t *testing.T) {
 		_, err := repo.GrantsForUser(ctx, user.PublicId, WithTestCacheMultiGrantTuples(multiGrantTuplesCache))
 		require.NoError(t, err)
 
-		// log.Println("multiGrantTuplesCache", pretty.Sprint(*multiGrantTuplesCache))
 		assert.ElementsMatch(t, *multiGrantTuplesCache, expMultiGrantTuples)
 	})
 
@@ -1984,14 +1975,7 @@ func TestGrantsForUser_Group(t *testing.T) {
 					},
 				},
 			}
-			/*
-				log.Println("org1", directGrantOrg1.PublicId)
-				log.Println("proj1a", directGrantProj1a.PublicId)
-				log.Println("proj1b", directGrantProj1b.PublicId)
-				log.Println("org2", directGrantOrg2.PublicId)
-				log.Println("proj2a", directGrantProj2a.PublicId)
-				log.Println("proj2b", directGrantProj2b.PublicId)
-			*/
+
 			assert.Len(t, directGrants, len(expDirectGrants))
 			for k, v := range directGrants {
 				assert.ElementsMatch(t, v, expDirectGrants[k])