Skip to content
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

Minor test improvements #7554

Merged
merged 2 commits into from
Sep 21, 2023
Merged

Minor test improvements #7554

merged 2 commits into from
Sep 21, 2023

Conversation

mnonnenmacher
Copy link
Member

Please see the commit messages for details.

Remove test data from `VersionControlSystemTest` which is not required
for the tests.

Signed-off-by: Martin Nonnenmacher <[email protected]>
Update a NuGet identifier to use a namespace as the analyzer would find
it since [1].

[1]: #6046

Signed-off-by: Martin Nonnenmacher <[email protected]>
@mnonnenmacher mnonnenmacher force-pushed the minor-test-improvements branch from 87d9e0f to 79f6577 Compare September 21, 2023 14:06
@mnonnenmacher mnonnenmacher enabled auto-merge (rebase) September 21, 2023 14:06
@codecov
Copy link

codecov bot commented Sep 21, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (eadb556) 68.02% compared to head (79f6577) 68.02%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #7554   +/-   ##
=========================================
  Coverage     68.02%   68.02%           
  Complexity     2023     2023           
=========================================
  Files           343      343           
  Lines         16723    16723           
  Branches       2371     2371           
=========================================
  Hits          11375    11375           
  Misses         4364     4364           
  Partials        984      984           
Flag Coverage Δ
test 35.51% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mnonnenmacher mnonnenmacher merged commit 2a7d3f5 into main Sep 21, 2023
22 checks passed
@mnonnenmacher mnonnenmacher deleted the minor-test-improvements branch September 21, 2023 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants