Skip to content

Refactor: Better Tests #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 15 commits into from
Jul 3, 2022
Merged

Refactor: Better Tests #46

merged 15 commits into from
Jul 3, 2022

Conversation

the-avid-engineer
Copy link
Member

@the-avid-engineer the-avid-engineer commented Jul 2, 2022 •
edited
Loading

  1. Show the docker-compose file for tests in the solution explorer
  2. Clean up generic testing
  3. Convert all of the non-generic tests that directly reference TestEntity and OneToOneProjection to generic tests

@the-avid-engineer
chore: update sln to show docker-compose file from sln explorer
93cec4a
@the-avid-engineer
refactor(test): better generic tests
beaae92
@the-avid-engineer
chore: consistent test names
f30f8e4
@the-avid-engineer
temp: make sure method name is the caller's method name prefixed with…
5a08492 
… Generic_
@the-avid-engineer
feat: entity adder
1435783 
(no snapshots)
@the-avid-engineer
refactor(test): make transaction tests generic
13fe6f6
@the-avid-engineer
chore: remove method name parameter
173ac3b 
 we can just generate it! :)
@the-avid-engineer
refactor(test): make single entity transaction builder tests generic
3602c56
@the-avid-engineer
refactor(test): make entity tests generic
fec9855
@the-avid-engineer
chore: remove unused interface
7051ad8
@the-avid-engineer
chore: didn't mean to commit this
286bb7a
@the-avid-engineer
refactor(test): make try catch snapshot repository tests generic
b552de3
@the-avid-engineer
oops: missed a spot
f599aad
@the-avid-engineer
oops: missed another spot
25651fd
@the-avid-engineer
bugfix(test): need to check ShouldNotBe(default) instead of ShouldNot…
5b3a2e6 
…BeNull() due to previous commit
@the-avid-engineer the-avid-engineer marked this pull request as ready for review July 3, 2022 00:04
@the-avid-engineer the-avid-engineer merged commit 913d55e into main Jul 3, 2022
@the-avid-engineer the-avid-engineer deleted the refactor/better-tests branch July 3, 2022 00:09
@the-avid-engineer the-avid-engineer added this to the 7.0.0 milestone Jul 3, 2022
@the-avid-engineer the-avid-engineer added the diff:refactor:internal Code has been refactored in a way that will not cause compiler errors label Jul 3, 2022
@the-avid-engineer the-avid-engineer self-assigned this Jul 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@the-avid-engineer the-avid-engineer

Labels
diff:refactor:internal Code has been refactored in a way that will not cause compiler errors
Projects
None yet
Milestone
7.0.0
Development

Successfully merging this pull request may close these issues.
1 participant
@the-avid-engineer