eclipse-platform / eclipse.platform

https://eclipse.dev/eclipse/
Eclipse Public License 2.0
84 stars 113 forks source link

Simplify and migrate resources content tests to JUnit 5 #903 #1605

Closed HeikoKlare closed 2 weeks ago

HeikoKlare commented 3 weeks ago

Simplifies the tests in org.eclipse.core.tests.resources.content:

Migrates the tests in org.eclipse.core.tests.resources.content to JUnit 5 by replacing according JUnit annotations and migrating assertions.

Contributes to https://github.com/eclipse-platform/eclipse.platform/issues/903

eclipse-platform-bot commented 3 weeks ago

This pull request changes some projects for the first time in this development cycle. Therefore the following files need a version increment:

runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF

An additional commit containing all the necessary changes was pushed to the top of this PR's branch. To obtain these changes (for example if you want to push more changes) either fetch from your fork or apply the git patch.

Git patch ``` From 5cfcfc4895d0fc7c6a8fe77ecb36c33537594625 Mon Sep 17 00:00:00 2001 From: Eclipse Platform Bot Date: Sun, 3 Nov 2024 11:13:29 +0000 Subject: [PATCH] Version bump(s) for 4.34 stream diff --git a/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF b/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF index 88086087ff..d70afdf6ca 100644 --- a/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF +++ b/runtime/tests/org.eclipse.core.tests.harness/META-INF/MANIFEST.MF @@ -2,7 +2,7 @@ Manifest-Version: 1.0 Bundle-ManifestVersion: 2 Bundle-Name: Eclipse Core Tests Harness Bundle-SymbolicName: org.eclipse.core.tests.harness;singleton:=true -Bundle-Version: 3.17.0.qualifier +Bundle-Version: 3.17.100.qualifier Bundle-Vendor: Eclipse.org Export-Package: org.eclipse.core.tests.harness;version="2.0", org.eclipse.core.tests.harness.session, -- 2.47.0 ```

Further information are available in Common Build Issues - Missing version increments.

github-actions[bot] commented 3 weeks ago

Test Results

 1 755 files  ±0   1 755 suites  ±0   1h 35m 18s ⏱️ + 8m 26s  4 170 tests +1   4 148 ✅ +1   22 💤 ±0  0 ❌ ±0  13 107 runs  +3  12 943 ✅ +3  164 💤 ±0  0 ❌ ±0 

Results for commit 0511cbc7. ± Comparison against base commit 4712d122.

This pull request removes 2 and adds 3 tests. Note that renamed tests count towards both. ``` AutomatedResourceTests AllContentTests IContentTypeManagerTest ‑ testContentDescription AutomatedResourceTests AllContentTests SpecificContextTest ‑ testContentTypeLookup ``` ``` AutomatedResourceTests AllContentTests IContentTypeManagerTest ‑ testContentDescription(boolean)[1] true AutomatedResourceTests AllContentTests IContentTypeManagerTest ‑ testContentDescription(boolean)[2] false AutomatedResourceTests AllContentTests SpecificContextTest ‑ testContentTypeLookup(TestInfo) ```

:recycle: This comment has been updated with latest results.