Add posibility to define a hidden job without 'script' in .gitlab-ci.yml

lib/ci/gitlab_ci_yaml_processor.rb

@@ -63,9 +63,8 @@ module Ci
       @cache = @config[:cache]
       @config.except!(*ALLOWED_YAML_KEYS)
 
-      # anything that doesn't have script is considered as unknown
       @config.each do |name, param|
-        raise ValidationError, "Unknown parameter: #{name}" unless param.is_a?(Hash) && param.has_key?(:script)
+        raise ValidationError, "Unknown parameter: #{name}" unless is_a_job?(name, param)
       end
 
       unless @config.values.any?{|job| job.is_a?(Hash)}
@@ -80,6 +79,12 @@ module Ci
       end
     end
 
+    def is_a_job?(name, value)
+      return true if value.is_a?(Hash) && value.has_key?(:script)
+      return true if name.to_s.start_with?('.')
+      false
+    end
+
     def build_job(name, job)
       {
         stage_idx: stages.index(job[:stage]),
@@ -112,8 +117,6 @@ module Ci
       true
     end
 
-    private
-
     def validate_global!
       unless validate_array_of_strings(@before_script)
         raise ValidationError, "before_script should be an array of strings"

spec/lib/ci/gitlab_ci_yaml_processor_spec.rb

@@ -648,17 +648,10 @@ module Ci
     end
 
     describe "Hidden jobs" do
-      let(:config) do
-        YAML.dump({
-                    '.hidden_job' => { script: 'test' },
-                    'normal_job' => { script: 'test' }
-                  })
-      end
-
       let(:config_processor) { GitlabCiYamlProcessor.new(config) }
-
       subject { config_processor.builds_for_stage_and_ref("test", "master") }
 
+      shared_examples 'hidden_job_handling' do
         it "doesn't create jobs that starts with dot" do
           expect(subject.size).to eq(1)
           expect(subject.first).to eq({
@@ -676,22 +669,40 @@ module Ci
         end
       end
 
-    describe "YAML Alias/Anchor" do
-      it "is correctly supported for jobs" do
-        config = <<EOT
-job1: &JOBTMPL
-  script: execute-script-for-job
+      context 'When hidden job have a script definition' do
+        let(:config) do
+          YAML.dump({
+                      '.hidden_job' => { image: 'ruby:2.1', script: 'test' },
+                      'normal_job' => { script: 'test' }
+                    })
+        end
 
-job2: *JOBTMPL
-EOT
+        it_behaves_like 'hidden_job_handling'
+      end
 
-        config_processor = GitlabCiYamlProcessor.new(config)
+      context "When hidden job doesn't have a script definition" do
+        let(:config) do
+          YAML.dump({
+                      '.hidden_job' => { image: 'ruby:2.1' },
+                      'normal_job' => { script: 'test' }
+                    })
+        end
 
-        expect(config_processor.builds_for_stage_and_ref("test", "master").size).to eq(2)
-        expect(config_processor.builds_for_stage_and_ref("test", "master").first).to eq({
+        it_behaves_like 'hidden_job_handling'
+      end
+    end
+
+    describe "YAML Alias/Anchor" do
+      let(:config_processor) { GitlabCiYamlProcessor.new(config) }
+      subject { config_processor.builds_for_stage_and_ref("build", "master") }
+
+      shared_examples 'job_templates_handling' do
+        it "is correctly supported for jobs" do
+          expect(subject.size).to eq(2)
+          expect(subject.first).to eq({
             except: nil,
-          stage: "test",
-          stage_idx: 1,
+            stage: "build",
+            stage_idx: 0,
             name: :job1,
             only: nil,
             commands: "execute-script-for-job",
@@ -700,10 +711,10 @@ EOT
             when: "on_success",
             allow_failure: false
           })
-        expect(config_processor.builds_for_stage_and_ref("test", "master").second).to eq({
+          expect(subject.second).to eq({
             except: nil,
-          stage: "test",
-          stage_idx: 1,
+            stage: "build",
+            stage_idx: 0,
             name: :job2,
             only: nil,
             commands: "execute-script-for-job",
@@ -715,6 +726,56 @@ EOT
         end
       end
 
+      context 'When template is a job' do
+        let (:config) do
+          <<EOT
+job1: &JOBTMPL
+  stage: build
+  script: execute-script-for-job
+
+job2: *JOBTMPL
+EOT
+        end
+
+        it_behaves_like 'job_templates_handling'
+      end
+
+      context 'When template is a hidden job' do
+        let (:config) do
+          <<EOT
+.template: &JOBTMPL
+  stage: build
+  script: execute-script-for-job
+
+job1: *JOBTMPL
+
+job2: *JOBTMPL
+EOT
+        end
+
+        it_behaves_like 'job_templates_handling'
+      end
+
+      context "When job adds it's own keys to a template definition" do
+        let (:config) do
+          <<EOT
+.template: &JOBTMPL
+  stage: build
+
+job1:
+  <<: *JOBTMPL
+  script: execute-script-for-job
+
+job2:
+  <<: *JOBTMPL
+  script: execute-script-for-job
+EOT
+        end
+
+        it_behaves_like 'job_templates_handling'
+      end
+    end
+
     describe "Error handling" do
       it "fails to parse YAML" do
         expect{GitlabCiYamlProcessor.new("invalid: yaml: test")}.to raise_error(Psych::SyntaxError)