Skip to content

Clean up test code#10658

Merged
Siedlerchr merged 4 commits into
mainfrom
enable-test-cleanup
Nov 21, 2023
Merged

Clean up test code#10658
Siedlerchr merged 4 commits into
mainfrom
enable-test-cleanup

Conversation

@koppor

@koppor koppor commented Nov 21, 2023

Copy link
Copy Markdown
Member

This enables rewrite of tests using OpenRewrite's JUni5 "cleanup assertions" recipe: https://docs.openrewrite.org/recipes/java/testing/junit5/cleanupassertions

Mandatory checks

  • Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
  • Tests created for changes (if applicable)
  • Manually tested changed features in running JabRef (always required)
  • Screenshots added in PR description (for UI changes)
  • Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
  • Checked 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 to the documentation repository.

@koppor koppor added the dev: code-quality Issues related to code or architecture decisions label Nov 21, 2023
@Siedlerchr Siedlerchr added this pull request to the merge queue Nov 21, 2023
Merged via the queue into JabRef:main with commit 75107ab Nov 21, 2023
@Siedlerchr Siedlerchr deleted the enable-test-cleanup branch November 21, 2023 08:38
Siedlerchr added a commit to xuanan20020/jabref that referenced this pull request Nov 21, 2023
* upstream/main: (24 commits)
  Clean up test code (JabRef#10658)
  Bump org.openrewrite.rewrite from 6.4.0 to 6.5.4 (JabRef#10650)
  Bump com.puppycrawl.tools:checkstyle from 10.12.4 to 10.12.5 (JabRef#10654)
  Bump com.fasterxml.jackson.dataformat:jackson-dataformat-yaml (JabRef#10653)
  Bump com.github.Dansoftowner:jSystemThemeDetector from 3.6 to 3.8
  Remove comments - don't work on forks
  Update journal abbreviation lists (JabRef#10645)
  Use clparse (instead of heylogs) (JabRef#10641)
  Update CSL styles (JabRef#10642)
  Added parent field to Hayagriva YAML export (JabRef#10633)
  Bump org.fxmisc.richtext:richtextfx from 0.11.1 to 0.11.2 (JabRef#10637)
  Bump io.github.classgraph:classgraph from 4.8.163 to 4.8.164 (JabRef#10634)
  Bump org.junit.platform:junit-platform-launcher from 1.10.0 to 1.10.1 (JabRef#10635)
  Bump org.mockito:mockito-core from 5.6.0 to 5.7.0 (JabRef#10638)
  Bump com.tngtech.archunit:archunit-junit5-engine from 1.1.0 to 1.2.0 (JabRef#10636)
  add some more spaces
  Fix CHANGELOG.md
  Remove SearchListener
  Refactor method name to reflect usage
  Revert "Remove filtering on query in maintable"
  ...
Siedlerchr added a commit that referenced this pull request Nov 26, 2023
* upstream/main:
  App semantic scholar api key to properties (#10656)
  Replace cache: gradle with "odern" gradle-build-action (#10657)
  Clean up test code (#10658)
  Bump org.openrewrite.rewrite from 6.4.0 to 6.5.4 (#10650)
  Bump com.puppycrawl.tools:checkstyle from 10.12.4 to 10.12.5 (#10654)
  Bump com.fasterxml.jackson.dataformat:jackson-dataformat-yaml (#10653)
  Bump com.github.Dansoftowner:jSystemThemeDetector from 3.6 to 3.8
  Remove comments - don't work on forks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dev: code-quality Issues related to code or architecture decisions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants