Skip to content

Enable parallel tests#14571

Merged
koppor merged 19 commits into
mainfrom
fix-tests
Dec 12, 2025
Merged

Enable parallel tests#14571
koppor merged 19 commits into
mainfrom
fix-tests

Conversation

@koppor

@koppor koppor commented Dec 11, 2025

Copy link
Copy Markdown
Member

This PR enables parallel testing.

I learned about @ResourceLock - this helps much.

Aim: High CPU usage during tests

image

Only Heisenbug - EntryComparatorTest fails in different entries:

image

Steps to test

Execute gradle task "check" in jablib

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • I manually tested my changes in running JabRef (always required)
  • I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • [/] I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@koppor koppor added dev: testing Related to tests dev: ci-cd status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers labels Dec 11, 2025
Comment thread docs/code-howtos/testing.md Outdated
subhramit
subhramit previously approved these changes Dec 12, 2025
Co-authored-by: Subhramit Basu <subhramit.bb@live.in>
@koppor koppor added this pull request to the merge queue Dec 12, 2025
Merged via the queue into main with commit 5168639 Dec 12, 2025
55 checks passed
@koppor koppor deleted the fix-tests branch December 12, 2025 20:55
Siedlerchr added a commit that referenced this pull request Dec 14, 2025
* upstream/main: (23 commits)
  Chore(deps): Bump org.mockito:mockito-core in /jablib (#14611)
  Chore(deps): Bump actions/download-artifact from 6 to 7 (#14605)
  Chore(deps): Bump com.squareup.okio:okio-jvm in /versions (#14613)
  Chore(deps): Bump DavidAnson/markdownlint-cli2-action from 21 to 22 (#14607)
  Chore(deps): Bump org.mockito:mockito-core in /versions (#14615)
  New translations jabref_en.properties (Italian) (#14596)
  Reset and Import for GroupsPreferences (#14572)
  Enable parallel tests (#14571)
  Copy over label in all cases
  Also allow adding label
  Fix workflow (#14586)
  Unassign issues on stale (#14585)
  Increase max assignments from 1 to 2
  Reduce time for initial PR to 14 days (#14580)
  New Crowdin updates (#14579)
  chore(deps): update dependency org.eclipse.jgit:org.eclipse.jgit.pgm to v7.5.0.202512021534-r (#14577)
  Fix stale closing
  Try to fix commenting on closed PRs
  Add 15 non-standard BibLaTeX entry types with .withType() method (#14534)
  New Crowdin updates (#14570)
  ...
@koppor koppor mentioned this pull request Dec 15, 2025
1 task
Siva-Sai22 pushed a commit to Siva-Sai22/jabref that referenced this pull request Dec 19, 2025
* Fix JabKit tests for parallel execution

* PosrgreServer is Autoclosable now

* Postgres workaround

* Sort-in existing tests

* Reduce number of relations at relations test

* Add TODO comment

* Some alphabet...

* Re-enable parallel tests

* Enable parallel tests

* Try to get SearchTest running with parallelism enabled

* Some more parallelism blockers

* Even more conservative

* Even more

* More conservative

* Try to fix OOM

* More conservative

* Some more...

* Add hint on lock in howto

* Update docs/code-howtos/testing.md

Co-authored-by: Subhramit Basu <subhramit.bb@live.in>

---------

Co-authored-by: Subhramit Basu <subhramit.bb@live.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev: ci-cd dev: testing Related to tests status: ready-for-review Pull Requests that are ready to be reviewed by the maintainers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants