magnum-payload 0.3.4 → 0.3.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c913ae966a34c495971151a1d01b8f42881d05b1
-  data.tar.gz: a0ae86fa239990cb949c7e6d7402958e6a7e540a
+  metadata.gz: 558f53b2e50b86aef9690d16883a12a1f1c8f5e8
+  data.tar.gz: a6197cb92a200599ee7a666324fae0e50fa8c8a4
 SHA512:
-  metadata.gz: 3a6cf68affc2203d60975ccc84d004f3be56bdb1188ea8066db34040a70c1897d154ac47f55692603d20f82d7d18051d97e957e02cca70e79951c021806e7024
-  data.tar.gz: feb1211463ab6c41e9ae16382c76cc8c9d315f7536e38a5601b8e813596376bd3ff8353d8e78fb8318656e15315f2205228dc6de825329e1b322eb87072ac6fe
+  metadata.gz: ac086b98433c9514ca56c2bdcd48a8d7dc9b297e7b4d87540bb281a818d4e2cf082986b5616aa084d2285d44fbb4dd72c9671929050676cc49b77875721c7472
+  data.tar.gz: 90b05ffc95d67e791df3530468ad3f009e60a5e1ae1fba1f6206de932b3f67d82a5a291789f4a78fa280fcdaa0320812b68616fb0100785da59416aaaeb52475

data/README.md

@@ -6,7 +6,7 @@ It accepts any commits payloads and transforms them into identical data structur
 If you need to integrate web-hooks from Github, Bitbucket or Gitlab this is definitely
 worth checking out. Check examples for details.
 
-[![build status](https://magnum-ci.com/status/1f87bb33961c21de5940142e86a741f1.png)](https://magnum-ci.com/public/0cb3a398347ebeeb90fb/builds)
+[![Build Status](https://magnum-ci.com/status/1f87bb33961c21de5940142e86a741f1.png)](https://magnum-ci.com/public/0cb3a398347ebeeb90fb/builds) [![Code Climate](https://codeclimate.com/github/magnumci/magnum-payload.png)](https://codeclimate.com/github/magnumci/magnum-payload)
 
 ## Supported platforms
 
@@ -106,4 +106,4 @@ The MIT License (MIT)
 
 Copyright (c) 2013-2014 Dan Sosedoff, <dan.sosedoff@gmail.com>
 
-Copyright (c) 2013-2014 Magnum CI, <support@magnum-ci.com>
+Copyright (c) 2013-2014 Magnum CI, <support@magnum-ci.com>

data/lib/magnum/payload/github.rb

@@ -19,8 +19,9 @@ module Magnum
     end
 
     def skip_payload?
-      @skip = true if head_deleted? || last_commit.nil?
-      @skip = true if data.ref =~ /tags/
+      return @skip = true if data.zen
+      @skip = true        if head_deleted? || last_commit.nil?
+      @skip = true        if data.ref =~ /tags/
 
       @skip
     end

data/lib/magnum/payload/version.rb

@@ -1,5 +1,5 @@
 module Magnum
   module Payload
-    VERSION = "0.3.4"
+    VERSION = "0.3.5"
   end
 end

data/spec/fixtures/github/test.json

@@ -0,0 +1,4 @@
+{
+  "zen": "Half measures are as bad as nothing at all.",
+  "hook_id": 12345
+}

data/spec/payload/github_spec.rb

@@ -1,73 +1,81 @@
-require 'spec_helper'
+require "spec_helper"
 
 describe Magnum::Payload::Github do
-  describe '#parse!' do
-    let(:data)    { fixture('github.json') }
+  describe "#parse!" do
+    let(:data)    { fixture("github.json") }
     let(:payload) { Magnum::Payload::Github.new(data) }
 
-    it 'sets commit sha' do
-      payload.commit.should eq '9d227f327e725164c3266be74cf5c00678edad13'
+    it "sets commit sha" do
+      expect(payload.commit).to eq "9d227f327e725164c3266be74cf5c00678edad13"
     end
 
-    it 'sets commit branch' do
-      payload.branch.should eq 'master'
+    it "sets commit branch" do
+      expect(payload.branch).to eq "master"
     end
 
-    it 'sets commit message' do
-      payload.message.should eq 'Remove jruby from test matrix'
+    it "sets commit message" do
+      expect(payload.message).to eq "Remove jruby from test matrix"
     end
 
-    it 'sets author' do
-      payload.committer.should eq 'Dan Sosedoff'
-      payload.committer_email.should eq 'dan.sosedoff@gmail.com'
+    it "sets author" do
+      expect(payload.committer).to eq "Dan Sosedoff"
+      expect(payload.committer_email).to eq "dan.sosedoff@gmail.com"
     end
 
-    it 'sets committer' do
-      payload.author.should eq 'Dan Sosedoff'
-      payload.author_email.should eq 'dan.sosedoff@gmail.com'
+    it "sets committer" do
+      expect(payload.author).to eq "Dan Sosedoff"
+      expect(payload.author_email).to eq "dan.sosedoff@gmail.com"
     end
 
-    it 'sets commit view url' do
-      payload.commit_url.should eq 'https://github.com/sosedoff/lxc-ruby/commit/9d227f327e725164c3266be74cf5c00678edad13'
+    it "sets commit view url" do
+      expect(payload.commit_url).to eq "https://github.com/sosedoff/lxc-ruby/commit/9d227f327e725164c3266be74cf5c00678edad13"
     end
 
-    it 'sets commit compare url' do
-      payload.compare_url.should eq 'https://github.com/sosedoff/lxc-ruby/compare/0e46f019e391...9d227f327e72'
+    it "sets commit compare url" do
+      expect(payload.compare_url).to eq "https://github.com/sosedoff/lxc-ruby/compare/0e46f019e391...9d227f327e72"
     end
 
-    context 'when push is forced' do
-      let(:data) { fixture 'github/forced.json' }
+    context "when push is forced" do
+      let(:data) { fixture "github/forced.json" }
 
-      it 'parses head commit' do
-        payload.commit.should eq 'e7508c4c70d2c5afc1e6c2f3a42ecc098f435103'
+      it "parses head commit" do
+        expect(payload.commit).to eq "e7508c4c70d2c5afc1e6c2f3a42ecc098f435103"
       end
     end
 
-    context 'for deleted branch' do
-      let(:data) { fixture 'github/deleted.json' }
+    context "for deleted branch" do
+      let(:data) { fixture "github/deleted.json" }
 
-      it 'marks payload as skipped' do
-        payload.skip.should be_true
+      it "marks payload as skipped" do
+        expect(payload.skip).to be_true
       end
     end
 
-    context 'for new branch' do
-      let(:data) { fixture 'github/new_branch.json' }
+    context "for new branch" do
+      let(:data) { fixture "github/new_branch.json" }
 
-      it 'sets commit sha' do
-        payload.commit.should eq 'd9cc46f9e7e6aa65df696d8d1efe86de755b46ae'
+      it "sets commit sha" do
+        expect(payload.commit).to eq "d9cc46f9e7e6aa65df696d8d1efe86de755b46ae"
       end
 
-      it 'sets branch' do
-        payload.branch.should eq 'hello'
+      it "sets branch" do
+        expect(payload.branch).to eq "hello"
       end
     end
 
-    context 'for new tag' do
-      let(:data) { fixture 'github/new_tag.json' }
+    context "for new tag" do
+      let(:data) { fixture "github/new_tag.json" }
 
-      it 'marks payload as skipped' do
-        payload.skip.should be_true
+      it "marks payload as skipped" do
+        expect(payload.skip).to be_true
+      end
+    end
+
+    context "for ping payload" do
+      let(:data) { fixture "github/test.json" }
+
+      it "marks payload as skipped" do
+        expect(payload.skip).to be_true
       end
     end
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: magnum-payload
 version: !ruby/object:Gem::Version
-  version: 0.3.4
+  version: 0.3.5
 platform: ruby
 authors:
 - Dan Sosedoff
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-06 00:00:00.000000000 Z
+date: 2014-02-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -117,6 +117,7 @@ files:
 - spec/fixtures/github/forced.json
 - spec/fixtures/github/new_branch.json
 - spec/fixtures/github/new_tag.json
+- spec/fixtures/github/test.json
 - spec/fixtures/gitlab/branch_with_slash.json
 - spec/fixtures/gitlab/commits.json
 - spec/fixtures/gitlab/create_branch.json
@@ -168,6 +169,7 @@ test_files:
 - spec/fixtures/github/forced.json
 - spec/fixtures/github/new_branch.json
 - spec/fixtures/github/new_tag.json
+- spec/fixtures/github/test.json
 - spec/fixtures/gitlab/branch_with_slash.json
 - spec/fixtures/gitlab/commits.json
 - spec/fixtures/gitlab/create_branch.json