chore: move break

brettkolodny · brettkolodny · commit c1f3619f4f0a · 2025-09-05T19:00:17.000Z
diff --git a/cli/sharing_test.go b/cli/sharing_test.go
@@ -211,9 +211,6 @@ func TestSharingStatus(t *testing.T) {
 		for _, line := range strings.Split(out.String(), "\n") {
 			if strings.Contains(line, toShareWithUser.Username) && strings.Contains(line, string(codersdk.WorkspaceRoleUse)) {
 				found = true
-			}
-
-			if found {
 				break
 			}
 		}
diff --git a/enterprise/cli/sharing_test.go b/enterprise/cli/sharing_test.go
@@ -238,9 +238,6 @@ func TestSharingStatus(t *testing.T) {
 		for _, line := range strings.Split(out.String(), "\n") {
 			if strings.Contains(line, orgMember.Username) && strings.Contains(line, string(codersdk.WorkspaceRoleUse)) && strings.Contains(line, group.Name) {
 				found = true
-			}
-
-			if found {
 				break
 			}
 		}

Original file line number	Diff line number	Diff line change
`@@ -211,9 +211,6 @@ func TestSharingStatus(t *testing.T) {`
`211`	`211`	`for _, line := range strings.Split(out.String(), "\n") {`
`212`	`212`	`if strings.Contains(line, toShareWithUser.Username) && strings.Contains(line, string(codersdk.WorkspaceRoleUse)) {`
`213`	`213`	`found = true`
`214`		`- }`
`215`		`-`
`216`		`- if found {`
`217`	`214`	`break`
`218`	`215`	`}`
`219`	`216`	`}`
Original file line number	Diff line number	Diff line change
`@@ -238,9 +238,6 @@ func TestSharingStatus(t *testing.T) {`
`238`	`238`	`for _, line := range strings.Split(out.String(), "\n") {`
`239`	`239`	`if strings.Contains(line, orgMember.Username) && strings.Contains(line, string(codersdk.WorkspaceRoleUse)) && strings.Contains(line, group.Name) {`
`240`	`240`	`found = true`
`241`		`- }`
`242`		`-`
`243`		`- if found {`
`244`	`241`	`break`
`245`	`242`	`}`
`246`	`243`	`}`