Skip to content
This repository was archived by the owner on Jul 25, 2024. It is now read-only.

Improve testcase show info #1141

Merged
merged 2 commits into from
Jul 16, 2024

Conversation

pascaleberhard0se
Copy link
Contributor

This proposal adds the possibility to jump directly to the line of the testcase in the Lava log from the "Tests" tab by clicking on the "Test Info" modal view. It makes the analysis of an issue easier.

For this, it fixes the modal "Test Info" view from the build "Tests" tab, and adds the URL of the Lava log of the testcase in it.

'Test Info' modal view is ok when called from 'test_history' view
but is empty when called from 'tests' view.

Fix index of the test data in status[] when calling modal view.

Signed-off-by: Pascal Eberhard <pascal.eberhard@se.com>
There is currently no simple way to access Lava testcase log from the
tests view and from the test history view.

Add the URL pointing to the first line of the Lava log of the testcase
in the 'Test Info' modal view.

Signed-off-by: Pascal Eberhard <pascal.eberhard@se.com>
@mwasilew
Copy link
Contributor

+1
very useful feature.

Copy link
Collaborator

@chaws chaws left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @pascaleberhard0se , nice feature! LGTM

@chaws
Copy link
Collaborator

chaws commented Jul 16, 2024

Tests failed due to empty data being passed to LAVABackend in https://github.com/Linaro/squad/blob/master/test/ci/backend/test_lava.py. That's a quick fix. I'm merging this PR now and will fix tests before rolling out a new release

@chaws chaws merged commit dea1698 into Linaro:master Jul 16, 2024
2 of 7 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants