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.

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>
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

    45 files   -   9      45 suites   - 9   37m 44s ⏱️ + 2m 27s
 4 405 tests  - 153   4 378 ✅  - 158   22 💤 ±0   5 ❌ + 5 
11 811 runs   - 459  11 640 ✅  - 474  156 💤 ±0  15 ❌ +15 

For more details on these failures, see this check.

Results for commit 3d92cfe. ± Comparison against base commit 61d50fa.

This pull request removes 153 tests.
org.eclipse.compare.tests.AsyncExecTests ‑ testCancelOnRequeue
org.eclipse.compare.tests.AsyncExecTests ‑ testQueueAdd
org.eclipse.compare.tests.AsyncExecTests ‑ testWorker
org.eclipse.compare.tests.CompareFileRevisionEditorInputTest ‑ testPrepareCompareInputWithNonLocalResourceTypedElements
org.eclipse.compare.tests.CompareUIPluginTest ‑ testFindContentViewerDescriptorForTextType_StreamAccessor
org.eclipse.compare.tests.CompareUIPluginTest ‑ testFindContentViewerDescriptor_TextType_NotStreamAccessor
org.eclipse.compare.tests.CompareUIPluginTest ‑ testFindContentViewerDescriptor_UnknownType
org.eclipse.compare.tests.ContentMergeViewerTest ‑ testFFFX
org.eclipse.compare.tests.ContentMergeViewerTest ‑ testFFTX
org.eclipse.compare.tests.ContentMergeViewerTest ‑ testFFXF
…

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