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

Neyberson/protocol verify json #163

Merged
merged 35 commits into from
Nov 13, 2024
Merged

Neyberson/protocol verify json #163

merged 35 commits into from
Nov 13, 2024

Conversation

ayjayt
Copy link
Collaborator

@ayjayt ayjayt commented Nov 13, 2024

Add some testing and more errors

neyberson and others added 30 commits October 8, 2024 17:45
@neyberson
Add type verifications in protocol.py:verify_json()
de29838
@neyberson
Merge branch 'main' into neyberson/protocol_verify_json
623bfa8
@neyberson
Create MessageTypeError and MissingKeyError in protocol.py
afc1f9e
@neyberson
Use MessageTypeError in protocol.py:verify_json()
1e740e5
@neyberson
Use MissingKeyError in verify_json()
28d5b02
@neyberson
Format protocol.py
8f6453d
@neyberson
Rename object to obj in MissingKeyError
a7a6a15
@neyberson
Improve conditionals and MissingKeyError use in verify_json()
c149b48
@neyberson
Add s in string of MissingKeyError
2710166
@neyberson
Improve conditionals to verify keys in verify_json()
4059f64
@neyberson
Simplify verifications with loop in verify_json()
40964ce
@neyberson
Improve test_browser_write_json() to verify errors
dd902e8
@neyberson
Improve tests about send_command()
040e3a3
@neyberson
Rename some test_send_command() adding the type obj
a5985d1
@neyberson
Format some tests
b8d9c08
@neyberson
Add comment in test_browser_write_json()
180b544
@neyberson
Improve comment in test_browser_write_json()
33aa72c
@ayjayt
Merge branch 'main' into neyberson/improve_tests
10cdad8
@ayjayt
Disallow writes after browser closed
3efc8d5
@ayjayt
Disallow some await after browser close
ca75e04
@ayjayt
Add todo comment
59323c1
@ayjayt
Decomment valid error
aa4d5a0
@ayjayt
Merge branch 'neyberson/improve_tests' into neyberson/protocol_verify…
bfbb42c 
…_json
@neyberson
Improve the Errors catch with pytest
48aeca1
@neyberson
Reorganize and format test_browser.py
1d6d213
@neyberson
Reorganize test_session.py
0c4b1e4
@neyberson
Reorganize and format test_tab.py
f08c0f4
@neyberson
Improve MessageTypeError with conditionals
48ac42f
@neyberson
Format protocol.py
e0d0ecc
@neyberson
Update the match of Errors catch in tests for protocol.py update
9e1f6ae
@ayjayt ayjayt merged commit 263ab5b into main Nov 13, 2024
4 checks passed
@ayjayt ayjayt deleted the neyberson/protocol_verify_json branch November 13, 2024 05:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ayjayt @neyberson