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

[Feature] Indicate Endpoint under test in UI as a test runs on a given endpoint #367

Open
tcarmelveilleux opened this issue Aug 14, 2024 · 3 comments

Comments

@tcarmelveilleux
Copy link
Contributor

Feature description

Every time a test runs on an endpoint, I can't tell which endpoint is targeted in the UI. I have to remember the endpoint test_parameter set in project and if it's forgotten to update test could run on wrong endpoint.

The TH UI should always include the current endpoint target by a test (if test is applicable to an endpoint and not global).

Use Cases

Running tests on devices with multiple endpoints (e.g. 0 and 1), which is all devices.

Test Cases

No response

Additional Information

No response

@raul-marquez-csa
Copy link
Contributor

"I have to remember the endpoint test_parameter set in project": In the project config screen?
"and if it's forgotten to update test could run on wrong endpoint": Again, in the project config screen?

"The TH UI should always include the current endpoint target by a test": Then is it per project or per test?

"(if test is applicable to an endpoint and not global)": Can you give more detail on this? if global, should it also be evident in the UI?

@fabiowmm
Copy link
Contributor

fabiowmm commented Aug 30, 2024

@raju-apple @antonio-amjr I have confirmed the frontend has no means to always know which endpoint is being tested at a given point in time without support from backend/SDK. There exist dynamic endpoint tests for which the endpoint is not known in advance (see for instance project-chip/connectedhomeip#34819 (review)).

@cecille correct me if I got it wrong.

@antonio-amjr
Copy link
Contributor

antonio-amjr commented Sep 20, 2024

Hi everyone,

I'm proposing some changes on both SDK and Backend repos for this feature.
Please take a look and feel free to comment and suggest:

@raul-marquez-csa Please take a special look to those changes if possible. If you spot any necessary change, specially if it's necessary to the frontend, let me know.
The PRs above have images from tests I made of Frontend retrieving the data from the web socket and showing poorly just a for testing purpose. I not sure how the Endpoint should be presented on the UI.

@tcarmelveilleux Let me know if my take on those PRs are on track for what you requested here.

CC: @fabiowmm, @raju-apple, @cecille

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants