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

Rework Mylyn Doc tests that requires package-local access #354

Open
5 tasks
ruspl-afed opened this issue Jan 11, 2024 · 0 comments
Open
5 tasks

Rework Mylyn Doc tests that requires package-local access #354

ruspl-afed opened this issue Jan 11, 2024 · 0 comments
Milestone

Comments

@ruspl-afed
Copy link
Contributor

ruspl-afed commented Jan 11, 2024

After switching from bundle to eclipse-plugin + eclipse-test-plugin we have a number of troubles with accessing package-local members during tests. Although we can use fragments to solve this, it may be better to improve implementation and tests instead.
It is good to have package local members, and strictly speaking the amount of public members for a package should be minimal
However, tests should be focused rather on testing type's contract than on accessing its internals.

The list of tests to be revived:

  • DefaultSplittingStrategyTest
  • HtmlEntitiesTest
  • HtmlParserTest
  • ServiceLocatorTest
  • WikiToDocTaskTest
@ruspl-afed ruspl-afed added this to the 4.2.0 milestone Jan 11, 2024
@ruspl-afed ruspl-afed modified the milestones: 4.2.0, 4.3.0 Mar 2, 2024
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

No branches or pull requests

1 participant