[CI] failing test: testNoUnusedInstanceVariablesLeft

[MarcusDenker]

Bug description
testNoUnusedInstanceVariablesLeft is failing:

the following classes have unused instance variables:

ReClassForGeneratingEqualAndHash ReClassForGeneratingPrintOn OupsDummyDebuggerSystem ReClassForGeneratingEqualAndHashExistingImplementors

[MarcusDenker]

Some where introduced by #16786, but we can not revert it anymore as there where too many things merged after.

So we have to fix it.

[MarcusDenker]

ReClassForGeneratingEqualAndHash is test data, so we can not just remove the ivar

[hernanmd]

Please notice there is a PR that fixes the issue for the Refactoring packages here: https://github.com/pharo-project/pharo/pull/16853/files#diff-78c29fe49f6e8997db820120f064e9b206e666f6281d26f883d491e1d8d926f3

Another PR should address the OupsDummyDebuggerSystem one.

GitHub

Fix multiple tests with refactoring driver for duplicate class. by hernanmd · Pull Request #16853 · pharo-project/pharo

Fix a couple of tests failing, reported in #16850, due to missing support to pre-configure refactorings for test purposes. Also configured missing mocks for the preview presenter. Fix also for #16825