🐛 Fix coolstore app tests with star imports

[jmle]

https://issues.redhat.com/browse/MTA-5387

Related to konveyor/java-analyzer-bundle#159

Summary by CodeRabbit

• Chores
  • Updated dependency analysis: increased estimated effort and added two new migration incidents to the analysis report to better track and guide namespace migration from older Java EE to Jakarta EE APIs.

[coderabbitai]

Walkthrough

The pull request updates analysis/tc_coolstore_deps.go, increasing the analysis effort metric from 113 to 115. It adds two incidents to Analysis.Incidents for CatalogItemEntity.java (replace javax.persistence imports with jakarta.persistence) and InventoryNotificationMDB.java (replace javax.jms imports with jakarta.jms). No public or exported entity signatures were changed.

Suggested reviewers

• dymurray

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title ":bug: Fix coolstore app tests with star imports" is clearly related to the changeset. It correctly identifies the target area (coolstore tests in tc_coolstore_deps.go), the subject matter (imports), and the action being taken (fixing). The raw summary shows modifications adding incidents that involve replacing javax imports with jakarta imports, which directly corresponds to the title's focus on import-related fixes. While the title may not exhaustively describe every detail of the change (such as the specific effort value updates), it provides sufficient specificity and clarity for a teammate to quickly understand that this addresses import-related fixes in the coolstore test configuration. The emoji use is minimal and does not substantially detract from the title's clarity.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-tests-import-star

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a005b3 and f23e9a1.

📒 Files selected for processing (1)

• analysis/tc_coolstore_deps.go (2 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: test-tier1 / e2e-api-integration-tests
• GitHub Check: test-tier2 / e2e-api-integration-tests
• GitHub Check: test-tier0 / e2e-api-integration-tests

🔇 Additional comments (2)

analysis/tc_coolstore_deps.go (2)

21-21: LGTM! Effort metric correctly updated.

The effort increase from 113 to 115 correctly reflects the addition of 2 new incidents.

---

442-451: LGTM! New incidents properly added.

The two new incidents for CatalogItemEntity.java and InventoryNotificationMDB.java are correctly formatted and consistent with existing incident patterns. They properly document the need to replace javax.persistence and javax.jms imports with their Jakarta equivalents.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[aufi]

Great 👍