adding test to ensure it is possible to recompile code within a bloc with a dead home context twice

[adri09070]

Fixes #722

Test corresponding to this PR that needs to be merged first: https://github.com/pharo-project/pharo/pull/16319

[adri09070]

@StevenCostiou

[Ducasse]

Hi @adri09070 This test is failing

StDebuggerActionModelTest
 ✗ #testRecompileMethodToInBlockContextWithDeadHomeTwice (7ms)
TestFailure: Got 'methodWithDeadBlockContext

	^ [ (1 + 1) printString ] yourself' instead of 'methodWithDeadBlockContext

	^ [ (1 + 1) printString ] yourself yourself'.
StDebuggerActionModelTest(TestAsserter)>>assert:description:resumable:
StDebuggerActionModelTest(TestAsserter)>>assert:description:
StDebuggerActionModelTest(TestAsserter)>>assert:equals:
StDebuggerActionModelTest>>testRecompileMethodToInBlockContextWithDeadHomeTwice ...assert: (self class >> #methodWithDeadBlockContext) ast sourceCode
		equals: methodWithBlockContextOriginalSource , ' yourself yourself'
StDebuggerActionModelTest(TestCase)>>performT

[adri09070]

Hi @Ducasse

Yes, this is because https://github.com/pharo-project/pharo/pull/16319 should be merged first. That's why I put it to draft

[Ducasse]

Kicking the build again

[Ducasse]

This is strange since I integrated https://github.com/pharo-project/pharo/pull/16319

[Ducasse]

May be https://github.com/pharo-project/pharo/pull/16319 is not available yet?

[adri09070]

Yes, I don't think https://github.com/pharo-project/pharo/pull/16319 is available yet. Moreover, the other PR has been merged into Pharo13 and this PR targets Pharo12. Both PRs should be merged into Pharo13 and Pharo12.

You can change the target branch of this PR to Pharo13 (I have the rights to do it, but I don't find the button to do it on Github) and if the test passes, you could later merge both PRs into Pharo12

[adri09070]

Ah, I found the button

[adri09070]

But we still need https://github.com/pharo-project/pharo/pull/16319 to be available. I don't know when new Pharo builds are built

[Ducasse]

Let us wait a bit and see.

[Ducasse]

I do not understand because the test is failing

StDebuggerActionModelTest
 ✗ #testRecompileMethodToInBlockContextWithDeadHomeTwice (5ms)

[adri09070]

We just had to wait for the next Pharo build. The NewTools CI does not create a full image with the bootstrap.

Failing test is unrelated