update config validator to assert arg.unique

Author: DannyBen

lib/bashly/config_validator.rb

@@ -95,12 +95,17 @@ def assert_arg(key, value)
       assert_optional_string "#{key}.validate", value['validate']
       assert_boolean "#{key}.required", value['required']
       assert_boolean "#{key}.repeatable", value['repeatable']
+      assert_boolean "#{key}.unique", value['unique']
 
       assert_array "#{key}.allowed", value['allowed'], of: :string
 
       refute value['name'].match(/^-/), "#{key}.name must not start with '-'"
 
       refute value['required'] && value['default'], "#{key} cannot have both nub`required` and nub`default`"
+
+      if value['unique']
+        assert value['repeatable'], "#{key}.unique does not make sense without nub`repeatable`"
+      end
     end
 
     def assert_flag(key, value)
@@ -143,8 +148,7 @@ def assert_flag(key, value)
       end
 
       if value['unique']
-        condition = value['arg'] && value['repeatable']
-        assert condition, "#{key}.unique does not make sense without nub`arg` and nub`repeatable`"
+        assert value['arg'] && value['repeatable'], "#{key}.unique does not make sense without nub`arg` and nub`repeatable`"
       end
     end
 

spec/approvals/validations/arg_unique_without_repeatable

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: root.args[0].unique does not make sense without nub`repeatable`>

spec/fixtures/script/validations.yml

@@ -42,6 +42,14 @@
   - name: target
     help: Target filename
 
+:arg_unique_without_repeatable:
+  name: invalid
+  help: arg must be repeatable when using unique
+  args:
+  - name: target
+    help: Target filename
+    unique: true
+
 :command_catch_all_type:
   name: invalid
   help: catch_all must be boolean, string, or hash