Skip to content

chores(x2): refactor X2ADatabaseService tests for better maintainability #2282

Merged
mareklibra merged 3 commits intoredhat-developer:mainfrom
mareklibra:FLPATH-3199.refactor.DBService
Feb 9, 2026
Merged

chores(x2): refactor X2ADatabaseService tests for better maintainability #2282
mareklibra merged 3 commits intoredhat-developer:mainfrom
mareklibra:FLPATH-3199.refactor.DBService

Conversation

@mareklibra
Copy link
Copy Markdown
Member

Part of: FLPATH-3199

Refactor X2ADatabaseService and its tests into multiple files.

The service and tests grew significantly, making it hard to maintain and understand.
This refactoring reduces the complexity by separating concerns based on projects, modules and jobs while still reusing common functions.

Additional test flows have been added.

@rhdh-gh-app
Copy link
Copy Markdown

rhdh-gh-app Bot commented Feb 9, 2026

Missing Changesets

The following package(s) are changed by this PR but do not have a changeset:

  • @red-hat-developer-hub/backstage-plugin-x2a

See CONTRIBUTING.md for more information about how to add changesets.

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-x2a-backend workspaces/x2a/plugins/x2a-backend patch v1.0.0
@red-hat-developer-hub/backstage-plugin-x2a workspaces/x2a/plugins/x2a none v1.0.0

@mareklibra mareklibra changed the title Flpath 3199.refactor.db service chores(x2): refactor X2ADatabaseService tests for better maintainability Feb 9, 2026
@mareklibra mareklibra marked this pull request as draft February 9, 2026 13:03
@mareklibra mareklibra force-pushed the FLPATH-3199.refactor.DBService branch from 461be7f to 5390688 Compare February 9, 2026 13:28
@mareklibra mareklibra force-pushed the FLPATH-3199.refactor.DBService branch from fa5e19c to 4891c1b Compare February 9, 2026 13:48
@mareklibra mareklibra marked this pull request as ready for review February 9, 2026 13:48
@mareklibra mareklibra force-pushed the FLPATH-3199.refactor.DBService branch from 4891c1b to 8eb1f67 Compare February 9, 2026 14:00
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 9, 2026

Copy link
Copy Markdown
Contributor

@elai-shalev elai-shalev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, well done
router refactor will be in a seperate PR?

@mareklibra
Copy link
Copy Markdown
Member Author

router refactor will be in a seperate PR?
Yes. This one is already big enough

@mareklibra mareklibra merged commit 4aae07f into redhat-developer:main Feb 9, 2026
13 of 15 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.

2 participants