Merge pull request #89 from hernanmd/fix_Sindarin_failing_test

Fix failing test in Pharo 12

src/Sindarin-Tests/SindarinDebuggerTest.class.st

@@ -506,8 +506,10 @@ SindarinDebuggerTest >> testIsExecutionFinished [
 	scdbg := SindarinDebugger debug: [ self methodWithTwoAssignments  ].
 	self deny: scdbg isExecutionFinished.
 
-	[ scdbg isExecutionFinished ] whileFalse: [ scdbg stepOver ].
-
+	self 
+		should: [ [ scdbg isExecutionFinished ] whileFalse: [ scdbg stepOver ]	] 
+		raise: DebuggedExecutionIsFinished.
+
 	self assert: scdbg currentProcess isTerminated
 ]
 
@@ -1787,11 +1789,13 @@ SindarinDebuggerTest >> testStepOver [
 
 { #category : #tests }
 SindarinDebuggerTest >> testStepOverFinishedExecution [
+	"This test tries to show is that using Sindarin on a block, it should raise an exception if you continue stepping over that code in your Sindarin script while the execution of that code is already finished (nothing more to step)"
+
 	|scdbg|
 	scdbg := SindarinDebugger debug: [ self methodWithImplicitReturn ].
 
 	"Stepping until the implicit return of #methodWithImplicitReturn"
-	scdbg stepOver: 3.		
+	scdbg stepOver: 2.		
 	self should: [scdbg stepOver] raise: DebuggedExecutionIsFinished
 ]