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.

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

Make our own low-level driver for testing #4146

Closed
AlekSi opened this issue Mar 5, 2024 · 0 comments · Fixed by #4193
Closed

Make our own low-level driver for testing #4146

AlekSi opened this issue Mar 5, 2024 · 0 comments · Fixed by #4193
Assignees
Labels
area/types Issues about data types, BSON, wire protocol, etc code/chore Code maintenance improvements

Comments

@AlekSi
Copy link
Member

AlekSi commented Mar 5, 2024

What should be done?

We should make our own low-level driver (essentially, a set of small helpers for writing binary data to TCP connection) for testing of sessions, transactions, etc.

Let's test it with one insert command with many documents and find + getMore for getting them back with a cursor.

Where?

https://github.com/FerretDB/FerretDB/tree/main/internal/driver

Definition of Done

  • code written;
  • unit tests added.
@AlekSi AlekSi added code/chore Code maintenance improvements not ready Issues that are not ready to be worked on; PRs that should skip CI labels Mar 5, 2024
@AlekSi AlekSi added the area/types Issues about data types, BSON, wire protocol, etc label Mar 5, 2024
@AlekSi AlekSi assigned AlekSi and noisersup and unassigned ferretdb-bot and AlekSi Mar 11, 2024
@AlekSi AlekSi removed the not ready Issues that are not ready to be worked on; PRs that should skip CI label Mar 12, 2024
AlekSi pushed a commit that referenced this issue Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/types Issues about data types, BSON, wire protocol, etc code/chore Code maintenance improvements
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants