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 AggregatedTestResultPublisherTest on 2.434+ #591

Merged
merged 1 commit into from
Dec 14, 2023
Merged

Conversation

timja
Copy link
Member

@timja timja commented Dec 14, 2023

Fixes #588

There's now an SVG in the link with the text Unstable. The previous API does a value comparison which isn't possible to make work without including the whole svg inline exactly the same.

Checking the link destination does the same thing here.

@timja timja requested a review from a team as a code owner December 14, 2023 09:01
@timja timja added the tests label Dec 14, 2023
@timja timja enabled auto-merge (squash) December 14, 2023 09:01
@timja timja merged commit fc2e5ef into master Dec 14, 2023
14 of 15 checks passed
@timja timja deleted the fix-test-on-new-core branch December 14, 2023 09:27
MarkEWaite added a commit to jenkinsci/bom that referenced this pull request Dec 14, 2023
https://github.com/jenkinsci/junit-plugin/releases/tag/1252.vfc2e5efa_294f
is the JUnit plugin release that fixes the test.

Special thanks to Tim Jacomb for fixing the test in
jenkinsci/junit-plugin#591

jenkinsci/junit-plugin#588 is the issue report
for the JUnit plugin tests that depend on the icon based build status
display.

jenkinsci/jenkins#8705 is the Jenkins core
pull request that switched from using icon based build status to symbol
based build status.  That change also fixes a layout issue in the Safari
web browser.

This reverts commit cb2376e.

Author: Mark Waite <mark.earl.waite@gmail.com>
Date:   Tue Nov 28 22:00:27 2023 -0700
github-actions bot pushed a commit to jenkinsci/bom that referenced this pull request Dec 14, 2023
…c2e5efa_294f in /bom-weekly (#2755)

* Bump org.jenkins-ci.plugins:junit in /bom-weekly

Bumps [org.jenkins-ci.plugins:junit](https://github.com/jenkinsci/junit-plugin) from 1240.vf9529b_881428 to 1252.vfc2e5efa_294f.
- [Release notes](https://github.com/jenkinsci/junit-plugin/releases)
- [Commits](https://github.com/jenkinsci/junit-plugin/commits)

---
updated-dependencies:
- dependency-name: org.jenkins-ci.plugins:junit
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Revert "Exclude AggregatedTestResultPublisherTest"

https://github.com/jenkinsci/junit-plugin/releases/tag/1252.vfc2e5efa_294f
is the JUnit plugin release that fixes the test.

Special thanks to Tim Jacomb for fixing the test in
jenkinsci/junit-plugin#591

jenkinsci/junit-plugin#588 is the issue report
for the JUnit plugin tests that depend on the icon based build status
display.

jenkinsci/jenkins#8705 is the Jenkins core
pull request that switched from using icon based build status to symbol
based build status.  That change also fixes a layout issue in the Safari
web browser.

This reverts commit cb2376e.

Author: Mark Waite <mark.earl.waite@gmail.com>
Date:   Tue Nov 28 22:00:27 2023 -0700

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Mark Waite <mark.earl.waite@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AggregatedTestResultPublisherTest fails with Jenkins 2.434
1 participant