1 files changed, 95 insertions, 0 deletions
diff --git a/spec/lib/gitlab/config/entry/factory_spec.rb b/spec/lib/gitlab/config/entry/factory_spec.rb
new file mode 100644
index 00000000000..c29d17eaee3
--- /dev/null
+++ b/spec/lib/gitlab/config/entry/factory_spec.rb
@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+describe Gitlab::Config::Entry::Factory do
+  describe '#create!' do
+    class Script < Gitlab::Config::Entry::Node
+      include Gitlab::Config::Entry::Validatable
+
+      validations do
+        validates :config, array_of_strings: true
+      end
+    end
+
+    let(:entry) { Script }
+    let(:factory) { described_class.new(entry) }
+
+    context 'when setting a concrete value' do
+      it 'creates entry with valid value' do
+        entry = factory
+          .value(%w(ls pwd))
+          .create!
+
+        expect(entry.value).to eq %w(ls pwd)
+      end
+
+      context 'when setting description' do
+        it 'creates entry with description' do
+          entry = factory
+            .value(%w(ls pwd))
+            .with(description: 'test description')
+            .create!
+
+          expect(entry.value).to eq %w(ls pwd)
+          expect(entry.description).to eq 'test description'
+        end
+      end
+
+      context 'when setting key' do
+        it 'creates entry with custom key' do
+          entry = factory
+            .value(%w(ls pwd))
+            .with(key: 'test key')
+            .create!
+
+          expect(entry.key).to eq 'test key'
+        end
+      end
+
+      context 'when setting a parent' do
+        let(:object) { Object.new }
+
+        it 'creates entry with valid parent' do
+          entry = factory
+            .value('ls')
+            .with(parent: object)
+            .create!
+
+          expect(entry.parent).to eq object
+        end
+      end
+    end
+
+    context 'when not setting a value' do
+      it 'raises error' do
+        expect { factory.create! }.to raise_error(
+          Gitlab::Config::Entry::Factory::InvalidFactory
+        )
+      end
+    end
+
+    context 'when creating entry with nil value' do
+      it 'creates an unspecified entry' do
+        entry = factory
+          .value(nil)
+          .create!
+
+        expect(entry)
+          .not_to be_specified
+      end
+    end
+
+    context 'when passing metadata' do
+      let(:entry) { spy('entry') }
+
+      it 'passes metadata as a parameter' do
+        factory
+          .value('some value')
+          .metadata(some: 'hash')
+          .create!
+
+        expect(entry).to have_received(:new)
+          .with('some value', { some: 'hash' })
+      end
+    end
+  end
+end