Merge pull request #688 from WordPress/687-add-security-checks-to-plu…

…gin-repo-category Add Security checks to plugin repo category
WordPress · Oct 4, 2024 · 98221d8 · 98221d8
2 parents a4a9785 + 94639e3
commit 98221d8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/includes/Checker/Checks/Security/Direct_DB_Queries_Check.php b/includes/Checker/Checks/Security/Direct_DB_Queries_Check.php
@@ -31,7 +31,10 @@ class Direct_DB_Queries_Check extends Abstract_PHP_CodeSniffer_Check {
 	 * @return array The categories for the check.
 	 */
 	public function get_categories() {
-		return array( Check_Categories::CATEGORY_SECURITY );
+		return array(
+			Check_Categories::CATEGORY_SECURITY,
+			Check_Categories::CATEGORY_PLUGIN_REPO,
+		);
 	}
 
 	/**

diff --git a/includes/Checker/Checks/Security/Late_Escaping_Check.php b/includes/Checker/Checks/Security/Late_Escaping_Check.php
@@ -31,7 +31,10 @@ class Late_Escaping_Check extends Abstract_PHP_CodeSniffer_Check {
 	 * @return array The categories for the check.
 	 */
 	public function get_categories() {
-		return array( Check_Categories::CATEGORY_SECURITY );
+		return array(
+			Check_Categories::CATEGORY_SECURITY,
+			Check_Categories::CATEGORY_PLUGIN_REPO,
+		);
 	}
 
 	/**

diff --git a/tests/behat/features/plugin-list-checks.feature b/tests/behat/features/plugin-list-checks.feature
@@ -35,5 +35,5 @@ Feature: Test that the WP-CLI plugin list checks command works.
       """
     And STDOUT should contain:
       """
-      late_escaping,security,stable
+      late_escaping,"security, plugin_repo",stable
       """