Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move some the feasible functionality of the 'ExperimentPageContentService' class into a newly created java class as this class constructor is growing with many dependencies. #494

Open
ke4 opened this issue Dec 5, 2024 · 0 comments
Labels
technical debt Technical debt

Comments

@ke4
Copy link
Contributor

ke4 commented Dec 5, 2024

  • ExperimentPageContentService currently has 8 DI classes as part of its constructor.
  • This makes the class hard to maintain and hard to test, as we need to inject mocks for all DI classes to write tests.
  • So, the feasible solution would be that let's move some functionality into a newly created class and write tests.
  • That way we can reduce some burden on ExperimentPageContentService or we can make the class easy to manage and test.
@ke4 ke4 added the technical debt Technical debt label Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
technical debt Technical debt
Projects
None yet
Development

No branches or pull requests

1 participant