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

BugFix: issue resulting from iterating over dictionary while deleting keys in realtime_service #18

Closed
wants to merge 25 commits into from

Conversation

m29h
Copy link
Collaborator

@m29h m29h commented Feb 5, 2023

further improved test coverage now at 94%

perfalle and others added 25 commits November 8, 2022 16:50
update to pocketbase version 0.8.0-rc1

fixed flake8 errors,
but there are still remaining errors:
F401 ... imported but unused
E501 line too long (... > 79 characters)
E722 do not use bare 'except'
* Switch from JSON to multipart file encoding for file upload

File upload is detected when body data contains a value of class FileUpload
Remaining JSON is converted to FormData
Enitre message is sent as multipart

* Switch from JSON to multipart file encoding for file upload (vaphes#6)

File upload is detected when body data contains a value of class FileUpload
Remaining JSON is converted to FormData
Enitre message is sent as multipart
* Switch from JSON to multipart file encoding for file upload (vaphes#6)

File upload is detected when body data contains a value of class FileUpload
Remaining JSON is converted to FormData
Enitre message is sent as multipart

* codebase reformat with black
* Remove "@" chars that led to empty collectionId, collectionName and expand

* Make load method more generic
add pytest tests testing against an actual pocketbase instance
tests are skipped if no fresh instance on http://127.0.0.1:8090 is found
- real-world API integration testing against an live instance of pocketbase
- increase test coverage
- api tested against pocketbase v0.11.3
Minor fixes from extended unit and  API integration testing
Code cleanup, improve test coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants