add specs for LibrarySourceConfig

lib/bashly/library_source_config.rb

@@ -30,7 +30,7 @@ def assert_root(path, value)
 
     def assert_lib(key, value)
       assert_string "#{key}.help", value['help']
-      
+
       assert_optional_string "#{key}.usage", value['usage']
       assert_optional_string "#{key}.handler", value['handler']
       assert_optional_string "#{key}.post_install_message", value['post_install_message']

spec/approvals/libraries_validation/files_array

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/files_array.yml] database.files must be an array>

spec/approvals/libraries_validation/files_source

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/files_source.yml] database.files[0].source must be a string>

spec/approvals/libraries_validation/files_target

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/files_target.yml] database.files[0].target must be a string>

spec/approvals/libraries_validation/handler_string

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/handler_string.yml] database.handler must be a string>

spec/approvals/libraries_validation/help_string

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/help_string.yml] database.help must be a string>

spec/approvals/libraries_validation/message_string

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/message_string.yml] database.post_install_message must be a string>

spec/approvals/libraries_validation/root_hash

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: spec/fixtures/libraries/errors/root_hash.yml must be a hash>

spec/approvals/libraries_validation/usage_string

@@ -0,0 +1 @@
+#<Bashly::ConfigurationError: [spec/fixtures/libraries/errors/usage_string.yml] database.usage must be a string>

spec/bashly/config_validator_spec.rb

@@ -1,14 +1,13 @@
 describe ConfigValidator do
   fixtures = load_fixture 'script/validations'
-  subject { described_class.new options }
 
   describe '#validate' do
     fixtures.each do |fixture, options|
-      context "with :#{fixture}" do
-        let(:options) { options }
+      validator = described_class.new options
 
+      context "with :#{fixture}" do
         it 'raises an error' do
-          expect { subject.validate }.to raise_approval("validations/#{fixture}")
+          expect { validator.validate }.to raise_approval("validations/#{fixture}")
         end
       end
     end

spec/bashly/library_source_config_spec.rb

@@ -0,0 +1,26 @@
+describe LibrarySourceConfig do
+  fixtures = Dir['spec/fixtures/libraries_errors/*.yml']
+  subject { described_class.new path }
+
+  let(:path) { 'spec/fixtures/libraries/libraries.yml' }
+
+  describe '#validated_data' do
+    it 'returns the data after validating it' do
+      expect(subject).to receive(:validate)
+      expect(subject.validated_data).to eq subject.data
+    end
+  end
+
+  describe '#validate' do
+    fixtures.each do |path|
+      fixture_name = File.basename path, '.yml'
+      config = described_class.new path
+
+      context "with #{fixture_name}.yml" do
+        it 'raises an error' do
+          expect { config.validate }.to raise_approval("libraries_validation/#{fixture_name}")
+        end
+      end
+    end
+  end
+end

spec/fixtures/libraries_validation/files_array.yml

@@ -0,0 +1,4 @@
+# error: files must be an array
+database:
+  help: Help text
+  files: one_file.txt

spec/fixtures/libraries_validation/files_source.yml

@@ -0,0 +1,6 @@
+# error: files must have a source
+database:
+  help: Help text
+  files:
+    - target: some_target
+

spec/fixtures/libraries_validation/files_target.yml

@@ -0,0 +1,6 @@
+# error: files must have a target
+database:
+  help: Help text
+  files:
+    - source: some_source
+

spec/fixtures/libraries_validation/handler_string.yml

@@ -0,0 +1,7 @@
+# error: handler must be a string (optional)
+database:
+  help: Help text
+  handler: 1
+  files:
+    - source: "database/database.sh"
+      target: "%{user_lib_dir}/database.%{user_ext}"

spec/fixtures/libraries_validation/help_string.yml

@@ -0,0 +1,6 @@
+# error: help must be a string
+database:
+  help: 1
+  files:
+    - source: "database/database.sh"
+      target: "%{user_lib_dir}/database.%{user_ext}"

spec/fixtures/libraries_validation/message_string.yml

@@ -0,0 +1,7 @@
+# error: post_install_message must be a string (optional)
+database:
+  help: Help text
+  post_install_message: 1
+  files:
+    - source: "database/database.sh"
+      target: "%{user_lib_dir}/database.%{user_ext}"

spec/fixtures/libraries_validation/root_hash.yml

@@ -0,0 +1,6 @@
+# error: must be a hash
+- database:
+    help: Add database utilities
+    files:
+      - source: "database/database.sh"
+        target: "%{user_lib_dir}/database.%{user_ext}"

spec/fixtures/libraries_validation/usage_string.yml

@@ -0,0 +1,7 @@
+# error: usage must be a string (optional)
+database:
+  help: Help text
+  usage: 1
+  files:
+    - source: "database/database.sh"
+      target: "%{user_lib_dir}/database.%{user_ext}"