Skip to content

Fix linter and add unit tests

Codecov / codecov/patch failed Nov 20, 2023 in 1s

66.00% of diff hit (target 80.00%)

View this Pull Request on Codecov

66.00% of diff hit (target 80.00%)

Annotations

Check warning on line 593 in controllers/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/factory.go#L593 

Added line #L593 was not covered by tests

Check warning on line 601 in controllers/factory.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/factory.go#L601 

Added line #L601 was not covered by tests

Check warning on line 62 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L59-L62 

Added lines #L59 - L62 were not covered by tests

Check warning on line 82 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L79-L82 

Added lines #L79 - L82 were not covered by tests

Check warning on line 88 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L87-L88 

Added lines #L87 - L88 were not covered by tests

Check warning on line 93 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L92-L93 

Added lines #L92 - L93 were not covered by tests

Check warning on line 97 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L96-L97 

Added lines #L96 - L97 were not covered by tests

Check warning on line 103 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L102-L103 

Added lines #L102 - L103 were not covered by tests

Check warning on line 109 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L108-L109 

Added lines #L108 - L109 were not covered by tests

Check warning on line 124 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L123-L124 

Added lines #L123 - L124 were not covered by tests

Check warning on line 138 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L137-L138 

Added lines #L137 - L138 were not covered by tests

Check warning on line 149 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L148-L149 

Added lines #L148 - L149 were not covered by tests

Check warning on line 155 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L153-L155 

Added lines #L153 - L155 were not covered by tests

Check warning on line 159 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L158-L159 

Added lines #L158 - L159 were not covered by tests

Check warning on line 166 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L164-L166 

Added lines #L164 - L166 were not covered by tests

Check warning on line 169 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L169 

Added line #L169 was not covered by tests

Check warning on line 176 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L175-L176 

Added lines #L175 - L176 were not covered by tests

Check warning on line 197 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L193-L197 

Added lines #L193 - L197 were not covered by tests

Check warning on line 216 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L202-L216 

Added lines #L202 - L216 were not covered by tests

Check warning on line 221 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L220-L221 

Added lines #L220 - L221 were not covered by tests

Check warning on line 230 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L228-L230 

Added lines #L228 - L230 were not covered by tests

Check warning on line 241 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L236-L241 

Added lines #L236 - L241 were not covered by tests

Check warning on line 317 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L300-L317 

Added lines #L300 - L317 were not covered by tests

Check warning on line 327 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L325-L327 

Added lines #L325 - L327 were not covered by tests

Check warning on line 338 in controllers/nodeupgrade_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/nodeupgrade_controller.go#L332-L338 

Added lines #L332 - L338 were not covered by tests
View more details on Codecov