From ffa16fa77162af70bb3029680dcb34508ad35a9e Mon Sep 17 00:00:00 2001
From: Felix Arntz <flixos90@git.wordpress.org>
Date: Fri, 14 Sep 2018 07:54:53 +0000
Subject: [PATCH] Upgrade/Install: Properly exclude the
 `admin/includesSchema.php` tests file for the
 `WordPress.DB.PreparedSQL.NotPrepared` sniff.

The way this was implemented in [43630] caused the file to be excluded for all sniffs, since PHPCS does not support file-wide ignores of a specific sniff at this point.

Props jrf.
See #44896.


git-svn-id: https://develop.svn.wordpress.org/trunk@43641 602fd350-edb4-49c9-b593-d223f7449a82
---
 phpcs.xml.dist                               | 3 +++
 tests/phpunit/tests/admin/includesSchema.php | 1 -
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/phpcs.xml.dist b/phpcs.xml.dist
index b8fd3c9cd7..612c772d0c 100644
--- a/phpcs.xml.dist
+++ b/phpcs.xml.dist
@@ -81,6 +81,9 @@
 	<rule ref="WordPress.DB.PreparedSQLPlaceholders.UnescapedLiteral">
 		<exclude-pattern>/tests/phpunit/tests/db\.php</exclude-pattern>
 	</rule>
+	<rule ref="WordPress.DB.PreparedSQL.NotPrepared">
+		<exclude-pattern>/tests/phpunit/tests/admin/includesSchema\.php</exclude-pattern>
+	</rule>
 
 	<!-- Whitelist the I18n functions file from issues identified by the I18n sniff
 		(such as calling the low-level translate() function). -->
diff --git a/tests/phpunit/tests/admin/includesSchema.php b/tests/phpunit/tests/admin/includesSchema.php
index 49f89a7187..e11c4ecd8b 100644
--- a/tests/phpunit/tests/admin/includesSchema.php
+++ b/tests/phpunit/tests/admin/includesSchema.php
@@ -1,5 +1,4 @@
 <?php
-// phpcs:ignoreFile WordPress.DB.PreparedSQL.NotPrepared
 
 /**
  * @group admin