BadFunctions/NoEvals: add unit tests + minor fix

Security/Sniffs/BadFunctions/NoEvalsSniff.php

@@ -26,8 +26,7 @@ public function register() {
 	* @return void
 	*/
 	public function process(File $phpcsFile, $stackPtr) {
-		$tokens = $phpcsFile->getTokens();
-		$error = 'Please do not use eval() functions';
+		$error = 'Please do not use eval()';
 		$phpcsFile->addError($error, $stackPtr, 'NoEvals');
 	}
 

Security/Tests/BadFunctions/NoEvalsUnitTest.inc

@@ -0,0 +1,5 @@
+<?php
+
+eval('$var = file_get_contents(\'filename.php\');');
+eval($string);
+eval("\$var = file_get_contents($filename);');

Security/Tests/BadFunctions/NoEvalsUnitTest.php

@@ -0,0 +1,44 @@
+<?php
+
+namespace PHPCS_SecurityAudit\Security\Tests\BadFunctions;
+
+use PHPCS_SecurityAudit\Security\Tests\AbstractSecurityTestCase;
+
+/**
+ * Unit test class for the NoEvals sniff.
+ *
+ * @covers \PHPCS_SecurityAudit\Security\Sniffs\BadFunctions\NoEvalsSniff
+ */
+class NoEvalsUnitTest extends AbstractSecurityTestCase
+{
+
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList()
+	{
+		return [
+			3 => 1,
+			4 => 1,
+			5 => 1,
+		];
+	}
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
+	}
+}