-
Notifications
You must be signed in to change notification settings - Fork 9
Implement register_listener and unregister_listener functionality #75
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
Closed
arakabCL
wants to merge
50
commits into
eclipse-uprotocol:main
from
uProtocol-cmu-team-Luna:listener-wildcard-removal-pr
Closed
Changes from 47 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
c76c856
looking through noellas work
kwarraich aec4d04
more of what noella has done
kwarraich 3ec7d93
fixed all the errors reguarding the cargo test
kwarraich c83ca8d
seperated out the test cases from lib.rs and then made adjustment so …
kwarraich aa81d20
added custom header case & muliple messages in seq case
kwarraich 89596de
checks that the umessage is correctly parsed into the transmission da…
kwarraich fff54b7
tested sending w empty payload, no payload, and max size payload
kwarraich d97041c
included the multiple concurent messages
kwarraich ee5116f
adding all files so can merge into one shared fork
kwarraich 7ee3b66
Merge pull request #1 from kwarraich/testing_send_reg_unreg
kwarraich a525b97
tests run and there are not issues anymore which prevent them from ru…
kwarraich b78e66f
Merge pull request #2 from uProtocol-cmu-team-Luna/fixingInitalErrors
kwarraich 761c7d7
skeleton code for receiver testing
vidishac2004 2a4e050
receiver test with compilation errors
vidishac2004 2e2ba85
modifed send test cases for muli sends, implemented send(), written a…
catappreciationhours2 7c43ba4
modified compute service name to work correctly with the service name…
catappreciationhours2 b83872b
updated encode uuri segments to handle type and instance from id sepa…
catappreciationhours2 a77699f
receiver.rs detected by cargo test
vidishac2004 94ce300
test cases to verify conversion from uri to iceoryx2 addressing schem…
catappreciationhours2 32afab3
fixed logger error
vidishac2004 86439c6
feat: Implement listener registration and unregistration
arakabCL 2c75b13
Merge remote-tracking branch 'origin/main' into receiver
vidishac2004 33f3303
mostly fixing the failing tests, only one test the wildcard is failin…
kwarraich 8b8b170
creating it so that it sends and recives on seperate terminals
kwarraich 5e79a23
Merge pull request #3 from uProtocol-cmu-team-Luna/twoTerminals
kwarraich 418af5b
made the transmission data completely genetic
kwarraich 6e8524c
adding a test case to make sure mockUListener works
kwarraich 1a0d554
testing unregister in receiver
vidishac2004 ed4cb86
added a test case
kwarraich 73456ff
adding some more tests and pushing the mockUListener into the test block
kwarraich 3aa4582
fixed uuri wildcard logic and made RawBytes compatible with iceoryx2 …
catappreciationhours2 e36dc08
modified cargo.toml to make test-utils availale in dependencies
catappreciationhours2 bf2f608
these tests pass without wildcard URIs so replaced Uuri::any with the…
vidishac2004 2f2e4ee
adjusted so the ternimals would send the data and properly with the n…
kwarraich 65768ab
adjusted teh reciving end so it would print in decimal
kwarraich fc0e8e4
Merge pull request #4 from uProtocol-cmu-team-Luna/makingTransmission…
arakabCL c3fa305
Merge branch 'main' into tests_without_wildcards
arakabCL 49fe222
Merge pull request #5 from uProtocol-cmu-team-Luna/tests_without_wild…
arakabCL 6aa08f6
Removed wildcard support from compute listener matching logic.
arakabCL f74cc6f
Completed wildcard removal from compute listener with tests.
arakabCL e367b19
Updated register with deserialized attributes handling
arakabCL 533e034
Complete wildcard removal and listener functionality improvements
arakabCL d815582
Implement register_listener and unregister_listener functionality
arakabCL 8e9b311
fixing workflow error
kwarraich 4a491ca
Merge branch 'main' into listener-wildcard-removal-pr
kwarraich d33aab8
Update lib.rs
vidishac2004 84c6d60
Update lib.rs
kwarraich 265fba1
Update src/lib.rs
vidishac2004 d453dec
Remove .DS_Store and add it to .gitignore
vidishac2004 9376043
removed rawbytes
kwarraich File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
|
PLeVasseur marked this conversation as resolved.
Outdated
|
Binary file not shown.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,20 +1,20 @@ | ||
| name: Rust Project CI | ||
|
|
||
| on: | ||
| push: | ||
| branches: [ "main" ] | ||
| pull_request: | ||
| branches: [ "main" ] | ||
|
|
||
| jobs: | ||
| build_and_test: | ||
| runs-on: ubuntu-latest | ||
|
|
||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Build | ||
| run: cargo build --verbose | ||
|
|
||
| - name: Run tests | ||
| name: Rust Project CI | ||
| on: | ||
| push: | ||
| branches: [ "main" ] | ||
| pull_request: | ||
| branches: [ "main" ] | ||
| jobs: | ||
| build_and_test: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - name: Build | ||
| run: cargo build --verbose | ||
| - name: Run tests | ||
| run: cargo test --verbose | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. missing EOL |
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| /target | ||
| /target |
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.