Fix pool validation (#369)
7.57% of diff hit (target 36.46%)
View this Pull Request on Codecov
7.57% of diff hit (target 36.46%)
Annotations
Check warning on line 409 in flyteadmin/pkg/manager/impl/execution_manager.go
codecov / codecov/patch
flyteadmin/pkg/manager/impl/execution_manager.go#L409
Added line #L409 was not covered by tests
Check warning on line 25 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L24-L25
Added lines #L24 - L25 were not covered by tests
Check warning on line 30 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L28-L30
Added lines #L28 - L30 were not covered by tests
Check warning on line 35 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L33-L35
Added lines #L33 - L35 were not covered by tests
Check warning on line 40 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L39-L40
Added lines #L39 - L40 were not covered by tests
Check warning on line 47 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L42-L47
Added lines #L42 - L47 were not covered by tests
Check warning on line 55 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L51-L55
Added lines #L51 - L55 were not covered by tests
Check warning on line 58 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L58
Added line #L58 was not covered by tests
Check warning on line 66 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L65-L66
Added lines #L65 - L66 were not covered by tests
Check warning on line 71 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L69-L71
Added lines #L69 - L71 were not covered by tests
Check warning on line 76 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L74-L76
Added lines #L74 - L76 were not covered by tests
Check warning on line 81 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L80-L81
Added lines #L80 - L81 were not covered by tests
Check warning on line 88 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L83-L88
Added lines #L83 - L88 were not covered by tests
Check warning on line 96 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L92-L96
Added lines #L92 - L96 were not covered by tests
Check warning on line 99 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L99
Added line #L99 was not covered by tests
Check warning on line 107 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L106-L107
Added lines #L106 - L107 were not covered by tests
Check warning on line 112 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L110-L112
Added lines #L110 - L112 were not covered by tests
Check warning on line 117 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L115-L117
Added lines #L115 - L117 were not covered by tests
Check warning on line 122 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L121-L122
Added lines #L121 - L122 were not covered by tests
Check warning on line 129 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L124-L129
Added lines #L124 - L129 were not covered by tests
Check warning on line 137 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L133-L137
Added lines #L133 - L137 were not covered by tests
Check warning on line 140 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L140
Added line #L140 was not covered by tests
Check warning on line 148 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L147-L148
Added lines #L147 - L148 were not covered by tests
Check warning on line 153 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L151-L153
Added lines #L151 - L153 were not covered by tests
Check warning on line 158 in flyteadmin/pkg/manager/mocks/resource_interface.go
codecov / codecov/patch
flyteadmin/pkg/manager/mocks/resource_interface.go#L156-L158
Added lines #L156 - L158 were not covered by tests