Added suite connector integration test #45
Merged
Conversation
This file contains 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
e-grigorov
changed the title
e-grigorov
suggested changes
Sep 16, 2022
refi64
reviewed
Oct 7, 2022
refi64
reviewed
Oct 19, 2022
dimiter-todorov-georgiev
force-pushed
the
int_test
branch
2 times, most recently
from
939f20d to
67a4cbd
Compare
e-grigorov
suggested changes
Oct 31, 2022
dimiter-todorov-georgiev
force-pushed
the
int_test
branch
from
c6a8e3c to
09a6dde
Compare
e-grigorov
reviewed
Nov 7, 2022
e-grigorov
reviewed
Nov 7, 2022
refi64
reviewed
Nov 9, 2022
hristobojilov
requested changes
Nov 14, 2022
hristobojilov
approved these changes
Nov 15, 2022
e-grigorov
suggested changes
Nov 16, 2022
e-grigorov
reviewed
Nov 16, 2022
e-grigorov
reviewed
Nov 16, 2022
hristobojilov
requested changes
Nov 24, 2022
e-grigorov
reviewed
Nov 24, 2022
e-grigorov
suggested changes
Nov 25, 2022
hristobojilov
approved these changes
Nov 25, 2022
e-grigorov
approved these changes
Nov 25, 2022
Signed-off-by: Nikola Zhechev <[email protected]>
Signed-off-by: Nikola Zhechev <[email protected]>
Signed-off-by: Nikola Zhechev <[email protected]>
Signed-off-by: Nikola Zhechev <[email protected]>
…ngle address property Signed-off-by: Nikola Zhechev <[email protected]>
New Eclipse Kanto license header used Renamed featureName to featureID StatusOK used for command response readyUntil now expected to be "9999-12-31T23:59:59Z" Signed-off-by: Nikola Zhechev <[email protected]>
…atus to be updated Signed-off-by: Nikola Zhechev <[email protected]>
added 27 commits
November 25, 2022 15:17
…back - Rename Hono environment variables to DeviceRegistryApi - Rename Ditto environment variables to DigitalTwinApi Signed-off-by: Dimiter Georgiev <[email protected]>
- Renamed mqtt package import to MQTT - Removed unneeded call to .Fail() - Properly failing the test when responses are unable to be sent to the backend - Refactored getPortOrDefault() Signed-off-by: Dimiter Georgiev <[email protected]>
- Fix error handling when receiving an unknown command. This can happen for reasons which are unrelated to this test, so instead of failign the test, log the unrecognized command and continue the test. Signed-off-by: Dimiter Georgiev <[email protected]>
- Use the Ditto library where appropriate, removing custom structs - Use constructors where required - Use proper error messages with require.NoError() where they were missing Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
…e test goroutine Signed-off-by: Dimiter Georgiev <[email protected]>
- Use 0 as default offset when comparing connection status' readySince to now - Added a sleep time between attempts to read connection status, defaulting to 2 seconds - Changed command test code to properly use the Message struct from the ditto library Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
…d of errors Signed-off-by: Dimiter Georgiev <[email protected]>
… integration test code Signed-off-by: Dimiter Georgiev <[email protected]>
…sWSMessages Signed-off-by: Dimiter Georgiev <[email protected]>
…ions Signed-off-by: Dimiter Georgiev <[email protected]>
…ig() Signed-off-by: Dimiter Georgiev <[email protected]>
…64-bit math to avoid overflow Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
- Switch to using the Digital Twin REST API for sending a command to the feature - Minor fixes Signed-off-by: Dimiter Georgiev <[email protected]>
- Removed unneeded lines - Use the DITTO API for checking of the expected topic Signed-off-by: Dimiter Georgiev <[email protected]>
…test logic - Removed variables which come from the common code - Use the new API functions provided by the common logic Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
- Update NOTICE.md with the updated version of the uuid library. - Remove the extra check (and relevant code) for sending an answer to an operation message. - Use the util function ExecuteOperation. - Compare time.Time using time.Equal. - Fix the initialization of the variable now. - Move the initialization of a variable closer to where it's used, reducing its scope. - Stop using GetFeatureOutboxMessagePath to obtain the inbox. Signed-off-by: Dimiter Georgiev <[email protected]>
…lue and GetFeatureInboxMessagePath. Signed-off-by: Dimiter Georgiev <[email protected]>
Signed-off-by: Dimiter Georgiev <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
[#44] Add suite connector integration test
Signed-off-by: Nikola Zhechev [email protected]