Claude.md tweaks
This commit is contained in:
@@ -27,12 +27,13 @@ MicCheck: open source. asp.net, c#, typescript, VueJS. Manages feature flags, pr
|
|||||||
- Wrap lines at 220 characters, leave single line if fewer
|
- Wrap lines at 220 characters, leave single line if fewer
|
||||||
- Place interfaces that are implemented by a single class at the bottom of the class file. An interface with multiple implementations of an interface should be in a seperate file.
|
- Place interfaces that are implemented by a single class at the bottom of the class file. An interface with multiple implementations of an interface should be in a seperate file.
|
||||||
- Do not use tuples for return types. Prefer records or classes for multiple values
|
- Do not use tuples for return types. Prefer records or classes for multiple values
|
||||||
- Do not use `sealed` on classes-
|
- Do not use `sealed`
|
||||||
- Use `record` for data objects, `class` for objects with behavior. Avoid mutable state where possible.
|
- Use `record` for data objects, `class` for objects with behavior. Avoid mutable state where possible.
|
||||||
|
|
||||||
## Testing
|
## Testing
|
||||||
- BDD-style unit tests, end-to-end as possible, no external resources (DB, filesystem). e.g. `WhenAUserDoesSomething_ThenAThingAppears`
|
- BDD-style unit tests, end-to-end as possible, no external resources (DB, filesystem). e.g. `WhenAUserDoesSomething_ThenAThingAppears`
|
||||||
- Mock external deps with Moq
|
- Mock external deps with Moq
|
||||||
|
- Mock EntityFramework DBContexts with an extracted interface and Moq. Do not rely on InMemory provider.
|
||||||
- New features need unit tests covering as much logic as possible
|
- New features need unit tests covering as much logic as possible
|
||||||
- Any modified file: evaluate for missing test coverage-
|
- Any modified file: evaluate for missing test coverage-
|
||||||
- No "Mock" in mocked object names
|
- No "Mock" in mocked object names
|
||||||
|
|||||||
Reference in New Issue
Block a user