Upstream/harbor
1
0
Fork 0
mirror of https://github.com/goharbor/harbor.git synced 2024-12-24 17:47:46 +01:00
Code Issues Projects Releases Wiki Activity

fix[UT]:improve the UT cases of enqueuer (#11358)

Browse Source 
- change the test cron spec
- use cretaed timer for timeout

Signed-off-by: Steven Zou <szou@vmware.com>
This commit is contained in:
Steven Zou 2020-03-31 00:09:20 +08:00 committed by GitHub
parent f2beed577f
commit 3d6c65f53b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/jobservice/period/enqueuer_test.go
View File

@ -96,9 +96,12 @@ func (suite *EnqueuerTestSuite) TestEnqueuer() {
}
}()
tk := time.NewTicker(500 * time.Millisecond)
tk := time.NewTicker(497 * time.Millisecond)
defer tk.Stop()
tm := time.NewTimer(15 * time.Second)
defer tm.Stop()
for {
select {
case <-tk.C:
@ -109,7 +112,7 @@ func (suite *EnqueuerTestSuite) TestEnqueuer() {
}, "at least one job should be scheduled for the periodic job policy") {
return
}
case <-time.After(15 * time.Second):
case <-tm.C:
require.NoError(suite.T(), errors.New("timeout (15s): expect at 1 scheduled job but still get nothing"))
return
}
@ -120,7 +123,7 @@ func (suite *EnqueuerTestSuite) prepare() {
now := time.Now()
minute := now.Minute()
coreSpec := fmt.Sprintf("0-59 %d * * * *", minute)
coreSpec := fmt.Sprintf("0-59 %d-%d * * * *", minute, minute+2)
// Prepare one
p := &Policy{