Skip to content
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

Move all possible tests to integration tests #417

Closed
AlekSi opened this issue Mar 31, 2022 · 0 comments · Fixed by #523
Closed

Move all possible tests to integration tests #417

AlekSi opened this issue Mar 31, 2022 · 0 comments · Fixed by #523
Assignees
@w84thesun
Labels
code/chore Code maintenance improvements
Milestone
v0.2.0

Comments

@AlekSi
Copy link
Member

AlekSi commented Mar 31, 2022 •
edited
Loading

Everything that can be tested by integration tests should be tested by them. Only some weird edge cases that can't be tested by the MongoDB Go driver should remain in handlers tests. There may be no such cases, and we can remove handlers_test.go completely.
@AlekSi AlekSi self-assigned this Mar 31, 2022
@AlekSi AlekSi added the code/chore Code maintenance improvements label Mar 31, 2022
@AlekSi AlekSi added this to the v0.1.0 milestone Mar 31, 2022
AlekSi added a commit to AlekSi/FerretDB that referenced this issue Mar 31, 2022
@AlekSi
Tweak hello/ismaster/isMaster responses
17e92dc 
Refs FerretDB#417.
@AlekSi AlekSi mentioned this issue Mar 31, 2022
Merged
AlekSi added a commit that referenced this issue Apr 1, 2022
@AlekSi
Tweak hello/ismaster/isMaster responses (#418)
8186b0a 
Refs #417.
@AlekSi AlekSi mentioned this issue Apr 1, 2022
Merged
AlekSi added a commit that referenced this issue Apr 1, 2022
@AlekSi
Add tests for getLog (#421)
0ec3341 
Refs #417.
@AlekSi AlekSi modified the milestones: v0.1.0, v0.1.1 Apr 4, 2022
@AlekSi AlekSi changed the title Move more tests to integration tests Move all possible tests to integration tests Apr 4, 2022
AlekSi added a commit to AlekSi/FerretDB that referenced this issue Apr 5, 2022
@AlekSi
Add test stub for bitwise operators
4d5e275 
Refs FerretDB#442.
Refs FerretDB#417.
This was referenced Apr 5, 2022
Merged
Closed
AlekSi added a commit that referenced this issue Apr 5, 2022
@AlekSi
Add test stub for bitwise operators (#443)
c56f3ec 
Refs #442.
Refs #417.
AlekSi added a commit to AlekSi/FerretDB that referenced this issue Apr 5, 2022
@AlekSi
Add tests for collections create and drop
dcb0ecf 
Refs FerretDB#417.
@AlekSi AlekSi mentioned this issue Apr 5, 2022
Merged
AlekSi added a commit that referenced this issue Apr 5, 2022
@AlekSi
Add tests for collections create and drop (#444)
35140b3 
Refs #417.
AlekSi added a commit to AlekSi/FerretDB that referenced this issue Apr 5, 2022
@AlekSi
Add tests for more diagnostic commands
ef59988 
Refs FerretDB#417.
Refs FerretDB#447.
@AlekSi AlekSi mentioned this issue Apr 5, 2022
Merged
AlekSi added a commit that referenced this issue Apr 5, 2022
@AlekSi
Add tests for more diagnostic commands (#448)
1c85998 
Refs #417.
Refs #447.
@AlekSi AlekSi mentioned this issue Apr 6, 2022
Merged
AlekSi pushed a commit that referenced this issue Apr 6, 2022
@ribaraka
Transfer existing comparison tests (#445)
e95eb88 
Refs #417.
AlekSi added a commit to AlekSi/FerretDB that referenced this issue Apr 6, 2022
@AlekSi
Move getParameter tests
151d19d 
Refs FerretDB#417.
@AlekSi AlekSi mentioned this issue Apr 6, 2022
Merged
AlekSi added a commit that referenced this issue Apr 6, 2022
@AlekSi
Move getParameter tests (#450)
5db705e 
Refs #417.
@AlekSi AlekSi assigned w84thesun and unassigned AlekSi Apr 12, 2022
@AlekSi AlekSi modified the milestones: v0.1.1, v0.2.0 Apr 18, 2022
@w84thesun w84thesun mentioned this issue Apr 25, 2022
Merged
AlekSi pushed a commit that referenced this issue Apr 27, 2022
Move all handler test to integration tests (#523)
f59707f 
Closes #417.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@w84thesun w84thesun

Labels
code/chore Code maintenance improvements
Projects
None yet
Milestone
v0.2.0
Development

Successfully merging a pull request may close this issue.

Move all handler test to integration tests w84thesun/FerretDB
2 participants
@AlekSi @w84thesun