Modify ↓

Opened 4 weeks ago

Last modified 3 days ago

#24924 new enhancement

[patch] Remove Junit 4 last remains

Reported by: gaben Owned by: team
Priority: normal Milestone: 26.11
Component: Unit tests Version:
Keywords: junit4 Cc:

Description

While working on some patches, I noticed that IntelliJ still has ancient Junit4 files on the classpath. I therefore started removing them completely from the core.

I only focused on the core, the core plugins still have to be checked.

FYI: There is an existing ticket for Junit 6 migration (#24921).

Attachments (1)

24924.patch​ (82.9 KB ) - added by gaben 3 days ago.

Download all attachments as: .zip

Change History (3)

comment:1 by gaben, 3 days ago

Also #23090 should be fixed with the changes.

by gaben, 3 days ago

Attachment: 24924.patch​ added

comment:2 by gaben, 3 days ago

Milestone: → 26.11
Summary: Remove Junit 4 last remains → [patch] Remove Junit 4 last remains

Beware, the patch introduces deprecations, but it should ultimately be leaner and cleaner for everyone in the long term.

Some adjustments will probably be needed.

Changes:

  • removed all references to JUnit 4 (dropped junit-vintage-engine, junit-jupiter-migrationsupport), JOSMTestRules now runs on plain Jupiter callbacks
  • build cleanup:
    • removed the unused commonslang/errorprone_javac ivy confs and the javacc.home property,
    • fixed the jdeps "all" graph and standalone ant extract-libraries, and
    • synced commons-lang3 between Ant and Maven.
  • josm-custom.jar no longer bundles commons-lang3 and Jetbrains annotations; the sources jar now ships jakarta.annotation-api 3.0.0 sources, matching the classes.
  • tests no longer write foobar/ or failed.png into the working copy (#23090)
  • PluginHandlerJOSMTooOldTest no longer fails on my machine with hu_HU locale. The expected strings were built before the test switched the locale to English, so they had the wrong thousands separator.
Last edited 3 days ago by gaben (previous) (diff)

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to gaben.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.