Skip to content

Commit d2f4152

Browse files
committed
style: replace em dashes with commas in test comments
1 parent 29c9cf8 commit d2f4152

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

cmd/ateapi/internal/controlapi/workflow_pause_test.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ import (
2727
// unknown.
2828
//
2929
// Old behavior: NodeVmsWithLocalSnapshots = []string{""}, which made
30-
// findFreeWorker search for a worker with node name ""never found a
30+
// findFreeWorker search for a worker with node name "", never found, a
3131
// permanent "no free workers available" on resume.
3232
//
3333
// Current behavior: NodeVmsWithLocalSnapshots is left nil, and the actor is
@@ -51,7 +51,7 @@ func TestFinalizePausedStep_WorkerGone(t *testing.T) {
5151
if _, err := st.CreateActor(ctx, actor); err != nil {
5252
t.Fatalf("CreateActor: %v", err)
5353
}
54-
// Intentionally NOT creating the worker in store simulates worker already gone.
54+
// Intentionally NOT creating the worker in store, simulates worker already gone.
5555

5656
step := &FinalizePausedStep{store: st}
5757
input := &PauseInput{Atespace: atespace, ActorName: actorName}
@@ -70,7 +70,7 @@ func TestFinalizePausedStep_WorkerGone(t *testing.T) {
7070
}
7171
for _, n := range got.GetLatestSnapshotInfo().GetLocal().GetNodeVmsWithLocalSnapshots() {
7272
if n == "" {
73-
t.Errorf("BUG: empty string in NodeVmsWithLocalSnapshots findFreeWorker would never match")
73+
t.Errorf("BUG: empty string in NodeVmsWithLocalSnapshots, findFreeWorker would never match")
7474
}
7575
}
7676

@@ -96,7 +96,7 @@ func TestFindFreeWorker_EmptyNodeRestriction(t *testing.T) {
9696

9797
s := &AssignWorkerStep{}
9898

99-
// Old behavior: []string{""} no worker has NodeName == "", returns nil.
99+
// Old behavior: []string{""}, no worker has NodeName == "", returns nil.
100100
got, err := s.findFreeWorker(workers, "", nil, nil, []string{""})
101101
if err != nil {
102102
t.Fatalf("findFreeWorker: %v", err)
@@ -105,7 +105,7 @@ func TestFindFreeWorker_EmptyNodeRestriction(t *testing.T) {
105105
t.Errorf("expected nil with old buggy input, got %v", got)
106106
}
107107

108-
// Fixed behavior: nil restrictions any free worker matches.
108+
// Fixed behavior: nil restrictions, any free worker matches.
109109
got, err = s.findFreeWorker(workers, "", nil, nil, nil)
110110
if err != nil {
111111
t.Fatalf("findFreeWorker: %v", err)

0 commit comments

Comments
 (0)