Skip to content

Commit

Permalink
Update conventions.md
Browse files Browse the repository at this point in the history
  • Loading branch information
potatoqualitee committed Oct 27, 2024
1 parent 6ae5f10 commit 37570d9
Showing 1 changed file with 16 additions and 10 deletions.
26 changes: 16 additions & 10 deletions .aider/prompts/conventions.md
Original file line number Diff line number Diff line change
Expand Up @@ -172,13 +172,19 @@ Describe "Get-DbaDatabase" -Tag "IntegrationTests" {
}
```

## Additional instructions

- DO NOT use `$MyInvocation.MyCommand.Name` to get command names
- DO NOT use the old `knownParameters` validation approach
- DO NOT include loose code outside of proper test blocks
- DO NOT remove comments like "#TestConfig.instance3" or "#$TestConfig.instance2 for appveyor"
- DO NOT use $_ DO use $PSItem instead
- Parameter validation is ALWAYS tagged as a Unit Test
- DO NOT change $results.Status.Count to $results.Count -- that secondary column is required for accurate counting
- NO trailing spaces
## Additional Instructions

### Test Structure
- Parameter validation must be tagged as Unit Test
- No loose code outside of proper test blocks
- Maintain all instance reference comments (#TestConfig.instance3, etc.)

### Syntax Requirements
- Use $PSItem instead of $_
- No trailing spaces
- Use $results.Status.Count for accurate counting

### Must Not Use
- $MyInvocation.MyCommand.Name for command names
- Old knownParameters validation approach
- Assumed parameter names - match original tests exactly

0 comments on commit 37570d9

Please sign in to comment.