Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Kiichiro YUKAWA <kyukawa315@gmail.com>
  • Loading branch information
kevindiu and vankichi committed Feb 3, 2021
1 parent c29b8b5 commit 7e4e03a
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions internal/errors/worker_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ func TestErrWorkerIsNotRunning(t *testing.T) {
}
tests := []test{
{
name: "return ErrWorkerIsNotRunning error when name is index",
name: "return an ErrWorkerIsNotRunning error when name is index",
args: args{
name: "index",
},
Expand All @@ -56,7 +56,7 @@ func TestErrWorkerIsNotRunning(t *testing.T) {
},
},
{
name: "return ErrWorkerIsNotRunning error when name is empty",
name: "return an ErrWorkerIsNotRunning error when name is empty",
args: args{
name: "",
},
Expand Down Expand Up @@ -113,7 +113,7 @@ func TestErrWorkerIsAlreadyRunning(t *testing.T) {
}
tests := []test{
{
name: "return ErrWorkerIsAlreadyRunning error when name is index",
name: "return an ErrWorkerIsAlreadyRunning error when name is index",
args: args{
name: "index",
},
Expand All @@ -122,7 +122,7 @@ func TestErrWorkerIsAlreadyRunning(t *testing.T) {
},
},
{
name: "return ErrWorkerIsAlreadyRunning error when name is empty",
name: "return an ErrWorkerIsAlreadyRunning error when name is empty",
args: args{
name: "",
},
Expand Down Expand Up @@ -175,7 +175,7 @@ func TestErrQueueIsNotRunning(t *testing.T) {
}
tests := []test{
{
name: "return ErrQueueIsNotRunning error",
name: "return an ErrQueueIsNotRunning error",
want: want{
err: New("queue is not running"),
},
Expand Down Expand Up @@ -225,7 +225,7 @@ func TestErrQueueIsAlreadyRunning(t *testing.T) {
}
tests := []test{
{
name: "return ErrQueueIsAlreadyRunning error",
name: "return an ErrQueueIsAlreadyRunning error",
want: want{
err: New("queue is already running"),
},
Expand Down Expand Up @@ -275,7 +275,7 @@ func TestErrJobFuncIsNil(t *testing.T) {
}
tests := []test{
{
name: "return ErrJobFuncIsNil error",
name: "return an ErrJobFuncIsNil error",
want: want{
err: New("JobFunc is nil"),
},
Expand Down

0 comments on commit 7e4e03a

Please sign in to comment.