-
Notifications
You must be signed in to change notification settings - Fork 16
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
Add Rust Test Agent #23
Merged
Merged
Commits on Apr 12, 2024
-
* Add changes to TCK to support Rust Test Agent for Register_and_Send and register_and_unregister Co-Authored-By: sagarsshah <157176449+sagarsshah@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for eb0531e - Browse repository at this point
Copy the full SHA eb0531eView commit details
Commits on Apr 24, 2024
-
Sagar and I have taken in the comments from Pete's PR review and addressed most of them in this commit. Mainly: * Refactored utils.rs * Removed extension trait for utransport * Aligned names to rust coding standard * Ran rustfmt on all files * Ran clippy on all files: we addressed most of the clippy issues, but some will require more intense refactoring * Minor fixes and idiomatic cleanup Co-Authored-By: sagarsshah <157176449+sagarsshah@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 832396e - Browse repository at this point
Copy the full SHA 832396eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 05c7823 - Browse repository at this point
Copy the full SHA 05c7823View commit details -
Need to remove duplicate code that is no longer used.
Configuration menu - View commit details
-
Copy full SHA for d380974 - Browse repository at this point
Copy the full SHA d380974View commit details -
Configuration menu - View commit details
-
Copy full SHA for a3cef51 - Browse repository at this point
Copy the full SHA a3cef51View commit details -
Aligning the utransport_socket with recommendations from Pete. Instead of importing the utransport_socket.rs, we are now making it into a library crate.
Configuration menu - View commit details
-
Copy full SHA for 62f4fc8 - Browse repository at this point
Copy the full SHA 62f4fc8View commit details
Commits on May 3, 2024
-
We went through the large majority of issues presented in the PR Review, including: * removing almost all expects and unwraps * changing many method names to match standards * reduce size of many methods through simplification and reorganization Notes for reviewer: * One 'expect' remains [main.rs, line 71]. would appreciate advice on how to remove that * One clippy error remains. It's regarding holding the socket across async calls. * A few unwraps remain, but those are mostly handled except for one regarding commstatus Co-Authored-By: sagarsshah <157176449+sagarsshah@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 67bc751 - Browse repository at this point
Copy the full SHA 67bc751View commit details
Commits on May 9, 2024
-
Configuration menu - View commit details
-
Copy full SHA for e78503f - Browse repository at this point
Copy the full SHA e78503fView commit details -
Update for comments, add rust to spawn
Fix last clippy issue related to holding mutex across awaits (testagent.rs, line 225--) Remove last expect for commstatus Change wrapperu_message var name to wrapper_umessage Add rust to new spawning paradigm in tck_step_implementations.py
Configuration menu - View commit details
-
Copy full SHA for d20bcca - Browse repository at this point
Copy the full SHA d20bccaView commit details
Commits on May 10, 2024
-
* Change dbg! to using log debug! crate. * Changed "loop" in test agent to "while" * Remove some comments
Configuration menu - View commit details
-
Copy full SHA for 82f8d38 - Browse repository at this point
Copy the full SHA 82f8d38View commit details
Commits on May 13, 2024
-
* update licenses * remove get_ustatus_code * add err_string to reuse error msgs * change signature of parse_uauthority * remove commented code * formatting on lib.rs * remove panic references since doesn't panic Co-Authored-By: Pete LeVasseur <11622119+plevasseur@users.noreply.github.com>
Configuration menu - View commit details
-
Copy full SHA for 02e09c9 - Browse repository at this point
Copy the full SHA 02e09c9View commit details
Commits on May 14, 2024
-
* Change specialfields to default * Add "not implemented" to notification type * Some fixes to debug msgs and comments
Configuration menu - View commit details
-
Copy full SHA for 8c8762a - Browse repository at this point
Copy the full SHA 8c8762aView commit details
Commits on May 15, 2024
-
Remove initialization of special_fields
* Remove special_fields from the uattributes deserializer
Configuration menu - View commit details
-
Copy full SHA for cb89638 - Browse repository at this point
Copy the full SHA cb89638View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.