Skip to content

Run addons SWT tests via the AllTests suite in the Tycho build#4106

Open
vogella wants to merge 1 commit into
eclipse-platform:masterfrom
vogella:addons-swt-test-suite
Open

Run addons SWT tests via the AllTests suite in the Tycho build#4106
vogella wants to merge 1 commit into
eclipse-platform:masterfrom
vogella:addons-swt-test-suite

Conversation

@vogella

@vogella vogella commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Instead of relying on
Tycho's class-name autodiscovery, use the standard Tycho approach via
build.properties.

Reintroduce the AllTests @suite and point Tycho at it through the
pom.model.property.testClass build property. This disables the
additional
name-based scan, so each test class is executed exactly once via the
suite.

The restored suite also includes StackDropAgentTest, which the previous
AllTests suite had omitted, so every test class in the bundle is now
part
of the suite.

@vogella vogella force-pushed the addons-swt-test-suite branch from 8a8ec3a to a8111b0 Compare June 15, 2026 19:14
Instead of relying on
Tycho's class-name autodiscovery, use the standard Tycho approach via
build.properties.

Reintroduce the AllTests @suite and point Tycho at it through the
pom.model.property.testClass build property. This disables the
additional
name-based scan, so each test class is executed exactly once via the
suite.

The restored suite also includes StackDropAgentTest, which the previous
AllTests suite had omitted, so every test class in the bundle is now
part
of the suite.
@vogella vogella force-pushed the addons-swt-test-suite branch from a8111b0 to 8996457 Compare June 15, 2026 19:15
@vogella

vogella commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

@HeikoKlare from Git history it looks like you created the suite in 79baeed (without licence header), so I attributed it to you. If that is wrong or should be changed, please let me know.

@HeikoKlare

Copy link
Copy Markdown
Contributor

I did not create that suite but just moved it in the referenced commit 79baeed. The suite was originally introduced in commit 4291d9c.

@vogella

vogella commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

So want should I put into the copyright header?

@github-actions

Copy link
Copy Markdown
Contributor

Test Results

   852 files   - 12     852 suites   - 12   54m 31s ⏱️ + 10m 2s
 8 050 tests ± 0   7 806 ✅  -  1  243 💤 ±0  1 ❌ +1 
20 052 runs  + 5  19 397 ✅ + 4  654 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 8996457. ± Comparison against base commit b268df2.

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