Merge pull request #103 from wp-cli/97-conditional-feature

Only create a sample feature file when none exist

Author: schlessera

features/scaffold-package-tests.feature

@@ -104,3 +104,13 @@ Feature: Scaffold the test suite for an existing package
       bash bin/test.sh
       """
     And the community-command/.travis.yml file should not exist
+
+  Scenario: Don't scaffold features/load-wp-cli.feature when a feature file already exists
+    When I run `wp scaffold package-tests community-command`
+    And I run `mv community-command/features/load-wp-cli.feature community-command/features/command.feature`
+    Then the community-command/features/load-wp-cli.feature file should not exist
+    And the community-command/features/command.feature file should exist
+
+    When I run `wp scaffold package-tests community-command --force`
+    Then the community-command/features/load-wp-cli.feature file should not exist
+    And the community-command/features/command.feature file should exist

src/ScaffoldPackageCommand.php

@@ -502,12 +502,16 @@ public function package_tests( $args, $assoc_args ) {
 				'features/extra/no-mail.php'                  => $extra_dir,
 			),
 			$package_root => array(
-				'templates/load-wp-cli.feature'               => $features_dir,
 				'bin/install-package-tests.sh'                => $bin_dir,
 				'bin/test.sh'                                 => $bin_dir,
 			),
 		);
 
+		// Only create a sample feature file when none exist
+		if ( ! glob( $features_dir . '/*.feature' ) ) {
+			$copy_source[ $package_root ]['templates/load-wp-cli.feature'] = $features_dir;
+		}
+
 		if ( 'travis' === $assoc_args['ci'] ) {
 			$copy_source[ $package_root ]['.travis.yml'] = $package_dir;
 		} else if ( 'circle' === $assoc_args['ci'] ) {