Skip to content

Migrate test fixtures to non-deprecated contentTypes extension point#2633

Open
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:migration-contenttypes-extension-to-non-deprecated-api
Open

Migrate test fixtures to non-deprecated contentTypes extension point#2633
vogella wants to merge 1 commit intoeclipse-platform:masterfrom
vogella:migration-contenttypes-extension-to-non-deprecated-api

Conversation

@vogella
Copy link
Copy Markdown
Contributor

@vogella vogella commented Apr 26, 2026

The org.eclipse.core.runtime.contentTypes extension point was deprecated in 2018 and is being removed in #1644. This swaps the four test fixture plugin.xml files in org.eclipse.core.tests.resources over to the canonical successor org.eclipse.core.contenttype.contentTypes. The child element schema is identical, so this is a mechanical id swap with no behavioral change.

The fixtures are intentionally kept — they provide negative-case coverage (missing names, invalid describers, empty file-extensions, missing alias targets) for the content-type registry tests.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 26, 2026

Test Results

    54 files  ±0      54 suites  ±0   35m 20s ⏱️ - 1m 38s
 4 849 tests ±0   4 827 ✅ ±0   22 💤 ±0  0 ❌ ±0 
12 435 runs  ±0  12 279 ✅ ±0  156 💤 ±0  0 ❌ ±0 

Results for commit e012f52. ± Comparison against base commit 5ccd447.

This pull request removes 387 and adds 276 tests. Note that renamed tests count towards both.
, nl=[Ljava.lang.String;@12fcf5dd]
, nl=[Ljava.lang.String;@140179c1]
, nl=[Ljava.lang.String;@149e4670]
, nl=[Ljava.lang.String;@160aba62]
, nl=[Ljava.lang.String;@18509921]
, nl=[Ljava.lang.String;@1858f81a]
, nl=[Ljava.lang.String;@19216cce]
, nl=[Ljava.lang.String;@19f24008]
, nl=[Ljava.lang.String;@1af1bfba]
, nl=[Ljava.lang.String;@1b4eee06]
…
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[10] Parameters[limit=2, chunks=5, repeat=1, input=====
====, output=== ...
== ...
== ...
== ...
== ...
== ...
, nl=[Ljava.lang.String;@56fd2e7c]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[10] Parameters[limit=2, chunks=5, repeat=1, input=====
====, output=== ...
== ...
== ...
== ...
== ...
== ...
, nl=[Ljava.lang.String;@7571a7a1]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[10] Parameters[limit=2, chunks=5, repeat=1, input=====
====, output=== ...
== ...
== ...
== ...
== ...
== ...
, nl=[Ljava.lang.String;@7dde760e]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[11] Parameters[limit=2, chunks=1, repeat=1, input==========, output=== ...
, nl=[Ljava.lang.String;@2350ee6d]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[11] Parameters[limit=2, chunks=1, repeat=1, input==========, output=== ...
, nl=[Ljava.lang.String;@24899e16]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[11] Parameters[limit=2, chunks=1, repeat=1, input==========, output=== ...
, nl=[Ljava.lang.String;@5260186c]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[12] Parameters[limit=2, chunks=1, repeat=1, input========
==, output=== ...
==, nl=[Ljava.lang.String;@2fc0b42a]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[12] Parameters[limit=2, chunks=1, repeat=1, input========
==, output=== ...
==, nl=[Ljava.lang.String;@3a603d0b]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[12] Parameters[limit=2, chunks=1, repeat=1, input========
==, output=== ...
==, nl=[Ljava.lang.String;@5379985a]
org.eclipse.debug.tests.console.ConsoleOutputLineTruncateTest ‑ test(Parameters)[13] Parameters[limit=3, chunks=1, repeat=1, input==========, output==== ...
, nl=[Ljava.lang.String;@28e29d98]
…

♻️ This comment has been updated with latest results.

The org.eclipse.core.runtime.contentTypes extension point was deprecated
in 2018 and is being removed. Replace with its canonical successor
org.eclipse.core.contenttype.contentTypes in the four test fixture
plugin.xml files. The child element schema is identical, so this is a
mechanical id swap with no behavioral change.

These fixtures provide negative-case coverage (missing names, invalid
describers, empty file-extensions, missing alias targets) for the
content-type registry tests in org.eclipse.core.tests.resources.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@vogella vogella force-pushed the migration-contenttypes-extension-to-non-deprecated-api branch from 3d92cfe to e012f52 Compare May 3, 2026 12:10
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.

1 participant