Skip to content

Commit a8a8160

Browse files
authored
[+] add tests for CopyToProgram/CopyFromProgram tasks (#752)
1 parent 83084eb commit a8a8160

File tree

1 file changed

+24
-17
lines changed

1 file changed

+24
-17
lines changed

internal/scheduler/tasks_test.go

Lines changed: 24 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@ import (
1313

1414
func TestExecuteTask(t *testing.T) {
1515
mock, err := pgxmock.NewPool() //
16-
assert.NoError(t, err)
16+
a := assert.New(t)
17+
a.NoError(err)
1718
pge := pgengine.NewDB(mock, "--log-database-level=none")
1819
mocksch := New(pge, log.Init(config.LoggingOpts{LogLevel: "panic", LogDBLevel: "none"}))
1920

@@ -22,30 +23,36 @@ func TestExecuteTask(t *testing.T) {
2223
return
2324
}
2425

25-
assert.Error(t, et("foo", []string{}))
26+
a.Error(et("foo", []string{}))
2627

27-
assert.Error(t, et("Sleep", []string{"foo"}))
28-
assert.NoError(t, et("Sleep", []string{"1"}))
28+
a.Error(et("Sleep", []string{"foo"}))
29+
a.NoError(et("Sleep", []string{"1"}))
2930

30-
assert.NoError(t, et("NoOp", []string{}))
31-
assert.NoError(t, et("NoOp", []string{"foo", "bar"}))
31+
a.NoError(et("NoOp", []string{}))
32+
a.NoError(et("NoOp", []string{"foo", "bar"}))
3233

33-
assert.NoError(t, et("Log", []string{"foo"}))
34+
a.NoError(et("Log", []string{"foo"}))
3435

35-
assert.Error(t, et("CopyFromFile", []string{"foo"}), "Invalid json")
36-
assert.Error(t, et("CopyFromFile", []string{`{"sql": "COPY", "filename": "foo"}`}), "Acquire() should fail")
36+
a.Error(et("CopyFromFile", []string{"foo"}), "Invalid json")
37+
a.Error(et("CopyFromFile", []string{`{"sql": "COPY", "filename": "foo"}`}), "Acquire() should fail")
3738

38-
assert.Error(t, et("CopyToFile", []string{"foo"}), "Invalid json")
39-
assert.Error(t, et("CopyToFile", []string{`{"sql": "COPY", "filename": "foo"}`}), "Acquire() should fail")
39+
a.Error(et("CopyToFile", []string{"foo"}), "Invalid json")
40+
a.Error(et("CopyToFile", []string{`{"sql": "COPY", "filename": "foo"}`}), "Acquire() should fail")
4041

41-
assert.Error(t, et("SendMail", []string{"foo"}), "Invalid json")
42-
assert.Error(t, et("SendMail", []string{`{"ServerHost":"smtp.example.com","ServerPort":587,"Username":"user"}`}))
42+
a.Error(et("CopyToProgram", []string{"foo"}), "Invalid json")
43+
a.Error(et("CopyToProgram", []string{`{"sql": "COPY", "program": "foo"}`}), "Acquire() should fail")
4344

44-
assert.Error(t, et("Download", []string{"foo"}), "Invalid json")
45-
assert.EqualError(t, et("Download", []string{`{"workersnum": 0, "fileurls": [] }`}),
45+
a.Error(et("CopyFromProgram", []string{"foo"}), "Invalid json")
46+
a.Error(et("CopyFromProgram", []string{`{"sql": "COPY", "program": "foo"}`}), "Acquire() should fail")
47+
48+
a.Error(et("SendMail", []string{"foo"}), "Invalid json")
49+
a.Error(et("SendMail", []string{`{"ServerHost":"smtp.example.com","ServerPort":587,"Username":"user"}`}))
50+
51+
a.Error(et("Download", []string{"foo"}), "Invalid json")
52+
a.EqualError(et("Download", []string{`{"workersnum": 0, "fileurls": [] }`}),
4653
"files to download are not specified", "Download with empty files should fail")
47-
assert.Error(t, et("Download", []string{`{"workersnum": 0, "fileurls": ["http://foo.bar"], "destpath": "" }`}),
54+
a.Error(et("Download", []string{`{"workersnum": 0, "fileurls": ["http://foo.bar"], "destpath": "" }`}),
4855
"Downlod incorrect url should fail")
4956

50-
assert.NoError(t, et("Shutdown", []string{}))
57+
a.NoError(et("Shutdown", []string{}))
5158
}

0 commit comments

Comments
 (0)