Fixed test for session:export:view, added CHANGELOG

Author: dantleech

CHANGELOG.md

@@ -0,0 +1,4 @@
+# alpha2 / dev-master
+
+- Added `--pretty` option to `session:export:view` command to output formatted XML
+- Ask confirmation before overwriting file in `session:export:view`

features/phpcr_session_export_view.feature

@@ -28,14 +28,10 @@ Feature: Export the repository to an XML file
         Invalid path 'cms'
         """
 
-    Scenario: Export to an existing file
+    Scenario: Export to an existing file (should overwrite as --no-interaction is specified)
         Given the file "foobar.xml" exists
-        And I execute the "session:export:view /tests_general_base foobar.xml" command
-        Then the command should fail
-        And the output should contain:
-        """
-        File "foobar.xml" already exists.
-        """
+        And I execute the "session:export:view /tests_general_base foobar.xml --no-interaction" command
+        Then the command should not fail
 
     Scenario: Export non recursive
         Given I execute the "session:export:view /tests_general_base foobar.xml --no-recurse" command
@@ -51,3 +47,10 @@ Feature: Export the repository to an XML file
     Scenario: Export the document view
         Given I execute the "session:export:view / foobar.xml --document" command
         Then the command should not fail
+
+    Scenario: Export the document view in pretty way
+        Given I execute the "session:export:view / foobar.xml --pretty" command
+        Then the command should not fail
+        And the file "foobar.xml" should exist
+        And the xpath count "/sv:node" is "1" in file "foobar.xml"
+        And the xpath count "/sv:node/sv:node" is "1" in file "foobar.xml"

src/PHPCR/Shell/Console/Command/Phpcr/SessionExportViewCommand.php

@@ -64,9 +64,13 @@ public function execute(InputInterface $input, OutputInterface $output)
         $dialog = $this->getHelper('dialog');
 
         if (file_exists($file)) {
-            $res = $dialog->askConfirmation($output, 'File already exists, overwrite?');
+            $confirmed = true;
 
-            if (false === $res) {
+            if (false === $input->getOption('no-interaction')) {
+                $confirmed = $dialog->askConfirmation($output, 'File already exists, overwrite?');
+            }
+
+            if (false === $confirmed) {
                 return;
             }
         }