Skip to content
This repository has been archived by the owner on Jul 6, 2022. It is now read-only.

Migrates to Pester5+ tests #59

Open
wants to merge 48 commits into
base: master
Choose a base branch
from
Open

Migrates to Pester5+ tests #59

wants to merge 48 commits into from

Conversation

OCram85
Copy link
Owner

@OCram85 OCram85 commented Jun 28, 2022

📖 Summary

📑 Test Plan

💡 Select your test plan for the code changes.

  • Tested via Drone.io pipeline
  • Custom test
  • No test plan
Details / Justification

📚 Additional Notes

* update pwsh style in store files

* update pwsh style in item files

* update pwsh style in connection files

* update pwsh style in certificate files

* update pwsh style in private files

* update pwsh style in drone helper

* update meta

* fix pwsh style

* fix output type

* fix typo in OutputType

* update appveyor build mode

* debugging build mode

* wip

* test windows pipeline

* fix typo

* simplify drone setup

* update readme

* remove deprecated cicd setup

* update pwsh style
* Update issue templates

* debug PSScriptAnalyzer
* fix pester verbosity config

* debug ScriptAnalayzer output
@codecov
Copy link

codecov bot commented Jun 28, 2022

Codecov Report

Merging #59 (01de79f) into master (c31ee79) will increase coverage by 40.06%.
The diff coverage is n/a.

❗ Current head 01de79f differs from pull request most recent head 9a5693d. Consider uploading reports for the commit 9a5693d to get more accurate results

@@             Coverage Diff             @@
##           master      #59       +/-   ##
===========================================
+ Coverage    0.00%   40.06%   +40.06%     
===========================================
  Files          26       26               
  Lines         659      659               
===========================================
+ Hits            0      264      +264     
+ Misses        659      395      -264     
Impacted Files Coverage Δ
src/Certificate/Get-CSCertificate.ps1 15.38% <0.00%> (+15.38%) ⬆️
src/Private/Get-TempDir.ps1 25.00% <0.00%> (+25.00%) ⬆️
src/Certificate/Use-CSCertificate.ps1 37.93% <0.00%> (+37.93%) ⬆️
src/Private/Get-DefaultCredentialStorePath.ps1 38.46% <0.00%> (+38.46%) ⬆️
src/Private/Import-CSPfxCertificate.ps1 50.00% <0.00%> (+50.00%) ⬆️
src/Certificate/Import-CSCertificate.ps1 50.00% <0.00%> (+50.00%) ⬆️
src/Certificate/New-CSCertificate.ps1 63.26% <0.00%> (+63.26%) ⬆️
src/Certificate/Test-CSCertificate.ps1 65.00% <0.00%> (+65.00%) ⬆️
src/Item/Get-CredentialStoreItem.ps1 67.85% <0.00%> (+67.85%) ⬆️
src/Item/New-CredentialStoreItem.ps1 72.09% <0.00%> (+72.09%) ⬆️
... and 10 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88743e9...9a5693d. Read the comment docs.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant