🪲 [Test]: Add test for removing all contexts

[Marius Storhaug (MariusStorhaug)]

Description

Type of change

• 📖 [Docs]
• 🪲 [Fix]
• 🩹 [Patch]
• ⚠️ [Security fix]
• 🚀 [Feature]
• 🌟 [Breaking change]

Checklist

• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas

[Copilot]

Pull Request Overview

Adds a Pester test to verify that piping Get-Context into Remove-Context clears all contexts in a given vault.

• Introduces setup for multiple contexts in VaultA
• Asserts pre-removal state and exercises the removal pipeline
• Verifies that no contexts remain after running Remove-Context

Comments suppressed due to low confidence (1)

tests/Context.Tests.ps1:239

• The test description mentions removing contexts specifically from VaultA, but the pipeline does not filter by vault. Consider updating the description or specifying the vault in the pipeline to align the test behavior with its title.

        It 'Get-Context | Remove-Context - Should remove all contexts from VaultA' {

[Copilot]

The removal pipeline does not specify -Vault 'VaultA', so it may remove contexts from other vaults or miss targeting VaultA. Change it to Get-Context -Vault 'VaultA' | Remove-Context to scope the operation correctly.

Suggested change

	{ Get-Context | Remove-Context } | Should -Not -Throw
	{ Get-Context -Vault 'VaultA' | Remove-Context -Vault 'VaultA' } | Should -Not -Throw

[Copilot]

The post-removal assertion runs Get-Context without a vault filter. To verify that VaultA is emptied, use Get-Context -Vault 'VaultA' | Should -BeNullOrEmpty.

Suggested change

	$contextsAfter = Get-Context
	$contextsAfter = Get-Context -Vault 'VaultA'