Fix/warnings revived #310
Open
Fix/warnings revived #310
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Added doc comments to all public fields (phew!) - Added nonempty constructors to entities where a field could be null after creation (like strings). > Some of these constuctors throw an error if app attempted to instantiate with empty values. > Entities still have 0-argument constructors to work with DbSet. - Some entities now have simplified constructors. > For example, the new Purchase constructor sets 8 fields using 3 parameters. > The new Ticket constructor sets 5 fields with 1 parameter. - Fixed tests as a result of breaking them. > Enable `nullable` in test project > Most should be more robust by avoiding magic strings > Test object getters, like a `testUser` getter, reduces code duplication > Redundant fields in test objects have been removed while ensuring the tests still pass. - Removed `LoginAttempt` entity as stated in a TODO comment - Removed pragma warning ignores - Removed unused usings
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR cherry-picks the xml comments created in #183, reduces warnings by marking properties as required, instead of using constructors.
Also removes unused using statements, and other low hanging warnings.
Note, there is still a fair bit of warnings since the tests haven't been refactored in this PR. I think the best way to address the warnings from there, would be to migrate them to use the builders, though that seems out of scope of this PR.