New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Objects: Adds tests to CI job #1621
Objects: Adds tests to CI job #1621
Conversation
Codecov ReportBase: 42.11% // Head: 1.67% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1621 +/- ##
==========================================
- Coverage 42.11% 1.67% -40.44%
==========================================
Files 61 226 +165
Lines 5499 10065 +4566
Branches 821 994 +173
==========================================
- Hits 2316 169 -2147
- Misses 2999 9887 +6888
+ Partials 184 9 -175
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Disabled CodeCov upload. Should coordinate later on if we want to add these to the coverage, as we'd need to also add Core tests so that the coverage is "complete". See current coverage report dropping from 40% to 1% because |
Adds a step in
build-objects
to run the Objects unit tests (currently quite lacking).Adds unit test to ensure all objects in
ObjectsKit
have an empty constructor implemented.In order to not duplicate builds,
build-objects
will now only build:Objects.csproj
Tests.csproj
All other projects correspond to
Converters
that are already being built on each of their respectiveConnector
builds.This also allowed to change the VM from windows to linux, which has less minute cost.