Merge pull request #121 from DannyBen/finalize/pr-114

Finalize #114

Author: DannyBen

lib/bashly/models/script.rb

@@ -29,10 +29,16 @@ def header!
         if File.exist? custom_header_path
           File.read custom_header_path
         else
-          render('header')
+          default_header
         end
       end
 
+      def default_header
+        result = render('header')
+        result += render('bash3_bouncer') unless function_name
+        result
+      end
+
       def body
         @body ||= command.render('master_script')
       end

lib/bashly/views/command/initialize.erb

@@ -4,10 +4,5 @@ initialize() {
   long_usage=''
   set -e
 
-  if [[ "${BASH_VERSINFO:-0}" -lt 4 ]]; then
-    printf "<%= strings[:unsupported_bash_version] -%>\n"
-    exit 1
-  fi
-
 <%= load_user_file("initialize.sh", placeholder: false).indent 2 %>
 }

lib/bashly/views/script/bash3_bouncer.erb

@@ -0,0 +1,5 @@
+# :script.bash3_bouncer
+if [[ "${BASH_VERSINFO:-0}" -lt 4 ]]; then
+  printf "<%= strings[:unsupported_bash_version] -%>\n"
+  exit 1
+fi

lib/bashly/views/script/header.erb

@@ -1,3 +1,4 @@
 #!/usr/bin/env bash
 # This script was generated by bashly (https://github.com/DannyBen/bashly)
 # Modifying it manually is not recommended
+

spec/approvals/cli/generate/wrap-script

@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+# This script was generated by bashly (https://github.com/DannyBen/bashly)
+# Modifying it manually is not recommended
+
+# :script.wrapper
+function() {
+
+  # :command.version_command
+  version_command() {
+    echo "$version"
+  }

spec/approvals/examples/bash-3-syntax

@@ -0,0 +1,15 @@
+creating user files in src
+skipped src/initialize.sh (exists)
+skipped src/test_command.sh (exists)
+created ./cli
+run ./cli --help to test your bash script
+cli - Sample application
+
+Usage:
+  cli [command]
+  cli [command] --help | -h
+  cli --version | -v
+
+Commands:
+  test   Run test
+

spec/approvals/models/script/code

@@ -0,0 +1,13 @@
+#!/usr/bin/env bash
+# This script was generated by bashly (https://github.com/DannyBen/bashly)
+# Modifying it manually is not recommended
+
+# :script.bash3_bouncer
+if [[ "${BASH_VERSINFO:-0}" -lt 4 ]]; then
+  printf "bash version 4 or higher is required\n"
+  exit 1
+fi
+
+# :command.root_command
+root_command() {
+  # :spec/tmp/src/root_command.sh

spec/approvals/models/script/code-wrapped

@@ -0,0 +1,13 @@
+#!/usr/bin/env bash
+# This script was generated by bashly (https://github.com/DannyBen/bashly)
+# Modifying it manually is not recommended
+
+# :script.wrapper
+my_super_function() {
+  # :command.root_command
+  root_command() {
+    # :spec/tmp/src/root_command.sh
+    echo "error: cannot load file"
+  }
+
+  # :command.version_command

spec/bashly/commands/generate_spec.rb

@@ -75,11 +75,11 @@
       expect(success).to be true
     end
 
-    it "generates the cli script wrapped in a function" do
+    it "generates the cli script wrapped in a function without bash3 bouncer" do
       expect { subject.run %w[generate -w function] }.to output_approval('cli/generate/wrap-function')
       expect(File).to exist(cli_script)
       lines = File.readlines cli_script
-      expect(lines[5]).to eq "function() {\n"
+      expect(lines[0..10].join).to match_approval('cli/generate/wrap-script')
     end
   end
 

spec/bashly/integration/bash_spec.rb

@@ -5,9 +5,9 @@
     before { system "docker pull bash:3 >/dev/null" }
 
     it "errors gracefully" do
-      command = "docker run --rm -v $PWD:/app bash:3 bash /app/download"
+      command = "docker run --rm -v $PWD:/app bash:3 bash /app/cli"
 
-      Dir.chdir "examples/minimal" do
+      Dir.chdir "spec/fixtures/workspaces/bash-3-syntax" do
         system "bashly generate 2>&1 >/dev/null"
         expect(`#{command} 2>&1`).to match_approval('bash/error')
       end