apache / apache/maven-surefire

Fix the deprecation of testng xml suites

Open
#3,452 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Java
Stars
461
Forks
588
Avg merge
1d 8h
Merged PRs (30d)
19

Description

### Affected version

3.6.0

### Bug description

The parameter got deprecated but

1. no warning or explicit log about it while keeping the support of the parameter
2. it was not dropped totally (since it is not supported)

Goal is to 1. ensure we do either align the impl on the `@Deprecated` annotation by supporting the runtime with a warning or 2. we totally drop the parameter and say we don't support it at all (note: maybe ensure all EE TCK moved away of this usage - or it is small enough to be worked around)

Contributor guide

Open the contributing guide

Research direction

Start by locating the deprecated TestNG XML suite parameter in the Maven Surefire code and tracing how version 3.6.0 handles it at runtime. Check whether the EE TCK still uses the parameter, then establish whether completion means warning while retaining support or removing support and documenting that outcome.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
build-system, testing
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.