lita-gitlab2jenkins_ghp 0.0.1 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 0908ce7f7863a694a3c39bb3259f8cfcccf42351
-  data.tar.gz: 17ba6f9d09848dfbca5316d81352a5e733866d67
+  metadata.gz: 6bba63e1a106fd9ebd7b3774b0bd2d5e05c8a49b
+  data.tar.gz: ae494b139b17f091b0f37378c679064424576948
 SHA512:
-  metadata.gz: c61e93f9e8cbbc44df5ffb918f26eeb25dfdf2b69d4ec34c798f61f763c13acd92a29d293a30102cd5d8a25bc064cd6f9c7dc58aa997201482651e6fa8886ae1
-  data.tar.gz: 895512be891ee28d08d326ec0a14b2ee2c28af861ea3921d9454d59a9f725939bf3f0ec8d2598566db956472b45ea6c38e1f6eb0b8450b088f9f37eec158356c
+  metadata.gz: b00508be370616f56cc348b8db4b5201b33e30fe2bad98ace1a4e729b8c5888f5cbd050da066862ab604a78a2de36e4aca4b58e243d0deb8f0b1ac14c75d717a
+  data.tar.gz: da05910f3094061400fb98614b4b4096681b7291667af966fed84624b48e203a952904a7fb592c13750d175b76262b1c6b038820725f69a012fe1696eccbc5e9

data/.ruby-version

@@ -1 +1 @@
-2.1.0
+2.0.0-p451

data/README.md

@@ -1,6 +1,12 @@
 # lita-gitlab2jenkins_ghp
 
-TODO: Add a description of the plugin.
+[![Build Status](https://travis-ci.org/jcalonsoh/lita-gitlab2jenkins_ghp.svg)](https://travis-ci.org/jcalonsoh/lita-gitlab2jenkins_ghp)
+[![Code Climate](https://codeclimate.com/github/jcalonsoh/lita-gitlab2jenkins_ghp.png)](https://codeclimate.com/github/jcalonsoh/lita-gitlab2jenkins_ghp)
+[![Coverage Status](https://coveralls.io/repos/jcalonsoh/lita-gitlab2jenkins_ghp/badge.png)](https://coveralls.io/r/jcalonsoh/lita-gitlab2jenkins_ghp)
+
+**lita-fitlab2jenkins_ghp** is a [Lita](https://github.com/jimmycuadra/lita) that uses [GitLab](https://www.gitlab.com/gitlab-ce/)
+webhooks and ci-status to be build with [Jenkins](http://jenkins-ci.org/).
+
 
 ## Installation
 
@@ -12,17 +18,37 @@ gem "lita-gitlab2jenkins_ghp"
 
 ## Configuration
 
-REQUERIMENTS:
+### REQUERIMENTS:
 
-from Jenkins =>
-Gitlab Hook Plugin              1.0.0
-Embeddable Build Status         1.4.0
+You will need to install on Jenkins the next plugins =>
 
+[Gitlab Hook Plugin 1.0.0](https://wiki.jenkins-ci.org/display/JENKINS/Gitlab+Hook+Plugin),
+[Embeddable Build Status 1.4](https://wiki.jenkins-ci.org/display/JENKINS/Embeddable+Build+Status+Plugin),
+[Notification Plugin 1.5](https://wiki.jenkins-ci.org/display/JENKINS/Notification+Plugin)
 
-## Usage
+### Example usage YML
 
-TODO: Describe the plugin's features and how to use them.
+``` ruby
+Lita.configure do |config|
+  config.handlers.gitlab2jenkins_ghp.room                 = '#error_channel'
+  config.handlers.gitlab2jenkins_ghp.url_gitlab           = 'http://gitlab.example.com'
+  config.handlers.gitlab2jenkins_ghp.url_jenkins          = 'http://jenkins.example.com'
+  config.handlers.gitlab2jenkins_ghp.url_jenkins_hook     = '/gitlab/build_now'
+  config.handlers.gitlab2jenkins_ghp.url_jenkins_img      = '/buildStatus/icon?job='
+  config.handlers.gitlab2jenkins_ghp.url_jenkins_icon     = '/static/843013a3/images/jenkins.png'
+  config.handlers.gitlab2jenkins_ghp.private_token_gitlab = 'some_gitlab_token_from_admin_user'
+  config.redis.host                                       = 'redis.example.com'
+end
+```
+
+For more understanding please read [wiki](https://github.com/jcalonsoh/lita-gitlab2jenkins_ghp/wiki)
 
 ## License
 
 [MIT](http://opensource.org/licenses/MIT)
+
+## TODO:
+
+* Make tests for GET scenarios when GitLab queries ci status
+* Refactoring gem to nit loggic flow
+* Add scenario for "Tag push events" on next version

data/lib/lita/handlers/gitlab2jenkins_ghp.rb

@@ -1,41 +1,67 @@
 module Lita
   module Handlers
     class Gitlab2jenkinsGhp < Handler
+
       def self.default_config(config)
-        config.room                  = '#test2'
+        config.room                  = '#test'
         config.group                 = 'group_name'
         config.gitlab_url            = 'http://example.gitlab'
         config.private_token         = 'orwejnweuf'
         config.jenkins_url           = 'http://example.jenkins'
         config.jenkins_hook          = '/gitlab/build'
-        config.saved_request         = ''
       end
 
-      http.post '/lita/gitlab2jenkinsghp', :do_mr
-
-      http.post '/lita/gitlab2jenkinsghp_mr_status', :do_mr_change_status
-
-      http.get '/lita/gitlab2jenkinsghp_ci_status/*id_project/builds/*sha_commit', :do_ci_change_status
+      http.post '/lita/gitlab2jenkinsghp/*to_route', :receive
 
-      http.get '/lita/gitlab2jenkinsghp_ci_status/*id_project', :do_img_jenkins
+      http.get '/lita/gitlab2jenkinsghp/*to_route/*id_project/builds/*sha_commit', :receive
 
-      def do_mr(request, response)
-        json_body = extract_json_from_request(request)
-        Lita.logger.info("Payload: #{json_body}")
-        data = symbolize parse_payload(json_body)
-        message = format_message(data, json_body)
+      http.get '/lita/gitlab2jenkinsghp/*to_route/*id_project', :receive
 
+      def receive(request, response)
+        routing_to = request.params['to_route'] || request.env['router.params'][:to_route][0].to_s
+        routing_to_request_method = request.params['request_method'] || request.request_method
+        if routing_to_request_method =='POST'
+          json_body = request.params['payload'] || extract_json_from_request(request)
+          data = symbolize parse_payload(json_body)
+          if data.key? :object_attributes
+            project_name_target = request.params['project_name_target'] || git_lab_data_project_info(data[:object_attributes][:target_project_id])['name']
+            project_source_id_commit = request.params['project_source_id_commit'] || git_lab_data_branch_info(data[:object_attributes][:source_project_id], data[:object_attributes][:source_branch])['commit']['id']
+          else
+            project_name_target = ''
+            project_source_id_commit = ''
+          end
+          if routing_to == 'gitlab'       #receive hook from gitlab to report commits and merge request event
+            do_mr(json_body, data, project_name_target, project_source_id_commit)
+          elsif routing_to == 'jenkins'   #receive hook from jenkins to report job status build
+            project_name_params = request.params['project_name_params']
+            do_mr_change_status(request, response, json_body, data, project_name_params)
+          end
+        elsif routing_to_request_method =='GET'
+          if routing_to == 'ci_status'
+            if request.params['ref'] == 'master'
+              # if request.env['router.params'][:sha_commit][0].nil? false
+              #   do_ci_change_status(request, response)
+              do_img_jenkins(request, response)
+            else
+              message = do_ci_change_status(request, response)
+              if message.to_s != 'true'
+                response['status'] = message
+                Lita.logger.info "CI Final Status: #{message.to_s}"
+              end
+              # do_img_jenkins(request, response)
+            end
+          end
+        end
+          # robot.send_message(target, message)
       rescue Exception => e
-        Lita.logger.error "Could not domr: #{e.inspect}"
+        Lita.logger.error "Could not receive: #{e.inspect}"
       end
 
-      def do_mr_change_status(request, response)
-        json_body = extract_json_from_request(request)
+      def do_mr_change_status(request, response, json_body, data, project_name_params) # do_mr_change_status(request, response, json_body, data, project_name_params)
         id_project = request.params['id_project'].to_s
         Lita.logger.info "Jenkins Proyect: #{id_project}"
         Lita.logger.info("Payload: #{json_body}")
-        data = symbolize parse_payload(json_body)
-        message = format_message_mr(data, json_body, id_project)
+        ci_status_setter(data, json_body, id_project, project_name_params)
 
       rescue Exception => e
         Lita.logger.error "Could not domr_change_status: #{e.inspect}"
@@ -45,13 +71,8 @@ module Lita
         Lita.logger.info "GitLab CI Project ID: #{request.env['router.params'][:id_project]}"
         Lita.logger.info "GitLab CI Commit SHA: #{request.env['router.params'][:sha_commit]}"
         Lita.logger.info "GitLab CI Token: #{request.params['token']}"
-        message = format_message_ci(request.env['router.params'][:id_project][0], request.env['router.params'][:sha_commit][0])
-        room = Lita.config.handlers.gitlab2jenkins_ghp.room
-        target = Source.new(room: room)
-        if message.to_s != 'true'
-          response['status'] = message
-          Lita.logger.info "CI Final Status: #{message.to_s}"
-        end
+        format_message_ci(request.env['router.params'][:id_project][0], request.env['router.params'][:sha_commit][0])
+
       rescue Exception => e
         Lita.logger.error "Could not domr_change_status: #{e.inspect}"
       end
@@ -59,9 +80,17 @@ module Lita
       def do_img_jenkins(request, response)
         project_name = git_lab_data_project_info(request.env['router.params'][:id_project][0])['name']
 
-        gkeys = @redis.keys("mr:#{project_name}:*")
+        gkeys = @redis.keys("jenkins:#{project_name}:*")
+        gkeys.each do |key|
+          json_off = redis.get(key)
+          jdata = symbolize parse_payload(json_off)
+          job = jdata[:name]
+          code_climate(job, response)
+        end
 
-        code_climate('QA-API', response)
+        if gkeys.empty?
+          nocode_climate(response)
+        end
 
       rescue Exception => e
         Lita.logger.error "Could not do_img_jenkins: #{e.inspect}"
@@ -85,6 +114,22 @@ module Lita
         Lita.logger.error "Could not do_img_jenkins: #{e.inspect}"
       end
 
+      def nocode_climate(response)
+        url = "#{Lita.config.handlers.gitlab2jenkins_ghp.url_jenkins.to_s}"<<"#{Lita.config.handlers.gitlab2jenkins_ghp.url_jenkins_icon.to_s}"
+        uri = URI(url)
+        res = Net::HTTP.get_response(uri)
+
+        if res.is_a?(Net::HTTPSuccess)
+          response.body << res.body
+          response['Content-Type'] = res['Content-Type']
+        end
+
+        Lita.logger.info "Sending Jenkins to Gitlab Code Climate #{url}"
+
+      rescue Exception => e
+        Lita.logger.error "Could not do_img_jenkins: #{e.inspect}"
+      end
+
       def extract_json_from_request(request)
         request.body.rewind
         request.body.read
@@ -103,21 +148,20 @@ module Lita
         obj
       end
 
-      def format_message(data, json)
+      def do_mr(json_body, data, project_name_target, project_source_id_commit)
         if data.key? :before
-          build_branch_hook(data, json)
+          build_branch_hook(json_body, data)
         elsif data.key? :object_kind
-          build_merge_hook(data, json)
+          build_merge_hook(json_body, data, project_name_target, project_source_id_commit)
         end
 
       rescue => e
         Lita.logger.error "Could not format message: #{e.inspect}"
       end
 
-      def build_branch_hook(data, json)
-        data[:link] = "<#{data[:repository][:homepage]}|#{data[:repository][:name]}>"
+      def build_branch_hook(json, data)
         redis.set("commit:#{data[:commits][0][:id]}", json.to_s)
-        "Commit Stored: #{data[:commits][0][:id]}"
+        return "Commit Stored: #{data[:commits][0][:id]}"
       end
 
       def gitlab_rescue_commit(project_id, branch)
@@ -156,19 +200,21 @@ module Lita
         parse_payload((((rescue_gitlab_project_name(id)).to_hash)[:body]))
       end
 
-      def build_merge_hook(data, json)
-        redis.set("mr:#{git_lab_data_project_info(data[:object_attributes][:target_project_id])['name']}:#{data[:object_attributes][:id]}", json.to_s)
+      def build_merge_hook(json, data, project_name_target, project_source_id_commit)
+        redis.set("mr:#{project_name_target}:#{data[:object_attributes][:id]}", json.to_s)
         if ['reopened', 'opened'].include? data[:object_attributes][:state]
-          Lita.logger.info "It's a merge request"
-          payload_rescue = redis.get("commit:#{git_lab_data_branch_info(data[:object_attributes][:source_project_id], data[:object_attributes][:source_branch])['commit']['id']}")
+          Lita.logger.info "It's a merge request for #{project_source_id_commit}"
+          payload_rescue = redis.get("commit:#{project_source_id_commit}")
           if (payload_rescue).size > 0
             Lita.logger.info "Merge request found"
             jenkins_hook_ghp(payload_rescue)
+            "Merge Request found to play #{payload_rescue}"
+          else
+            "No commit found"
           end
         else
-          redis.del("mr:#{git_lab_data_project_info(data[:object_attributes][:target_project_id])['name']}:#{data[:object_attributes][:id]}")
-          jproject_name = git_lab_data_project_info(data[:object_attributes][:target_project_id])['name']
-          gkeys = @redis.keys("jenkins:#{jproject_name}:*")
+          redis.del("mr:#{project_name_target}:#{data[:object_attributes][:id]}")
+          gkeys = @redis.keys("jenkins:#{project_name_target}:*")
           gkeys.each do |key|
             Lita.logger.info "Jenkins Key for Delete: #{key}"
             json_off = redis.get(key)
@@ -176,29 +222,33 @@ module Lita
             Lita.logger.info "Branch found for delete: #{jdata[:build][:parameters][:ANY_BRANCH_PATTERN]}"
             redis.del(key) if data[:object_attributes][:source_branch] == jdata[:build][:parameters][:ANY_BRANCH_PATTERN]
           end
+          "Merge request for delete"
         end
 
       rescue Exception => e
         Lita.logger.error "Could not make Build Merge Reques #{e.inspect}"
       end
 
-      def key_value_source_project_finder_mr(source_project, project_name)
-        gkeys = @redis.keys("mr:#{project_name}:*")
-        gkeys.each do |key|
-          json = redis.get(key)
-          data = symbolize parse_payload(json)
-          source_project_id = data[:object_attributes][:source_project_id] if data[:object_attributes][:source_branch] == source_project
+      # def key_value_source_project_finder_mr(source_project, project_name)
+      #   gkeys = @redis.keys("mr:#{project_name}:*")
+      #   gkeys.each do |key|
+      #     json = redis.get(key)
+      #     data = symbolize parse_payload(json)
+      #     source_project_id = data[:object_attributes][:source_project_id] if data[:object_attributes][:source_branch] == source_project
+      #   end
+      #
+      # rescue Exception => e
+      #   Lita.logger.error "Could not key_value_source_project_finder_mr #{e.inspect}"
+      # end
+
+      def ci_status_setter(data, json, project_id, project_name_params)               # ci_status_setter(data, json_body, id_project, project_name_params)
+        if project_name_params.nil?
+          project_name = git_lab_data_project_info(project_id)['name']
+          redis.set("jenkins:#{project_name}:#{data[:build][:number]}", json)
+        else
+          redis.set("jenkins:#{project_name_params}:#{data[:build][:number]}", json)
         end
 
-      rescue Exception => e
-        Lita.logger.error "Could not key_value_source_project_finder_mr #{e.inspect}"
-      end
-
-      def format_message_mr(data, json, id)
-        project_name = git_lab_data_project_info(id)['name']
-        source_project_id = key_value_source_project_finder_mr(data[:build][:parameters]['ANY_BRANCH_PATTERN'],project_name)
-        redis.set("jenkins:#{project_name}:#{data[:build][:number]}", json)
-
       rescue Exception => e
         Lita.logger.error "Could not format_message_mr #{e.inspect}"
       end

data/lita-gitlab2jenkins_ghp.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |spec|
   spec.name          = 'lita-gitlab2jenkins_ghp'
-  spec.version       = '0.0.1'
+  spec.version       = '1.0.0'
   spec.authors       = ['Juan Carlos Alonso Holmstron']
   spec.email         = ['j.carlos.alonso.h@gmail.com']
   spec.description   = %q{A Lita Handler for storing gitlabs's pushed events for after send them in a merge request trigger}
@@ -15,8 +15,6 @@ Gem::Specification.new do |spec|
   spec.require_paths = ['lib']
 
   spec.add_runtime_dependency 'lita', '>= 3.0'
-  spec.add_runtime_dependency 'redis', '~> 3.0.7'
-  spec.add_runtime_dependency 'gitlab', '~> 3.0.0'
 
   spec.add_development_dependency 'bundler', '~> 1.3'
   spec.add_development_dependency 'rake'

data/spec/fixtures/gitlab/merge_request_event_closed.json

@@ -0,0 +1,21 @@
+{
+    "object_kind":"merge_request",
+    "object_attributes":
+    {
+        "id":82,
+        "target_branch":"master",
+        "source_branch":"testing_hooking_lita",
+        "source_project_id":36,
+        "author_id":2,
+        "assignee_id":null,
+        "title":"Testing Hooking Lita",
+        "created_at":"2014-04-08T15:44:37.038Z",
+        "updated_at":"2014-04-08T17:01:06.705Z",
+        "milestone_id":null,
+        "state":"closed",
+        "merge_status":"can_be_merged",
+        "target_project_id":7,
+        "iid":37,
+        "description":""
+    }
+}

data/spec/fixtures/gitlab/merge_request_event_open.json

@@ -0,0 +1,21 @@
+{
+    "object_kind":"merge_request",
+    "object_attributes":
+    {
+        "id":82,
+        "target_branch":"master",
+        "source_branch":"testing_hooking_lita",
+        "source_project_id":36,
+        "author_id":2,
+        "assignee_id":null,
+        "title":"Testing Hooking Lita",
+        "created_at":"2014-04-08T15:44:37.038Z",
+        "updated_at":"2014-04-08T17:01:06.705Z",
+        "milestone_id":null,
+        "state":"opened",
+        "merge_status":"can_be_merged",
+        "target_project_id":7,
+        "iid":37,
+        "description":""
+    }
+}

data/spec/fixtures/gitlab/merge_request_event_reopened.json

@@ -0,0 +1,21 @@
+{
+    "object_kind":"merge_request",
+    "object_attributes":
+    {
+        "id":82,
+        "target_branch":"master",
+        "source_branch":"testing_hooking_lita",
+        "source_project_id":36,
+        "author_id":2,
+        "assignee_id":null,
+        "title":"Testing Hooking Lita",
+        "created_at":"2014-04-08T15:44:37.038Z",
+        "updated_at":"2014-04-08T17:01:06.705Z",
+        "milestone_id":null,
+        "state":"reopened",
+        "merge_status":"can_be_merged",
+        "target_project_id":7,
+        "iid":37,
+        "description":""
+    }
+}

data/spec/fixtures/gitlab/push_event_commit.json

@@ -0,0 +1,27 @@
+{
+    "before":"a21ad2af8142546d837d8b811b8c60c60746147e",
+    "after":"cfa9e9a8421e45407d03dc8d2e0ac0e80ea9dba1",
+    "ref":"refs/heads/testing_hooking_lita",
+    "user_id":2,
+    "user_name":"jcalonsoh",
+    "project_id":36,
+    "repository":
+    {
+        "name":"api",
+        "url":"git@gitlab.bbrmobile.me:jcalonso/api.git",
+        "description":"",
+        "homepage":"http://gitlab.bbrmobile.me/jcalonso/api"},
+        "commits":
+            [{
+                "id":"cfa9e9a8421e45407d03dc8d2e0ac0e80ea9dba1",
+                "message":"setting database.yml.sample pool to 12",
+                "timestamp":"2014-04-09T16:00:54-03:00",
+                "url":"http://gitlab.bbrmobile.me/jcalonso/api/commit/cfa9e9a8421e45407d03dc8d2e0ac0e80ea9dba1",
+                "author":
+                {
+                    "name":"Juan Carlos Alonso Holmstron",
+                    "email":"jcalonso@bbr.cl"
+                }
+            }],
+    "total_commits_count":1
+}

data/spec/fixtures/jenkins/job_finished.json

@@ -0,0 +1,16 @@
+{
+    "name":"QA-API",
+    "url":"job/QA-API/",
+    "build":
+    {
+        "full_url":"http://jenkins.bbrmobile.me/job/QA-API/130/",
+        "number":130,
+        "phase":"COMPLETED",
+        "status":"SUCCESS",
+        "url":"job/QA-API/130/",
+        "parameters":
+        {
+            "ANY_BRANCH_PATTERN":"master"
+        }
+    }
+}

data/spec/fixtures/jenkins/job_started.json

@@ -0,0 +1,15 @@
+{
+    "name":"QA-API",
+    "url":"job/QA-API/",
+    "build":
+    {
+        "full_url":"http://jenkins.bbrmobile.me/job/QA-API/130/",
+        "number":130,
+        "phase":"STARTED",
+        "url":"job/QA-API/130/",
+        "parameters":
+        {
+            "ANY_BRANCH_PATTERN":"master"
+        }
+    }
+}

data/spec/lita/handlers/gitlab2jenkins_ghp_spec.rb

@@ -1,4 +1,153 @@
 require 'spec_helper'
 
 describe Lita::Handlers::Gitlab2jenkinsGhp, lita_handler: true do
+
+  http_route_path =  '/lita/gitlab2jenkinsghp'
+
+  it 'registers with Lita' do
+    expect(Lita.handlers).to include(described_class)
+  end
+
+  let(:request) do
+    request = double('Rack::Request')
+    allow(request).to receive(:params).and_return(params)
+    request
+  end
+  let(:response) { Rack::Response.new }
+  let(:params) { {} }
+  let(:id_project) { '7' }
+  let(:commit_id) {'cfa9e9a8421e45407d03dc8d2e0ac0e80ea9dba1'}
+  let(:redis_key_commit) {"commit:#{commit_id}"}
+  let(:merge_request) { '82' }
+  let(:to_route_gitlab) { 'gitlab' }
+  let(:request_method_post) { 'POST' }
+  let(:request_method_get) { 'GET' }
+  let(:project_name_target) { 'api' }
+  let(:redis_key_merge_request) {"mr:#{project_name_target}:#{merge_request}"}
+
+  let(:to_route_jenkins) { 'jenkins' }
+  let(:jenkins_id_project) { '7' }
+  let(:redis_key_jenkins_job) { 'jenkins:api:130' }
+  let(:project_name_params) { 'api' }
+  let(:to_route_ci_status) { 'ci_status' }
+
+  describe '#receive' do
+
+    context 'Using POST endpoint for gitlab' do
+
+      it "route gitlab push event POST #{http_route_path}/gitlab to :receive" do
+        routes_http(:post, "#{http_route_path}/#{to_route_gitlab}").to(:receive)
+      end
+
+      context 'when is push event commit hook' do
+        let(:push_event_commit_payload) { fixture_file('gitlab/push_event_commit') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(push_event_commit_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_gitlab)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what commit its stored' do
+          expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_commit}")).to eq push_event_commit_payload
+        end
+      end
+
+      context 'when is a open merge request hook' do
+        let(:merge_request_event_open_payload) { fixture_file('gitlab/merge_request_event_open') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(merge_request_event_open_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_gitlab)
+          allow(params).to receive(:[]).with('project_name_target').and_return(project_name_target)
+          allow(params).to receive(:[]).with('project_source_id_commit').and_return(commit_id)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what merge request its stored' do
+            expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_merge_request}")).to eq merge_request_event_open_payload
+        end
+      end
+
+      context 'when is a reopened merge request hook' do
+        let(:merge_request_event_reopened_payload) { fixture_file('gitlab/merge_request_event_reopened') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(merge_request_event_reopened_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_gitlab)
+          allow(params).to receive(:[]).with('project_name_target').and_return(project_name_target)
+          allow(params).to receive(:[]).with('project_source_id_commit').and_return(commit_id)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what merge request its stored' do
+          expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_merge_request}")).to eq merge_request_event_reopened_payload
+        end
+      end
+
+      context 'when is a closed merge request hook' do
+        let(:merge_request_event_closed_payload) { fixture_file('gitlab/merge_request_event_closed') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(merge_request_event_closed_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_gitlab)
+          allow(params).to receive(:[]).with('project_name_target').and_return(project_name_target)
+          allow(params).to receive(:[]).with('project_source_id_commit').and_return(commit_id)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what merge request its stored' do
+          expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_merge_request}")).to be_nil
+        end
+      end
+    end
+
+    context 'Using POST endpoint for jenkins' do
+      it "route gitlab push event POST #{http_route_path}/jenkins to :receive" do
+        routes_http(:post, "#{http_route_path}/#{to_route_jenkins}").to(:receive)
+      end
+
+      context 'when jenkins post status job when is STARTED' do
+        let(:jenkins_started_payload) { fixture_file('jenkins/job_started') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(jenkins_started_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_jenkins)
+          allow(params).to receive(:[]).with('id_project').and_return(jenkins_id_project)
+          allow(params).to receive(:[]).with('project_name_params').and_return(project_name_params)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what merge request its stored' do
+          expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_jenkins_job}")).to eq jenkins_started_payload
+        end
+      end
+
+      context 'when jenkins post status job when is FINISHED' do
+        let(:jenkins_finished_payload) { fixture_file('jenkins/job_finished') }
+        before do
+          allow(params).to receive(:[]).with('payload').and_return(jenkins_finished_payload)
+          allow(params).to receive(:[]).with('request_method').and_return(request_method_post)
+          allow(params).to receive(:[]).with('to_route').and_return(to_route_jenkins)
+          allow(params).to receive(:[]).with('id_project').and_return(jenkins_id_project)
+          allow(params).to receive(:[]).with('project_name_params').and_return(project_name_params)
+          Lita::Handlers::Gitlab2jenkinsGhp.new(robot).receive(request, response)
+        end
+        it 'notifies what merge request its stored' do
+          expect(Lita.redis.redis.get("lita.test:handlers:gitlab2jenkins_ghp:#{redis_key_jenkins_job}")).to eq jenkins_finished_payload
+        end
+      end
+    end
+
+    context 'Using GET endpoint from gitlab' do
+      it "route gitlab fetch data of a GET #{http_route_path}/ci_status to :receive" do
+        routes_http(:post, "#{http_route_path}/#{to_route_ci_status}").to(:receive)
+      end
+
+      context 'when gitlab get de ci status from commit' do
+          # TODO: Add tests for the case when Gitlab requests the status
+      end
+
+      context 'when gitlab get de ci status from master' do
+        # TODO: Add tests for the case when Gitlab requests the status
+      end
+
+    end
+  end
+
 end

data/spec/spec_helper.rb

@@ -1,10 +1,17 @@
 require 'simplecov'
 require 'coveralls'
+require 'lita-gitlab2jenkins_ghp'
+require 'lita/rspec'
+
 SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
-  SimpleCov::Formatter::HTMLFormatter,
-  Coveralls::SimpleCov::Formatter
+   SimpleCov::Formatter::HTMLFormatter,
+   Coveralls::SimpleCov::Formatter
 ]
+
 SimpleCov.start { add_filter '/spec/' }
 
-require 'lita-gitlab2jenkins_ghp'
-require 'lita/rspec'
+def fixture_file(filename)
+  return '' if filename == ''
+  file_path = File.expand_path("#{File.dirname(__FILE__)}/fixtures/#{filename}.json")
+  File.read(file_path)
+end

metadata

@@ -1,139 +1,111 @@
 --- !ruby/object:Gem::Specification
 name: lita-gitlab2jenkins_ghp
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Juan Carlos Alonso Holmstron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-21 00:00:00.000000000 Z
+date: 2014-05-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lita
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: redis
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.0.7
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.0.7
-- !ruby/object:Gem::Dependency
-  name: gitlab
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.0.0
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: 3.0.0
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ~>
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.0.0.beta2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.0.0.beta2
 - !ruby/object:Gem::Dependency
   name: shoulda
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.5.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: 3.5.0
 - !ruby/object:Gem::Dependency
   name: simplecov
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: A Lita Handler for storing gitlabs's pushed events for after send them
@@ -144,9 +116,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".ruby-version"
-- ".travis.yml"
+- .gitignore
+- .ruby-version
+- .travis.yml
 - Gemfile
 - LICENSE
 - README.md
@@ -155,6 +127,12 @@ files:
 - lib/lita/handlers/gitlab2jenkins_ghp.rb
 - lita-gitlab2jenkins_ghp.gemspec
 - locales/en.yml
+- spec/fixtures/gitlab/merge_request_event_closed.json
+- spec/fixtures/gitlab/merge_request_event_open.json
+- spec/fixtures/gitlab/merge_request_event_reopened.json
+- spec/fixtures/gitlab/push_event_commit.json
+- spec/fixtures/jenkins/job_finished.json
+- spec/fixtures/jenkins/job_started.json
 - spec/lita/handlers/gitlab2jenkins_ghp_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/jcalonsoh/lita-gitlab2jenkins_ghp
@@ -168,21 +146,27 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.0.14
 signing_key: 
 specification_version: 4
 summary: A Lita Handler for storing gitlabs's pushed events for after send them in
   a merge request trigger
 test_files:
+- spec/fixtures/gitlab/merge_request_event_closed.json
+- spec/fixtures/gitlab/merge_request_event_open.json
+- spec/fixtures/gitlab/merge_request_event_reopened.json
+- spec/fixtures/gitlab/push_event_commit.json
+- spec/fixtures/jenkins/job_finished.json
+- spec/fixtures/jenkins/job_started.json
 - spec/lita/handlers/gitlab2jenkins_ghp_spec.rb
 - spec/spec_helper.rb