-
Notifications
You must be signed in to change notification settings - Fork 2
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
8 communication tests #39
8 communication tests #39
Conversation
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
README.md
Outdated
|
||
### Smart Home Mashup | ||
|
||
See the mashup's [readme](./mashups//smart-home/README.md). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the double forward slash correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here.
@@ -0,0 +1,66 @@ | |||
"use strict"; | |||
/******************************************************************************** | |||
* Copyright (c) 2023 Contributors to the Eclipse Foundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Copyright (c) 2023 Contributors to the Eclipse Foundation | |
* Copyright (c) 2024 Contributors to the Eclipse Foundation |
Since it is new?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here.
@@ -0,0 +1,216 @@ | |||
"use strict"; | |||
/******************************************************************************** | |||
* Copyright (c) 2023 Contributors to the Eclipse Foundation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* Copyright (c) 2023 Contributors to the Eclipse Foundation | |
* Copyright (c) 2024 Contributors to the Eclipse Foundation |
There are many more...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here.
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
Signed-off-by: Hasan Eroglu <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR is ok with some changes needed in a follow up PR:
- fixtures usage for flask tests
- Summarizing all the tests in the end. Currently, I need to scroll up to see if something has failed
- Node v22 does not seem to work
Closes #8, Closes #23.
This PR MUST be merged after #38 is merged.
Note: MQTT tests are problematic since the thing in the repository has properties with "readproperty" operations. We need to either change the logic or fix issues node-wot #980 and node-wot #1241.