Better message if the user tries to run the interactive commands in non-interactive mode

Author: genail

app/core/base_tool2.rb

@@ -87,7 +87,14 @@ def parse_options
 
     def ask(question)
       print "#{question}: "
-      STDIN.gets.strip
+      str = STDIN.gets
+
+      if str.nil?
+        raise CannotReadStdinError, "Cannot read from stdin. This could mean that you're trying to run the tool in a "\
+          "non-interactive mode. Please run the tool in an interactive mode or provide all the required options as command line arguments."
+      end
+
+      str.strip
     end
 
     def ask_yes_or_no(question, default)

app/core/patchkit_error.rb

@@ -20,4 +20,10 @@ class APIJobError < APIError
   end
   class APIPublishError < StandardError
   end
+
+  class CannotReadStdinError < StandardError
+    def initialize(message = 'Cannot read from stdin')
+      super(message)
+    end
+  end
 end

app/make_version.rb

@@ -120,16 +120,21 @@ def execute
       validate_source_version! unless mode_files?
 
       if !draft_version.nil?
-        if draft_version.processing_finished_at
-          if !@overwrite_draft && !ask_yes_or_no("A draft version already exists and contains processed content. "\
-                      "Are you sure you want to overwrite it?", "n")
-                      exit
-          end
-        else
-          if !@overwrite_draft && !ask_yes_or_no("A draft version already exists, but no content was uploaded. "\
-                      "Are you sure you want to upload the content and overwrite its metadata?", "y")
-                      exit
+        begin
+          if draft_version.processing_finished_at
+            if !@overwrite_draft && !ask_yes_or_no("A draft version already exists and contains processed content. "\
+                        "Are you sure you want to overwrite it?", "n")
+                        exit
+            end
+          else
+            if !@overwrite_draft && !ask_yes_or_no("A draft version already exists, but no content was uploaded. "\
+                        "Are you sure you want to upload the content and overwrite its metadata?", "y")
+                        exit
+            end
           end
+        rescue CannotReadStdinError => e
+          raise CannotReadStdinError, "Running in non-interactive mode and not allowed to overwrite draft version. "\
+                                      "Please use --overwrite-draft option or run in interactive mode."
         end
       else
         create_draft_version!