Skip to content

Update code coverage config#403

Open
vexx32 wants to merge 4 commits intomainfrom
Test/CodeCoverage
Open

Update code coverage config#403
vexx32 wants to merge 4 commits intomainfrom
Test/CodeCoverage

Conversation

@vexx32
Copy link
Copy Markdown
Owner

@vexx32 vexx32 commented Jul 10, 2020

PR Summary

Attempt to manually configure tests for Pester v5 and add code coverage that should (hopefully) work correctly.

Context

The automatic test coverage generation that -CI enables is not reading in our function files since we don't store function files alongside test files. Hopefully a more specific manual configuration should do the trick.

Changes

  • Replace -CI switch with manually configured -Configuration settings.

Checklist

  • Pull Request has a meaningful title.
  • Summarised changes.
  • Pull Request is ready to merge & is not WIP.
  • Added tests / only testable interactively.
    • Make sure you add a new test if old tests do not effectively test the code changed.
  • Added documentation / opened issue to track adding documentation at a later date.

@vexx32 vexx32 added PR-Needs-Review 🔍 Let's take a closer look! Category-Tests ✅ Pertaining to tests for the module functions. labels Jul 10, 2020
@vexx32 vexx32 requested a review from steviecoaster as a code owner July 10, 2020 22:15
@vexx32 vexx32 force-pushed the Test/CodeCoverage branch from 38d7b48 to caccbf1 Compare July 10, 2020 22:17
@vexx32 vexx32 force-pushed the Test/CodeCoverage branch from caccbf1 to 6374cfc Compare July 10, 2020 22:31
vexx32 added 3 commits July 17, 2020 14:07
Manually cast nested configuration options to their respective concrete types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category-Tests ✅ Pertaining to tests for the module functions. PR-Needs-Review 🔍 Let's take a closer look!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant