Skip to content

Commit bf077d3

Browse files
aslilacaslilac
committed
fix claude nonsense
1 parent d8ea897 commit bf077d3

File tree

214 files changed

+0
-371
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

214 files changed

+0
-371
lines changed

‎agent/agent.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -547,7 +547,6 @@ func (a *agent) reportMetadata(ctx context.Context, aAPI proto.DRPCAgentClient26
547547
// channel to synchronize the results and avoid both messy
548548
// mutex logic and overloading the API.
549549
for _, md := range manifest.Metadata {
550-
551550
// We send the result to the channel in the goroutine to avoid
552551
// sending the same result multiple times. So, we don't care about
553552
// the return values.

‎agent/agent_test.go

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,6 @@ func TestAgent_Stats_SSH(t *testing.T) {
130130
t.Parallel()
131131

132132
for _, port := range sshPorts {
133-
134133
t.Run(fmt.Sprintf("(:%d)", port), func(t *testing.T) {
135134
t.Parallel()
136135

@@ -342,7 +341,6 @@ func TestAgent_SessionExec(t *testing.T) {
342341
t.Parallel()
343342

344343
for _, port := range sshPorts {
345-
346344
t.Run(fmt.Sprintf("(:%d)", port), func(t *testing.T) {
347345
t.Parallel()
348346

@@ -468,7 +466,6 @@ func TestAgent_SessionTTYShell(t *testing.T) {
468466
}
469467

470468
for _, port := range sshPorts {
471-
472469
t.Run(fmt.Sprintf("(%d)", port), func(t *testing.T) {
473470
t.Parallel()
474471

@@ -611,7 +608,6 @@ func TestAgent_Session_TTY_MOTD(t *testing.T) {
611608
}
612609

613610
for _, test := range tests {
614-
615611
t.Run(test.name, func(t *testing.T) {
616612
t.Parallel()
617613
session := setupSSHSession(t, test.manifest, test.banner, func(fs afero.Fs) {
@@ -688,15 +684,13 @@ func TestAgent_Session_TTY_MOTD_Update(t *testing.T) {
688684

689685
//nolint:paralleltest // These tests need to swap the banner func.
690686
for _, port := range sshPorts {
691-
692687
sshClient, err := conn.SSHClientOnPort(ctx, port)
693688
require.NoError(t, err)
694689
t.Cleanup(func() {
695690
_ = sshClient.Close()
696691
})
697692

698693
for i, test := range tests {
699-
700694
t.Run(fmt.Sprintf("(:%d)/%d", port, i), func(t *testing.T) {
701695
// Set new banner func and wait for the agent to call it to update the
702696
// banner.
@@ -1209,7 +1203,6 @@ func TestAgent_CoderEnvVars(t *testing.T) {
12091203
t.Parallel()
12101204

12111205
for _, key := range []string{"CODER", "CODER_WORKSPACE_NAME", "CODER_WORKSPACE_AGENT_NAME"} {
1212-
12131206
t.Run(key, func(t *testing.T) {
12141207
t.Parallel()
12151208

@@ -1232,7 +1225,6 @@ func TestAgent_SSHConnectionEnvVars(t *testing.T) {
12321225
// For some reason this test produces a TTY locally and a non-TTY in CI
12331226
// so we don't test for the absence of SSH_TTY.
12341227
for _, key := range []string{"SSH_CONNECTION", "SSH_CLIENT"} {
1235-
12361228
t.Run(key, func(t *testing.T) {
12371229
t.Parallel()
12381230

@@ -1275,7 +1267,6 @@ func TestAgent_SSHConnectionLoginVars(t *testing.T) {
12751267
},
12761268
}
12771269
for _, tt := range tests {
1278-
12791270
t.Run(tt.key, func(t *testing.T) {
12801271
t.Parallel()
12811272

@@ -1795,7 +1786,6 @@ func TestAgent_ReconnectingPTY(t *testing.T) {
17951786
t.Setenv("LANG", "C")
17961787

17971788
for _, backendType := range backends {
1798-
17991789
t.Run(backendType, func(t *testing.T) {
18001790
if backendType == "Screen" {
18011791
if runtime.GOOS != "linux" {
@@ -2495,7 +2485,6 @@ func TestAgent_Dial(t *testing.T) {
24952485
}
24962486

24972487
for _, c := range cases {
2498-
24992488
t.Run(c.name, func(t *testing.T) {
25002489
t.Parallel()
25012490

‎agent/agentcontainers/containers_internal_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ func TestWrapDockerExec(t *testing.T) {
4141
},
4242
}
4343
for _, tt := range tests {
44-
// appease the linter even though this isn't needed anymore
4544
t.Run(tt.name, func(t *testing.T) {
4645
t.Parallel()
4746
actualCmd, actualArgs := wrapDockerExec("my-container", tt.containerUser, tt.cmdArgs[0], tt.cmdArgs[1:]...)
@@ -54,7 +53,6 @@ func TestWrapDockerExec(t *testing.T) {
5453
func TestConvertDockerPort(t *testing.T) {
5554
t.Parallel()
5655

57-
//nolint:paralleltest // variable recapture no longer required
5856
for _, tc := range []struct {
5957
name string
6058
in string
@@ -101,7 +99,6 @@ func TestConvertDockerPort(t *testing.T) {
10199
expectError: "invalid port",
102100
},
103101
} {
104-
//nolint: paralleltest // variable recapture no longer required
105102
t.Run(tc.name, func(t *testing.T) {
106103
t.Parallel()
107104
actualPort, actualNetwork, actualErr := convertDockerPort(tc.in)
@@ -151,7 +148,6 @@ func TestConvertDockerVolume(t *testing.T) {
151148
expectError: "invalid volume",
152149
},
153150
} {
154-
155151
t.Run(tc.name, func(t *testing.T) {
156152
t.Parallel()
157153
})

‎agent/agentscripts/agentscripts.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,6 @@ func (r *Runner) Init(scripts []codersdk.WorkspaceAgentScript, scriptCompleted S
177177
if script.Cron == "" {
178178
continue
179179
}
180-
181180
_, err := r.cron.AddFunc(script.Cron, func() {
182181
err := r.trackRun(r.cronCtx, script.WorkspaceAgentScript, ExecuteCronScripts)
183182
if err != nil {

‎agent/agentssh/x11_internal_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,6 @@ func Test_addXauthEntry(t *testing.T) {
228228
require.NoError(t, err)
229229

230230
for _, tt := range tests {
231-
232231
t.Run(tt.name, func(t *testing.T) {
233232
t.Parallel()
234233

‎agent/proto/compare_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,6 @@ func TestLabelsEqual(t *testing.T) {
6767
eq: false,
6868
},
6969
} {
70-
7170
t.Run(tc.name, func(t *testing.T) {
7271
t.Parallel()
7372
require.Equal(t, tc.eq, proto.LabelsEqual(tc.a, tc.b))

‎agent/proto/resourcesmonitor/queue_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@ func TestResourceMonitorQueue(t *testing.T) {
6565
}
6666

6767
for _, tt := range tests {
68-
6968
t.Run(tt.name, func(t *testing.T) {
7069
t.Parallel()
7170
queue := resourcesmonitor.NewQueue(20)

‎agent/proto/resourcesmonitor/resources_monitor_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,6 @@ func TestPushResourcesMonitoringWithConfig(t *testing.T) {
195195
}
196196

197197
for _, tt := range tests {
198-
199198
t.Run(tt.name, func(t *testing.T) {
200199
t.Parallel()
201200

‎apiversion/apiversion_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,6 @@ func TestAPIVersionValidate(t *testing.T) {
7272
expectedError: "no longer supported",
7373
},
7474
} {
75-
7675
t.Run(tc.name, func(t *testing.T) {
7776
t.Parallel()
7877

‎buildinfo/buildinfo_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ func TestBuildInfo(t *testing.T) {
9393
}
9494

9595
for _, c := range cases {
96-
9796
t.Run(c.name, func(t *testing.T) {
9897
t.Parallel()
9998
require.Equal(t, c.expectMatch, buildinfo.VersionsMatch(c.v1, c.v2),

0 commit comments

Comments
 (0)