Skip to content

test(tests): rename Context blocks to use BDD given/when language - #136

Merged
ndomako10 merged 1 commit into
masterfrom
test/121-bdd-context-naming
Mar 22, 2026
Merged

test(tests): rename Context blocks to use BDD given/when language#136
ndomako10 merged 1 commit into
masterfrom
test/121-bdd-context-naming

Conversation

@ndomako10

Copy link
Copy Markdown
Owner

Summary

  • Renames 12 non-conforming Context blocks across four test files to use given/when/with language per the BDD naming convention in CLAUDE.md
  • Files affected: ConvertFrom-ReceiptFileName.Tests.ps1 (5), Get-Methods.Tests.ps1 (1), Write-SyncLog.Tests.ps1 (4), Set-SubcategoryValidationXml.Tests.ps1 (2)
  • No test logic changes — rename-only

Test plan

  • CI passes — all 154 tests green
  • Invoke-Pester -Output Detailed output reads as a behavioural specification

Closes #121

Renames non-conforming Context descriptions across
ConvertFrom-ReceiptFileName, Get-Methods, Write-SyncLog, and
Set-SubcategoryValidationXml test files to use given/when/with
language so test output reads as a behavioural specification.

closes #121
@github-actions github-actions Bot added the test Test additions or changes label Mar 22, 2026
@ndomako10
ndomako10 merged commit 57a556a into master Mar 22, 2026
3 checks passed
@ndomako10
ndomako10 deleted the test/121-bdd-context-naming branch March 22, 2026 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test Test additions or changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test: BDD Context naming violations across multiple test files

2 participants