search

pharo-contributions / taskit

TaskIt is a library that ease Process usage in Pharo. It provides abstractions to execute and synchronize concurrent tasks, and several pre-built mechanisms that are useful for many application developers.
MIT License
43 stars 24 forks source link

Fix failing tests in TKTRawProcessHolderTest + Pharo 9 #111

Open tinchodias opened 3 years ago

tinchodias commented 3 years ago

Tests: testStackContainsContextsOfMaster and testStackStartingAt.

After debugging a bit, I think it's the same root. A context is expected to head the stack but it isn't (identity equals is false). No idea.

tinchodias commented 2 years ago

In fact, these tests are part of TaskItDebugger, and it is broken since Pharo 9 since it depends on GTDebugger. Time passed and I think we should remove this package from the default group, and open an issue to recover it.

tinchodias commented 2 years ago

Related issue: https://github.com/pharo-contributions/taskit/issues/109

tinchodias commented 2 years ago

In 6961207, I marked both tests as expected failures and I plan to let this issue open to remind they need attention.