add specs for default_string (arg and flag)

Author: DannyBen

.github/workflows/test.yml

@@ -47,7 +47,7 @@ jobs:
       run: bundle exec rspec
 
   static_analysis:
-    name: Static analysis of Example files
+    name: Example files static analysis
 
     runs-on: ubuntu-latest
 
@@ -85,7 +85,7 @@ jobs:
       run: bundle exec run shfmt
 
   json_schema:
-    name: Validate JSON schemas
+    name: JSON schemas validation
 
     runs-on: ubuntu-latest
 

spec/bashly/script/argument_spec.rb

@@ -6,6 +6,32 @@
 
   let(:fixture) { :basic_argument }
 
+  describe '#default_string' do
+    context 'when default is an array' do
+      let(:fixture) { :default_array }
+
+      it 'returns a shell-escaped string suitable to be shell array source' do
+        expect(subject.default_string).to eq 'spaced\\ one two'
+      end
+    end
+
+    context 'when default is string and repeatable is false' do
+      let(:fixture) { :default_string }
+
+      it 'returns it as is' do
+        expect(subject.default_string).to eq 'spaced one'
+      end
+    end
+
+    context 'when default is string and repeatable is true' do
+      let(:fixture) { :default_string_with_repeatable }
+
+      it 'returns a single string' do
+        expect(subject.default_string).to eq 'spaced\\ one'
+      end
+    end
+  end
+
   describe '#usage_string' do
     it 'returns a string suitable to be used as a usage pattern' do
       expect(subject.usage_string).to eq '[FILE]'

spec/bashly/script/flag_spec.rb

@@ -30,6 +30,32 @@
     end
   end
 
+  describe '#default_string' do
+    context 'when default is an array' do
+      let(:fixture) { :default_array }
+
+      it 'returns a shell-escaped string suitable to be shell array source' do
+        expect(subject.default_string).to eq 'spaced\\ one two'
+      end
+    end
+
+    context 'when default is string and repeatable is false' do
+      let(:fixture) { :default_string }
+
+      it 'returns it as is' do
+        expect(subject.default_string).to eq 'spaced one'
+      end
+    end
+
+    context 'when default is string and repeatable is true' do
+      let(:fixture) { :default_string_with_repeatable }
+
+      it 'returns a single string' do
+        expect(subject.default_string).to eq 'spaced\\ one'
+      end
+    end
+  end
+
   describe '#name' do
     context 'with both short and long options' do
       it 'returns the long option' do

spec/fixtures/script/arguments.yml

@@ -8,3 +8,18 @@
 :repeatable:
   name: file
   repeatable: true
+
+:default_string:
+  name: file
+  repeatable: false
+  default: spaced one
+
+:default_array:
+  name: file
+  repeatable: true
+  default: [spaced one, two]
+
+:default_string_with_repeatable:
+  name: file
+  repeatable: true
+  default: spaced one

spec/fixtures/script/flags.yml

@@ -25,4 +25,22 @@
 :completions_completions:
   long: --path
   short: -p
-  completions: [<file>, README.md]
+  completions: [<file>, README.md]
+
+:default_string:
+  long: --file
+  arg: path
+  repeatable: false
+  default: spaced one
+
+:default_array:
+  long: --file
+  arg: path
+  repeatable: true
+  default: [spaced one, two]
+
+:default_string_with_repeatable:
+  long: --file
+  arg: path
+  repeatable: true
+  default: spaced one