Skip to content

Fix "Message not found for id: 0" in API tools compatible delta messages#2336

Merged
laeubi merged 1 commit into
eclipse-pde:masterfrom
laeubi:issue_2332
May 17, 2026
Merged

Fix "Message not found for id: 0" in API tools compatible delta messages#2336
laeubi merged 1 commit into
eclipse-pde:masterfrom
laeubi:issue_2332

Conversation

@laeubi
Copy link
Copy Markdown
Contributor

@laeubi laeubi commented May 17, 2026

Add missing message mappings in Messages.getKey() for:

  • INTERFACE + ADDED + DEFAULT_METHOD (new key 113)
  • INTERFACE + ADDED + SUPER_INTERFACE_DEFAULT_METHOD (new key 115)
  • INTERFACE + CHANGED + INCREASE_ACCESS (new key 114)
  • ENUM + ADDED + TYPE_MEMBER (reuse key 16)
  • ANNOTATION + ADDED + TYPE_MEMBER (reuse key 16)

Also fix ClassFileComparator to use SUPER_INTERFACE_DEFAULT_METHOD
(not DEFAULT_METHOD) when reporting methods contributed by newly
added superinterfaces in the expanded set path (line 501).

Add reproducing tests: InterfaceDeltaTests.test42/43,
EnumDeltaTests.test17, AnnotationDeltaTests.test12.

Fixes #2332

 Add missing message mappings in Messages.getKey() for:
 - INTERFACE + ADDED + DEFAULT_METHOD (new key 113)
 - INTERFACE + ADDED + SUPER_INTERFACE_DEFAULT_METHOD (new key 115)
 - INTERFACE + CHANGED + INCREASE_ACCESS (new key 114)
 - ENUM + ADDED + TYPE_MEMBER (reuse key 16)
 - ANNOTATION + ADDED + TYPE_MEMBER (reuse key 16)

 Also fix ClassFileComparator to use SUPER_INTERFACE_DEFAULT_METHOD
 (not DEFAULT_METHOD) when reporting methods contributed by newly
 added superinterfaces in the expanded set path (line 501).

 Add reproducing tests: InterfaceDeltaTests.test42/43,
 EnumDeltaTests.test17, AnnotationDeltaTests.test12.

 Fixes eclipse-pde#2332
@github-actions
Copy link
Copy Markdown

Test Results

  126 files  ±0    126 suites  ±0   34m 59s ⏱️ - 3m 22s
3 506 tests +4  3 452 ✅ +4   54 💤 ±0  0 ❌ ±0 
9 321 runs  ±0  9 191 ✅ ±0  130 💤 ±0  0 ❌ ±0 

Results for commit 0d14706. ± Comparison against base commit 8e9a708.

@laeubi laeubi merged commit f095774 into eclipse-pde:master May 17, 2026
19 checks passed
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.

API tooling fails to provide root cause in the quick fix

1 participant