Merge branch 'main' into 140-remove-grunt

wp-cli · Jan 29, 2025 · 6ae983a · 6ae983a
2 parents a0fd65d + 036c515
commit 6ae983a
Show file tree

Hide file tree

Showing 9 changed files with 126 additions and 126 deletions.
diff --git a/.github/workflows/testing.yml b/.github/workflows/testing.yml
@@ -1,6 +1,7 @@
 name: Testing
 
 on:
+  workflow_dispatch:
   pull_request:
   push:
     branches:

diff --git a/README.md b/README.md
@@ -238,6 +238,7 @@ The following files are also included unless the `--skip-tests` is used:
 		options:
 		  - circle
 		  - gitlab
+		  - bitbucket
 		  - github
 		---
 

diff --git a/composer.json b/composer.json
@@ -12,7 +12,7 @@
         }
     ],
     "require": {
-        "wp-cli/wp-cli": "^2.5"
+        "wp-cli/wp-cli": "^2.12"
     },
     "require-dev": {
         "wp-cli/extension-command": "^1.2 || ^2",
@@ -24,7 +24,8 @@
         "allow-plugins": {
             "dealerdirect/phpcodesniffer-composer-installer": true,
             "johnpbloch/wordpress-core-installer": true
-        }
+        },
+        "lock": false
     },
     "extra": {
         "branch-alias": {

diff --git a/features/scaffold-plugin-tests.feature b/features/scaffold-plugin-tests.feature
@@ -172,42 +172,29 @@ Feature: Scaffold plugin unit tests
     And the {PLUGIN_DIR}/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:5.6
-              name: "PHP 5.6"
+              image: php:7.4
+              name: "PHP 7.4"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {PLUGIN_DIR}/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:7.0
-              name: "PHP 7.0"
+              image: php:8.0
+              name: "PHP 8.0"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {PLUGIN_DIR}/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:7.1
-              name: "PHP 7.1"
+              image: php:8.2
+              name: "PHP 8.2"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
-      """
-    And the {PLUGIN_DIR}/bitbucket-pipelines.yml file should contain:
-      """
-          - step:
-              image: php:7.2
-              name: "PHP 7.2"
-              script:
-                # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {PLUGIN_DIR}/bitbucket-pipelines.yml file should contain:
       """
@@ -309,3 +296,16 @@ Feature: Scaffold plugin unit tests
       """
       require dirname( dirname( __FILE__ ) ) . '/bar.php';
       """
+
+  Scenario: Accept bitbucket as valid CI in plugin scaffold
+    Given a WP install
+    When I run `wp plugin path`
+    Then save STDOUT as {PLUGIN_DIR}
+
+    When I run `wp scaffold plugin hello-world --ci=bitbucket`
+    Then STDOUT should not be empty
+    And the {PLUGIN_DIR}/hello-world/.editorconfig file should exist
+    And the {PLUGIN_DIR}/hello-world/hello-world.php file should exist
+    And the {PLUGIN_DIR}/hello-world/readme.txt file should exist
+    And the {PLUGIN_DIR}/hello-world/bitbucket-pipelines.yml file should exist
+    And the {PLUGIN_DIR}/hello-world/tests directory should exist
diff --git a/features/scaffold-theme-tests.feature b/features/scaffold-theme-tests.feature
@@ -168,42 +168,29 @@ Feature: Scaffold theme unit tests
     And the {THEME_DIR}/t12child/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:5.6
-              name: "PHP 5.6"
+              image: php:7.4
+              name: "PHP 7.4"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {THEME_DIR}/t12child/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:7.0
-              name: "PHP 7.0"
+              image: php:8.0
+              name: "PHP 8.0"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {THEME_DIR}/t12child/bitbucket-pipelines.yml file should contain:
       """
           - step:
-              image: php:7.1
-              name: "PHP 7.1"
+              image: php:8.2
+              name: "PHP 8.2"
               script:
                 # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
-      """
-    And the {THEME_DIR}/t12child/bitbucket-pipelines.yml file should contain:
-      """
-          - step:
-              image: php:7.2
-              name: "PHP 7.2"
-              script:
-                # Install Dependencies
-                - docker-php-ext-install mysqli
-                - apt-get update && apt-get install -y subversion --no-install-recommends
+                - apt-get update && apt-get install -y subversion git zip libzip-dev --no-install-recommends
       """
     And the {THEME_DIR}/t12child/bitbucket-pipelines.yml file should contain:
       """

diff --git a/src/Scaffold_Command.php b/src/Scaffold_Command.php
@@ -608,6 +608,7 @@ private function get_output_path( $assoc_args, $subdir ) {
 	 * options:
 	 *   - circle
 	 *   - gitlab
+	 *   - bitbucket
 	 *   - github
 	 * ---
 	 *

diff --git a/templates/install-wp-tests.sh b/templates/install-wp-tests.sh
@@ -22,6 +22,17 @@ download() {
         curl -s "$1" > "$2";
     elif [ `which wget` ]; then
         wget -nv -O "$2" "$1"
+    else
+        echo "Error: Neither curl nor wget is installed."
+        exit 1
+    fi
+}
+
+# Check if svn is installed
+check_svn_installed() {
+    if ! command -v svn > /dev/null; then
+        echo "Error: svn is not installed. Please install svn and try again."
+        exit 1
     fi
 }
 
@@ -64,6 +75,7 @@ install_wp() {
 	if [[ $WP_VERSION == 'nightly' || $WP_VERSION == 'trunk' ]]; then
 		mkdir -p $TMPDIR/wordpress-trunk
 		rm -rf $TMPDIR/wordpress-trunk/*
+        check_svn_installed
 		svn export --quiet https://core.svn.wordpress.org/trunk $TMPDIR/wordpress-trunk/wordpress
 		mv $TMPDIR/wordpress-trunk/wordpress/* $WP_CORE_DIR
 	else
@@ -108,6 +120,7 @@ install_test_suite() {
 		# set up testing suite
 		mkdir -p $WP_TESTS_DIR
 		rm -rf $WP_TESTS_DIR/{includes,data}
+        check_svn_installed
 		svn export --quiet --ignore-externals https://develop.svn.wordpress.org/${WP_TESTS_TAG}/tests/phpunit/includes/ $WP_TESTS_DIR/includes
 		svn export --quiet --ignore-externals https://develop.svn.wordpress.org/${WP_TESTS_TAG}/tests/phpunit/data/ $WP_TESTS_DIR/data
 	fi

diff --git a/templates/phpunit.xml.dist b/templates/phpunit.xml.dist
@@ -4,8 +4,9 @@
 	backupGlobals="false"
 	colors="true"
 	convertErrorsToExceptions="true"
-	convertNoticesToExceptions="true"
 	convertWarningsToExceptions="true"
+	convertNoticesToExceptions="true"
+	convertDeprecationsToExceptions="true"
 	>
 	<testsuites>
 		<testsuite name="testing">