Merge pull request #168 from SeasideSt/fix-testEmptyStatements

Fixes Slime test for empty statements (will work correctly after fix …

Author: Johan Brichau

repository/Grease-Tests-Pharo-Slime.package/GRReSlimeTest.class/instance/expectedFailures.st

@@ -0,0 +1,6 @@
+running
+expectedFailures
+
+	SystemVersion current major >= 12 ifTrue:[ ^ #() ].
+	
+	^ #(#testEmptyStatements)

repository/Grease-Tests-Pharo-Slime.package/GRReSlimeTest.class/instance/rules.st

@@ -5,4 +5,6 @@ rules
 		addAll: GRSlimeBlockLintRule allSubclasses;
 		addAll: GRSlimeParseTreeLintRule allSubclasses;
 		addAll: GRSlimeTransformationRule allSubclasses;
+		add: ReMultiplePeriodsTerminatingStatementRule;
+		add: ReMethodSignaturePeriodRule;
 		yourself) reject: #isAbstract

repository/Grease-Tests-Pharo-Slime.package/GRReSlimeTest.class/instance/testEmptyStatements.st

@@ -1,15 +1,18 @@
 tests-block
 testEmptyStatements
 	"We added this test to make sure that this Pharo-supplied rule works because we previously had this one in Grease and it is important."
-	<expectedFailure>
 
 	| class |
 	class := self defineSubClassOf: #GRObject.
-	self compile: 'emptyStatement1. self and' in: class.
-	self compile: 'emptyStatement2 self and..' in: class.
-	self compile: 'emptyStatement3 self and."foo".' in: class.
-	self compile: 'emptyStatement4 self and."foo".self and' in: class.
+	self compile: 'emptyStatement1.', (String with: Character cr), ' self and' in: class.
+	self compile: 'emptyStatement2', (String with: Character cr), ' self and..' in: class.
+	self compile: 'emptyStatement3', (String with: Character cr), ' self and."foo".' in: class.
+	self compile: 'emptyStatement4', (String with: Character cr), ' self and."foo".self and' in: class.
 	self 
 		assertRule: ReMultiplePeriodsTerminatingStatementRule
-		matches: { class>>#emptyStatement1 . class>>#emptyStatement2 . class>>#emptyStatement3 . class>>#emptyStatement4 }
-		ignoring: (Array with: ReUnnecessaryLastPeriodRule)
+		matches: { class>>#emptyStatement2 . class>>#emptyStatement3 . class>>#emptyStatement4 }
+		ignoring: { ReMethodSignaturePeriodRule }.
+	self 
+		assertRule: ReMethodSignaturePeriodRule
+		matches: { class>>#emptyStatement1 }
+		ignoring: { ReMultiplePeriodsTerminatingStatementRule }.