Skip to content

refactor(component): decouple export summary assembly from repositories#3948

Open
VanKhanhAnny wants to merge 1 commit intoeclipse-sw360:mainfrom
VanKhanhAnny:refactor/3947-component-export-summary-boundary
Open

refactor(component): decouple export summary assembly from repositories#3948
VanKhanhAnny wants to merge 1 commit intoeclipse-sw360:mainfrom
VanKhanhAnny:refactor/3947-component-export-summary-boundary

Conversation

@VanKhanhAnny
Copy link
Contributor

Fixes #3947

This PR refactors the component export summary path so that ComponentSummary no longer performs direct repository-backed data fetching for export-related summaries.

  • The change keeps the current export behavior but moves release and vendor loading responsibilities to the caller side, which makes the export summary path easier to reason about and reduces coupling in the component backend.

No new dependencies were added

Suggest Reviewer

@GMishx @bibhuti230185 @amritkv @rudra-superrr

How To Test?

  1. Ensure the local CouchDB test instance is available and the backend test environment is configured.
  2. Run:
    mvn --% -pl backend/components -Dbase.deploy.dir=D:/sw360/.deploy -Denforcer.skip=true -Dsurefire.failIfNoSpecifiedTests=false -Dtest=ComponentDatabaseHandlerTest test
  3. Verify that the test suite passes.
  4. Confirm that export-related component summary behavior remains unchanged, including:
  • export summary generation
  • detailed export summary generation
  • export-related component search behavior

Checklist

Must:

  • All related issues are referenced in commit messages and in PR

Refs: eclipse-sw360#3947
Signed-off-by: VanKhanhAnny <arianne.dangvankhanh@gmail.com>
@GMishx GMishx added needs code review needs general test This is general testing, meaning that there is no org specific issue to check for labels Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs code review needs general test This is general testing, meaning that there is no org specific issue to check for

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor component export summary assembly to remove direct repository fetching from ComponentSummary

2 participants