opendefensecloud / solution-arsenal / 29093298464
81%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 65
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 81.228% (+5.4%) from 75.819%
29093298464

push

github

web-flow 
chore: split up unit and integration tests, added tests (#684)

## What
Split tests between unit- and integration-tests, added make targets for
testing and for watching unit tests, added tests
Closes #667

## Why
Quicker feedback cycles for (some) tests, better developer ergonomy. 

## Testing
`make test` is unchanged
`make test-quick`: new target for unit tests
`make test-watch`: new target for watching unit tests -> takes ~5s at
most

## Notes for reviewers
none

## Checklist
- [x] Tests added/updated
- [x] No breaking changes (or upgrade path documented above)
- [x] Readable commit history (squashed and cleaned up as desired)
- [x] AI code review considered and comments resolved


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **New Features**
* Added quicker Go test targets for running focused unit tests,
including a watch mode for repeated local runs.

* **Bug Fixes**
* Expanded validation coverage for Solar API resources to better catch
missing or invalid fields.
* Improved checks around updates so invalid changes are rejected more
consistently.
* Added coverage for controller and discovery behaviors, including
retries, event handling, name truncation, and registry access grants.

* **Tests**
* Added broad new test suites for Solar API, controller, and discovery
components.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

4340 of 5343 relevant lines covered (81.23%)

39.43 hits per line

Coverage Regressions

Lines Coverage ∆ File
2
74.38
 0.96% pkg/controller/target_controller.go
Jobs
ID Job ID Ran Files Coverage
1 29093298464.1 65
81.23
 GitHub Action Run
Source Files on build 29093298464