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.

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

[Fix] Updating WebSocket Schema and Methods #6

Draft
wants to merge 1 commit into
base: v1.3-TE1
Choose a base branch
from

Conversation

antonio-amjr
Copy link
Contributor

Overview

Fixing the WebSocket schema and some methods to support the messages received from the server.
Currently, the income messages are including _index instead of _id to update test cases, steps and suites.

Therefore, the schema were updated and the methods are now using index logic to return the correct logging.

Developer Testing

Before, the log returned errors about the WebSocket messages, not showing the steps flow correctly:

[...]
Received invalid socket message: {"type": "test_update", "payload": {"test_type": "Test Suite", "body": {"test_suite_execution_index": 0, "state": "passed", "errors": []}}}
[...]
Test Run [PASSED]
Log output in: './output_logs/test_run_Dev_CLI_Test.log'

After the update, the logs from TH during a CLI run now completes and show the results accordingly:

[...]
[1701194908.023421][3588060:3588060] CHIP:EM: Found matching exchange: 25442r, Delegate: (nil)
[1701194908.023440][3588060:3588060] CHIP:EM: Rxd Ack; Removing MessageCounter:159778699 from Retrans Table on exchange 25442r
[1701194908.023445][3588060:3588060] CHIP:IN: SecureSession[0xaaaaea9e2300]: Released - Type:2 LSID:28339
  - FirstChipToolSuite [PASSED]
Test Run [PASSED]
Log output in: './output_logs/test_run_Dev_CLI_Test.log'

@antonio-amjr antonio-amjr added the bug Something isn't working label Nov 28, 2023
@antonio-amjr antonio-amjr self-assigned this Nov 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants