Reworked MockMessageSend printon so that it's display matches that of…

… a normal messsage send, making it easier to compare in the debugger on a failure. (simpler is better)
macta · Apr 30, 2024 · e6b60dd · e6b60dd
1 parent 7738c55
commit e6b60dd
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/src/SUnit-MockObjects-Tests/MockMessageSendTest.class.st b/src/SUnit-MockObjects-Tests/MockMessageSendTest.class.st
@@ -27,7 +27,7 @@ MockMessageSendTest >> testBinaryMessagePrintOn [
 
 	mockMessage printOn: testStream.
 
-	self assert: (testStream contents includesSubstring: '#binaryMsg: true')
+	self assert: (testStream contents includesSubstring: 'binaryMsg: true')
 ]
 
 { #category : 'running' }
@@ -36,7 +36,7 @@ MockMessageSendTest >> testKeywordMessageArrayPrintOn [
 
 	mockMessage printOn: testStream.
 
-	self assert: (testStream contents includesSubstring: '#keyWord:msg: ( ''hello'', #(1 2 3) )')
+	self assert: (testStream contents includesSubstring: 'keyWord:msg: ( ''hello'', #(1 2 3) )')
 ]
 
 { #category : 'running' }
@@ -45,7 +45,7 @@ MockMessageSendTest >> testKeywordMessagePrintOn [
 
 	mockMessage printOn: testStream.
 
-	self assert: (testStream contents includesSubstring: '#keyWord:msg: ( ''hello'', ''world'' )')
+	self assert: (testStream contents includesSubstring: 'keyWord:msg: ( ''hello'', ''world'' )')
 ]
 
 { #category : 'running' }
@@ -54,5 +54,5 @@ MockMessageSendTest >> testUnaryMessagePrintOn [
 
 	mockMessage printOn: testStream.
 
-	self assert: (testStream contents includesSubstring: '#unaryMsg')
+	self assert: (testStream contents includesSubstring: 'unaryMsg')
 ]
diff --git a/src/SUnit-MockObjects/MockMessageSend.class.st b/src/SUnit-MockObjects/MockMessageSend.class.st
@@ -58,8 +58,7 @@ MockMessageSend >> on: aSymbol with: anArray do: aBlock [
 { #category : 'printing' }
 MockMessageSend >> printOn: aStream [
 
-	aStream nextPutAll: 'Expected Message: '.
-	selector printOn: aStream.
+	aStream nextPutAll: selector.
 
 	arguments isEmptyOrNil ifFalse: [
 		arguments size > 1 ifTrue: [ aStream nextPutAll: ' (' ].