Improve testing #25
Improve testing #25
Conversation
There was a problem hiding this comment.
Этот код не стоит смотреть, он максимально на коленке сделан из моего импорта на еджадж и понятное дело что при появлении нормального клиента с условиями задач он должен быть переделан
| fillInStorageConfig(config.Storage) | ||
| } | ||
| if config.Invoker != nil { | ||
| FillInInvokerConfig(config.Invoker) |
There was a problem hiding this comment.
можно не экспортировать видимо
There was a problem hiding this comment.
Не, оно в тестах инвокера нужно
| i.givenJobs[job.ID] = job | ||
| i.firstNotGivenTest++ | ||
|
|
||
| for givenJobID, testingJob := range i.givenJobs { |
There was a problem hiding this comment.
Выдача теста за линию теперь...
There was a problem hiding this comment.
Всм, given jobs это те джобы, которые сейчас работают, а не все вообще которые были
|
|
||
| if result.Verdict != verdict.OK { | ||
| for runningJobID, runningJob := range r.testingJobs { | ||
| if slices.Contains(runningJob.RequiredJobIDs, result.Job.ID) { |
There was a problem hiding this comment.
мб сделать RequiredJobIDs сетом?
There was a problem hiding this comment.
Хз, мне кажется там зависимость будет обычно от настолько малого числа джобов, что это ни на что не повлияет. Плюс в json тогда превращать сложнее будет
Add job cancel, polygon import and auto ip detection for invoker