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

Correct PropertyEditor exclusion mac+windows excludes under one entry #5863

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

andrew-m-leonard
Copy link
Contributor

Some corrections for existing excludes in jdk17 and jdk21 openjdk excludes for:

  • PropertyEditor * on mac and windows had two entries, and should have been under the one entry for bug JDK-8173082
  • java_awt_ScrollPane entry should have included windows-all
  • jdk17 JPEGsNotAcceleratedTest should be windows-all

@andrew-m-leonard andrew-m-leonard self-assigned this Jan 15, 2025
@sophia-guo sophia-guo merged commit 4b7bd7d into adoptium:master Jan 15, 2025
3 checks passed
andrew-m-leonard added a commit to andrew-m-leonard/aqa-tests that referenced this pull request Jan 16, 2025
smlambert pushed a commit that referenced this pull request Jan 16, 2025
* Symantec/Distrust exclude renamed for jdk-17+ (#5862)

Signed-off-by: Andrew Leonard <[email protected]>

* Correct PropertyEditor exclusion mac+windows excludes under one entry (#5863)

Signed-off-by: Andrew Leonard <[email protected]>

---------

Signed-off-by: Andrew Leonard <[email protected]>
@sophia-guo
Copy link
Contributor

sophia-guo commented Jan 16, 2025

If there are more entries only the last one takes effect, which explains why there are still failures in jdk17 mac #5852 (comment). It's better to check this duplicate exclusion in https://github.com/adoptium/aqa-tests/blob/master/scripts/disabled_tests/exclude_parser.py so it can be detected when added initially.

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

Successfully merging this pull request may close these issues.

3 participants