Labels Milestones

fix Test-CredentialStoreItem return #66

Merged

OCram85 merged 2 commits from fixTestItem into master 8 months ago

Conversation 1 Commits 2 Files Changed 19

OCram85 commented 8 months ago

Owner

📖 Summary

• Test-CredentialStoreItem now returns $false and not a terminating error if store doesn't exists.
• Updates platyPS generated docs

📑 Test Plan

💡 Select your test plan for the code changes.

• Tested via Drone.io pipeline
• Custom test
• No test plan

Details / Justification

📚 Additional Notes

• fixes #50

#### :book: Summary - `Test-CredentialStoreItem` now returns `$false` and not a terminating error if store doesn't exists. - Updates platyPS generated docs #### :bookmark_tabs: Test Plan > :bulb: Select your test plan for the code changes. - [x] Tested via Drone.io pipeline - [ ] Custom test - [ ] No test plan ##### Details / Justification <!-- Add your test details or justification for missing tests here. --> #### :books: Additional Notes - fixes #50

OCram85 added the bug label 8 months ago

OCram85 added 2 commits 8 months ago

cff5b6f04a remove terminating error

d4633d4b80 update docs

ci-bot commented 8 months ago

Collaborator

Drone.io PR Build No. #63: success

/drone/src/build/FileLinter-Results.log

✔️ No FileLinter violations in 32 files found.

/drone/src/build/Pester-Results.log

Result	Test	Duration
⚠️	Test-CredentialStore.Basic logic tests.Read a broken CredentialStore	0ms

🧪 55 Total Tests (✔️ 54 Passed ▫️🔱 1 Skipped / NotRun ▫️ ⚠️ Unknown ▫️ ❗ 0 Failed)
📑 Covered 40.83% / 75%. (🔖 791 analyzed Commands 📄 in 26 Files)

/drone/src/build/ScriptAnalyzer-Results.log

✔️ No violations found.

end.

> Drone.io PR Build No. [#63](https://drone.ocram85.com/OCram85/PSCredentialStore/63): ``success`` #### ``/drone/src/build/FileLinter-Results.log`` :heavy_check_mark: No FileLinter violations in 32 files found. #### ``/drone/src/build/Pester-Results.log`` | Result | Test | Duration | | :----: | :--- | -------: | | :warning: | `Test-CredentialStore.Basic logic tests.Read a broken CredentialStore` | *0ms* | :test_tube: **55** Total Tests (:heavy_check_mark: ``54 Passed`` :white_small_square::trident: ``1 Skipped / NotRun`` :white_small_square: :warning: ``Unknown`` :white_small_square: :heavy_exclamation_mark: ``0 Failed``) :bookmark_tabs: Covered **40.83%** / 75%. (:bookmark: ``791 analyzed Commands`` :page_facing_up: ``in 26 Files``) #### ``/drone/src/build/ScriptAnalyzer-Results.log`` :heavy_check_mark: No violations found. end.

OCram85 referenced this pull request 8 months ago

Fix Test-CredentialStoreItem returning a terminating fault, if Creden… #51

OCram85 merged commit b76668c1e7 into master 8 months ago

OCram85 deleted branch fixTestItem 8 months ago

OCram85 referenced this issue from a commit 8 months ago

fix Test-CredentialStoreItem return (#66)

continuous-integration/drone/push Build is passing

Details

continuous-integration/drone/pr Build is passing

Details

The pull request has been merged as b76668c1e7.

Sign in to join this conversation.

Reviewers

Request review

No reviewers

Labels

Apply labels

Clear labels

bug
Something is not working docs
Changes related to the docs output duplicate
This issue or pull request already exists enhancement
New feature feature
Adds a new feature help wanted
Need some help invalid
Something is wrong meta
Changes related to general project files question
More information is needed wontfix
This won't be fixed

No Label bug docs duplicate enhancement feature help wanted invalid meta question wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: OCram85/PSCredentialStore#66

Write Preview

Loading…

Cancel

Save

There is no content yet.