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

common_test: Color failing line in test suite HTML source file #7917

Merged

Conversation

sverker
Copy link
Contributor

@sverker sverker commented Nov 30, 2023

For test suite source HTML files like my_test_suite.src.html, if the URL has an anchor part at the end (# with line number), color that line to make it easier to find on the screen.

Here is an example how it looks: ct_smoke_test_suite.src.html#432

If the URL has an anchor part at the end (# with line number),
color that line to make it easier to find on the screen.
@sverker sverker requested a review from u3s November 30, 2023 14:03
@sverker sverker self-assigned this Nov 30, 2023
Copy link
Contributor

github-actions bot commented Nov 30, 2023

CT Test Results

    2 files    58 suites   1h 25m 39s ⏱️
451 tests 438 ✔️ 12 💤 1
486 runs  470 ✔️ 15 💤 1

For more details on these failures, see this check.

Results for commit b90f1aa.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@rickard-green rickard-green added the team:PS Assigned to OTP team PS label Dec 4, 2023
@sverker sverker added enhancement testing currently being tested, tag is used by OTP internal CI labels Dec 4, 2023
@IngelaAndin IngelaAndin added the team:VM Assigned to OTP team VM label Dec 5, 2023
@sverker sverker force-pushed the sverker/common_test/color-linked-source-line branch from b90f1aa to 30e0efd Compare December 7, 2023 17:09
@sverker sverker merged commit 9cbb1fe into erlang:master Dec 7, 2023
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement team:PS Assigned to OTP team PS team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants