eclipse-mylyn / org.eclipse.mylyn

Eclipse Public License 2.0
14 stars 9 forks source link

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

Open ruspl-afed opened 9 months ago

ruspl-afed commented 9 months ago

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:

ruspl-afed commented 4 months ago

Removing from 4.3.0 , we are near RC2 already