tests: copy attr per subtest to avoid data race on parallel tests

Each parallel subtest needs its own copy of the Attributes struct since
NewTestState mutates the tag field via the shared pointer. Without the
copy, both HTTP and SSH subtests would race on writing attr.tag.

Signed-off-by: Brian Goff <cpuguy83@gmail.com>

Author: cpuguy83

test/gomod_git_auth_test.go

@@ -104,6 +104,7 @@ go {{ .ModFileGoVersion }}
 		ctx := startTestSpan(baseCtx, t)
 
 		testEnv.RunTest(ctx, t, func(ctx context.Context, client gwclient.Client) {
+			attr := attr
 			testState := gitservices.NewTestState(t, client, &attr)
 
 			worker, _, gitHost := initStates(&testState)
@@ -161,6 +162,7 @@ go {{ .ModFileGoVersion }}
 		agentErrChan := startSSHAgent(t, privkey, sockaddr)
 
 		testEnv.RunTest(ctx, t, func(ctx context.Context, client gwclient.Client) {
+			attr := attr
 			testState := gitservices.NewTestState(t, client, &attr)
 
 			_, repo, gitHost := initStates(&testState)