Improve error messages for invalid <dir> option. Adds tests.

Author: johnbillion

features/scaffold-package-github.feature

@@ -1,13 +1,25 @@
 Feature: Scaffold GitHub configuration for an existing package
 
-  Scenario: Fails when invalid package directory provided
+  Scenario: Fails when invalid directory provided
     Given an empty directory
 
-    When I run `wp package path`
-    Then save STDOUT as {PACKAGE_PATH}
+    When I try `wp scaffold package-github bar`
+    Then the bar directory should not exist
+    And STDERR should be:
+      """
+      Error: Directory does not exist.
+      """
+
+  Scenario: Fails when invalid package provided
+    Given an empty directory
+    And a baz/empty file:
+      """
+      """
 
-    When I try `wp scaffold package-github {PACKAGE_PATH}/local/wp-cli/default-github`
-    Then STDERR should be:
+    When I try `wp scaffold package-github baz`
+    Then the baz directory should exist
+    But the baz/.github directory should not exist
+    And STDERR should be:
       """
       Error: Invalid package directory. composer.json file must be present.
       """

features/scaffold-package-readme.feature

@@ -1,5 +1,29 @@
 Feature: Scaffold a README.md file for an existing package
 
+  Scenario: Fails when invalid directory provided
+    Given an empty directory
+
+    When I try `wp scaffold package-readme bar`
+    Then the bar directory should not exist
+    And STDERR should be:
+      """
+      Error: Directory does not exist.
+      """
+
+  Scenario: Fails when invalid package provided
+    Given an empty directory
+    And a baz/empty file:
+      """
+      """
+
+    When I try `wp scaffold package-readme baz`
+    Then the baz directory should exist
+    But the baz/README.md file should not exist
+    And STDERR should be:
+      """
+      Error: Invalid package directory. composer.json file must be present.
+      """
+
   Scenario: Scaffold a README.md based on the defaults
     Given an empty directory
 

features/scaffold-package-tests.feature

@@ -24,6 +24,30 @@ Feature: Scaffold the test suite for an existing package
       }
       """
 
+  Scenario: Fails when invalid directory provided
+    Given an empty directory
+
+    When I try `wp scaffold package-tests bar`
+    Then the bar directory should not exist
+    And STDERR should be:
+      """
+      Error: Directory does not exist.
+      """
+
+  Scenario: Fails when invalid package provided
+    Given an empty directory
+    And a baz/empty file:
+      """
+      """
+
+    When I try `wp scaffold package-tests baz`
+    Then the baz directory should exist
+    But the baz/features directory should not exist
+    And STDERR should be:
+      """
+      Error: Invalid package directory. composer.json file must be present.
+      """
+
   Scenario: Scaffold package tests
     Given a invalid-command/command.php file:
       """

src/ScaffoldPackageCommand.php

@@ -194,8 +194,12 @@ public function package_readme( $args, $assoc_args ) {
 
 		list( $package_dir ) = $args;
 
-		if ( ! is_dir( $package_dir ) || ! file_exists( $package_dir . '/composer.json' ) ) {
-			WP_CLI::error( "Invalid package directory. composer.json file must be present." );
+		if ( ! is_dir( $package_dir ) ) {
+			WP_CLI::error( 'Directory does not exist.' );
+		}
+
+		if ( ! file_exists( $package_dir . '/composer.json' ) ) {
+			WP_CLI::error( 'Invalid package directory. composer.json file must be present.' );
 		}
 
 		$composer_obj = json_decode( file_get_contents( $package_dir . '/composer.json' ), true );
@@ -401,9 +405,14 @@ public function package_github( $args, $assoc_args ) {
 			$package_dir = rtrim( $package_dir, '/' );
 		}
 
-		if ( ! is_dir( $package_dir ) || ! file_exists( $package_dir . '/composer.json' ) ) {
-			WP_CLI::error( "Invalid package directory. composer.json file must be present." );
+		if ( ! is_dir( $package_dir ) ) {
+			WP_CLI::error( 'Directory does not exist.' );
+		}
+
+		if ( ! file_exists( $package_dir . '/composer.json' ) ) {
+			WP_CLI::error( 'Invalid package directory. composer.json file must be present.' );
 		}
+
 		$force = Utils\get_flag_value( $assoc_args, 'force' );
 		$template_path = dirname( dirname( __FILE__ ) ) . '/templates';
 
@@ -522,8 +531,12 @@ public function package_tests( $args, $assoc_args ) {
 			$package_dir = rtrim( $package_dir, '/' );
 		}
 
-		if ( ! is_dir( $package_dir ) || ! file_exists( $package_dir . '/composer.json' ) ) {
-			WP_CLI::error( "Invalid package directory. composer.json file must be present." );
+		if ( ! is_dir( $package_dir ) ) {
+			WP_CLI::error( 'Directory does not exist.' );
+		}
+
+		if ( ! file_exists( $package_dir . '/composer.json' ) ) {
+			WP_CLI::error( 'Invalid package directory. composer.json file must be present.' );
 		}
 
 		$package_dir .= '/';