Add test case for automatic string conversion (#33)

* Add test case for automatic string conversion

This may be confusing because if PHP encounters an undefined symbol, it
will issue a warning and try converting it to a string. Because of this
fallback, using unquoted strings may seem like valid PHP, but it's not
doing what you might think it's doing; it's actually still using a
string. In a future version of php, this fallback may be removed, so
it's recommended to avoid it (see the warning here:
https://www.php.net/manual/en/migration72.deprecated.php).

* Add case to test

* Fix another test that uses that fixture

Author: sirbrillig

tests/Sniffs/Imports/RequireImportsSniffTest.php

@@ -169,6 +169,7 @@ public function testRequireImportsSniffFindsWordPressPatternsIfNotSet() {
 			61,
 			63,
 			74,
+			80,
 		];
 		$this->assertEquals($expectedLines, $lines);
 	}
@@ -185,7 +186,7 @@ public function testRequireImportsSniffIgnoresWordPressPatternsIfSet() {
 		);
 		$phpcsFile->process();
 		$lines = $helper->getWarningLineNumbersFromFile($phpcsFile);
-		$expectedLines = [ 38, 61, 63 ];
+		$expectedLines = [ 38, 61, 63, 80 ];
 		$this->assertEquals($expectedLines, $lines);
 	}
 

tests/Sniffs/Imports/WordPressFixture.php

@@ -75,3 +75,6 @@ public function test_additional_constants() {
 		);
 	}
 }
+
+function foo() {}
+add_action('init', foo);