Skip to content

Commit b06ff39

Browse files
committed
fix: update error messages in BrickCreate tests for consistency
1 parent e1f746b commit b06ff39

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

internal/orchestrator/bricks/bricks_test.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ func TestBrickCreate(t *testing.T) {
3434
t.Run("fails if brick id does not exist", func(t *testing.T) {
3535
err = brickService.BrickCreate(BrickCreateUpdateRequest{ID: "not-existing-id"}, f.Must(app.Load("testdata/dummy-app")))
3636
require.Error(t, err)
37-
require.Equal(t, "brick 'not-existing-id' not found", err.Error())
37+
require.Equal(t, "brick \"not-existing-id\" not found", err.Error())
3838
})
3939

4040
t.Run("fails if the requestes variable is not present in the brick definition", func(t *testing.T) {
@@ -43,7 +43,7 @@ func TestBrickCreate(t *testing.T) {
4343
}}
4444
err = brickService.BrickCreate(req, f.Must(app.Load("testdata/dummy-app")))
4545
require.Error(t, err)
46-
require.Equal(t, "variable 'NON_EXISTING_VARIABLE' does not exist on brick 'arduino:arduino_cloud'", err.Error())
46+
require.Equal(t, "variable \"NON_EXISTING_VARIABLE\" does not exist on brick \"arduino:arduino_cloud\"", err.Error())
4747
})
4848

4949
t.Run("fails if a required variable is set empty", func(t *testing.T) {
@@ -53,7 +53,7 @@ func TestBrickCreate(t *testing.T) {
5353
}}
5454
err = brickService.BrickCreate(req, f.Must(app.Load("testdata/dummy-app")))
5555
require.Error(t, err)
56-
require.Equal(t, "variable 'ARDUINO_DEVICE_ID' cannot be empty", err.Error())
56+
require.Equal(t, "variable \"ARDUINO_DEVICE_ID\" cannot be empty", err.Error())
5757
})
5858

5959
t.Run("fails if a mandatory variable is not present in the request", func(t *testing.T) {
@@ -62,7 +62,7 @@ func TestBrickCreate(t *testing.T) {
6262
}}
6363
err = brickService.BrickCreate(req, f.Must(app.Load("testdata/dummy-app")))
6464
require.Error(t, err)
65-
require.Equal(t, "required variable 'ARDUINO_DEVICE_ID' is mandatory", err.Error())
65+
require.Equal(t, "required variable \"ARDUINO_DEVICE_ID\" is mandatory", err.Error())
6666
})
6767

6868
t.Run("the brick is added if it does not exist in the app", func(t *testing.T) {

0 commit comments

Comments
 (0)