tutter-sppuppet 0.0.4 → 0.0.6

checksums.yaml

@@ -1,7 +1,15 @@
 ---
-SHA1:
-  metadata.gz: 6d1ab8e26a1410be66395110033e1fbb11747d82
-  data.tar.gz: 30bfe37515d891861ab8b6d4003cc53c3ce47696
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    MGM2NDBhOGZmNWI4NTgzZTk5ZDY2MDY2N2UxYTM3Njc1NDA2NTFkYQ==
+  data.tar.gz: !binary |-
+    NjBjMjBmNWE5ZmJjMDJjZTRlYTE3NjVhMDBlYzc3NTZhOTI2MDhlMw==
 SHA512:
-  metadata.gz: 21ffd61b8c3c054cb9152b3f1f525b7369524db635d07f5eae6c1ace9a47e491dab6de7fd89e29efa2ddc92315e2fb09c47b7f2a578f56de782246321471bfd2
-  data.tar.gz: 48a6176522bbadb736e957fbed09a1f040748da6e4c820d96bbd0d2b033ae070ffe3f6eb6b467a5846c7f80c48c62245cbe63eb7655b696a9b150eeec5ff1727
+  metadata.gz: !binary |-
+    NzFhZmFmZTYxMmYwMmZiNjQ1MDJkOGUzZjc5MTUyNzdiNDBiZmJlNzE4MTE0
+    MTExMzVlMjQyYjg1OThmMzFjOTY5NTJkZWM0MGUzYWVjN2EzOGQ3YzUyMjRm
+    MDgxYjk1ZDNiMGQwNWQ0MjE5YzUyMDYzZjEzYjcxMjlkMDBlNmY=
+  data.tar.gz: !binary |-
+    MzllYzE0MDU5MGFlNjI2NjIxZmJlNTY3ZTE2YTRmNDdlYTAzZDI1MjFkZDQz
+    OTczYTc5Y2ZiNWVkMjgxYzk1ZTE3N2YxODMxZGY5MjExOTQ5NzhmYWI4Yjcy
+    ZmY2YmI4NTI1MmUxN2M1YmJlZTM1YjY3MmEzMTIwNGJhMmYxMGI=

data/.travis.yml

@@ -0,0 +1,9 @@
+deploy:
+  provider: rubygems
+  api_key:
+    secure: lyrstQ+LJHg2zic7oD1xl/7nDAZIXthKh5l9CEhq1gIBCRMxf/lfsQusKNbWuImy+Mk0ovOeLaMPDRDdK9VRpgW3tReuY3hlJIURioHrZbXDks09Ul/B4dk2mpR3ax67/+5kSeEESMOKU31g6YFfVx8TrUioorqLpmKppwqrGFo=
+  gem: tutter-sppuppet
+  on:
+    tags: true
+    repo: jhaals/tutter-sppuppet
+    all_branches: true

data/Gemfile

@@ -0,0 +1,13 @@
+source 'https://rubygems.org'
+
+gem 'tutter'
+
+group :development do
+  gem 'sinatra-reloader'
+end
+
+group :test do
+  gem 'rake'
+  gem 'rspec'
+  gem 'json'
+end

data/README.md

@@ -20,7 +20,8 @@ sppuppet specific settings (goes into tutter.yaml)
     action: 'sppuppet'
     action_settings:
       plus_ones_required: 3
-      debug: false
+      post_instructions: true
+      instructions: 'To merge, post 3 +1s and then !merge'
 
 ### TODO
 * whitelist

data/Rakefile

@@ -0,0 +1,4 @@
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec)
+task :default => [:spec]

data/lib/tutter/action/sppuppet.rb

@@ -14,6 +14,22 @@ class Sppuppet
   end
 
   def run
+    # If a new pull request is opened, comment with instructions
+    if @data['action'] == 'opened' && @settings['post_instructions']
+      issue = @data['issue']['number']
+      comment = @settings['instructions'] || "To merge at least #{@settings['plus_ones_required']} person other than the submitter needs to write a comment with saying _+1_ or _:+1:_. Then write _!merge_ to trigger the merging."
+      begin
+        @client.add_comment(@project, issue, comment)
+        return 200, "Commented!"
+      rescue Octokit::NotFound
+        return 404, "Octokit returned 404, this could be an issue with your access token"
+      rescue Octokit::Unauthorized
+        return 401, "Authorization to #{@project} failed, please verify your access token"
+      rescue Octokit::TooManyLoginAttempts
+        return 429, "Account for #{@project} has been temporary locked down due to to many failed login attempts"
+      end
+    end
+
     pull_request_id = @data['issue']['number']
     pr = @client.pull_request @project, pull_request_id
     plus_one = {}

data/spec/fixtures/new_issue.json

@@ -0,0 +1,154 @@
+{
+  "action": "opened",
+  "issue": {
+    "url": "https://api.github.com/repos/JHaals/testing/issues/9",
+    "labels_url": "https://api.github.com/repos/JHaals/testing/issues/9/labels{/name}",
+    "comments_url": "https://api.github.com/repos/JHaals/testing/issues/9/comments",
+    "events_url": "https://api.github.com/repos/JHaals/testing/issues/9/events",
+    "html_url": "https://github.com/JHaals/testing/issues/9",
+    "id": 28705860,
+    "number": 9,
+    "title": "the internet is broken",
+    "user": {
+      "login": "JHaals",
+      "id": 68980,
+      "avatar_url": "https://avatars.githubusercontent.com/u/68980",
+      "gravatar_id": "4e51de553ebfe258f038e040c8b25892",
+      "url": "https://api.github.com/users/JHaals",
+      "html_url": "https://github.com/JHaals",
+      "followers_url": "https://api.github.com/users/JHaals/followers",
+      "following_url": "https://api.github.com/users/JHaals/following{/other_user}",
+      "gists_url": "https://api.github.com/users/JHaals/gists{/gist_id}",
+      "starred_url": "https://api.github.com/users/JHaals/starred{/owner}{/repo}",
+      "subscriptions_url": "https://api.github.com/users/JHaals/subscriptions",
+      "organizations_url": "https://api.github.com/users/JHaals/orgs",
+      "repos_url": "https://api.github.com/users/JHaals/repos",
+      "events_url": "https://api.github.com/users/JHaals/events{/privacy}",
+      "received_events_url": "https://api.github.com/users/JHaals/received_events",
+      "type": "User",
+      "site_admin": false
+    },
+    "labels": [
+
+    ],
+    "state": "open",
+    "assignee": null,
+    "milestone": null,
+    "comments": 0,
+    "created_at": "2014-03-04T13:46:06Z",
+    "updated_at": "2014-03-04T13:46:06Z",
+    "closed_at": null,
+    "pull_request": {
+      "html_url": null,
+      "diff_url": null,
+      "patch_url": null
+    },
+    "body": "what to do?"
+  },
+  "repository": {
+    "id": 16474769,
+    "name": "testing",
+    "full_name": "JHaals/testing",
+    "owner": {
+      "login": "JHaals",
+      "id": 68980,
+      "avatar_url": "https://avatars.githubusercontent.com/u/68980",
+      "gravatar_id": "4e51de553ebfe258f038e040c8b25892",
+      "url": "https://api.github.com/users/JHaals",
+      "html_url": "https://github.com/JHaals",
+      "followers_url": "https://api.github.com/users/JHaals/followers",
+      "following_url": "https://api.github.com/users/JHaals/following{/other_user}",
+      "gists_url": "https://api.github.com/users/JHaals/gists{/gist_id}",
+      "starred_url": "https://api.github.com/users/JHaals/starred{/owner}{/repo}",
+      "subscriptions_url": "https://api.github.com/users/JHaals/subscriptions",
+      "organizations_url": "https://api.github.com/users/JHaals/orgs",
+      "repos_url": "https://api.github.com/users/JHaals/repos",
+      "events_url": "https://api.github.com/users/JHaals/events{/privacy}",
+      "received_events_url": "https://api.github.com/users/JHaals/received_events",
+      "type": "User",
+      "site_admin": false
+    },
+    "private": false,
+    "html_url": "https://github.com/JHaals/testing",
+    "description": "this is just for personal tests",
+    "fork": false,
+    "url": "https://api.github.com/repos/JHaals/testing",
+    "forks_url": "https://api.github.com/repos/JHaals/testing/forks",
+    "keys_url": "https://api.github.com/repos/JHaals/testing/keys{/key_id}",
+    "collaborators_url": "https://api.github.com/repos/JHaals/testing/collaborators{/collaborator}",
+    "teams_url": "https://api.github.com/repos/JHaals/testing/teams",
+    "hooks_url": "https://api.github.com/repos/JHaals/testing/hooks",
+    "issue_events_url": "https://api.github.com/repos/JHaals/testing/issues/events{/number}",
+    "events_url": "https://api.github.com/repos/JHaals/testing/events",
+    "assignees_url": "https://api.github.com/repos/JHaals/testing/assignees{/user}",
+    "branches_url": "https://api.github.com/repos/JHaals/testing/branches{/branch}",
+    "tags_url": "https://api.github.com/repos/JHaals/testing/tags",
+    "blobs_url": "https://api.github.com/repos/JHaals/testing/git/blobs{/sha}",
+    "git_tags_url": "https://api.github.com/repos/JHaals/testing/git/tags{/sha}",
+    "git_refs_url": "https://api.github.com/repos/JHaals/testing/git/refs{/sha}",
+    "trees_url": "https://api.github.com/repos/JHaals/testing/git/trees{/sha}",
+    "statuses_url": "https://api.github.com/repos/JHaals/testing/statuses/{sha}",
+    "languages_url": "https://api.github.com/repos/JHaals/testing/languages",
+    "stargazers_url": "https://api.github.com/repos/JHaals/testing/stargazers",
+    "contributors_url": "https://api.github.com/repos/JHaals/testing/contributors",
+    "subscribers_url": "https://api.github.com/repos/JHaals/testing/subscribers",
+    "subscription_url": "https://api.github.com/repos/JHaals/testing/subscription",
+    "commits_url": "https://api.github.com/repos/JHaals/testing/commits{/sha}",
+    "git_commits_url": "https://api.github.com/repos/JHaals/testing/git/commits{/sha}",
+    "comments_url": "https://api.github.com/repos/JHaals/testing/comments{/number}",
+    "issue_comment_url": "https://api.github.com/repos/JHaals/testing/issues/comments/{number}",
+    "contents_url": "https://api.github.com/repos/JHaals/testing/contents/{+path}",
+    "compare_url": "https://api.github.com/repos/JHaals/testing/compare/{base}...{head}",
+    "merges_url": "https://api.github.com/repos/JHaals/testing/merges",
+    "archive_url": "https://api.github.com/repos/JHaals/testing/{archive_format}{/ref}",
+    "downloads_url": "https://api.github.com/repos/JHaals/testing/downloads",
+    "issues_url": "https://api.github.com/repos/JHaals/testing/issues{/number}",
+    "pulls_url": "https://api.github.com/repos/JHaals/testing/pulls{/number}",
+    "milestones_url": "https://api.github.com/repos/JHaals/testing/milestones{/number}",
+    "notifications_url": "https://api.github.com/repos/JHaals/testing/notifications{?since,all,participating}",
+    "labels_url": "https://api.github.com/repos/JHaals/testing/labels{/name}",
+    "releases_url": "https://api.github.com/repos/JHaals/testing/releases{/id}",
+    "created_at": "2014-02-03T09:25:47Z",
+    "updated_at": "2014-02-16T20:25:07Z",
+    "pushed_at": "2014-02-16T20:25:06Z",
+    "git_url": "git://github.com/JHaals/testing.git",
+    "ssh_url": "git@github.com:JHaals/testing.git",
+    "clone_url": "https://github.com/JHaals/testing.git",
+    "svn_url": "https://github.com/JHaals/testing",
+    "homepage": null,
+    "size": 312,
+    "stargazers_count": 0,
+    "watchers_count": 0,
+    "language": null,
+    "has_issues": true,
+    "has_downloads": true,
+    "has_wiki": true,
+    "forks_count": 0,
+    "mirror_url": null,
+    "open_issues_count": 5,
+    "forks": 0,
+    "open_issues": 5,
+    "watchers": 0,
+    "default_branch": "master",
+    "master_branch": "master"
+  },
+  "sender": {
+    "login": "JHaals",
+    "id": 68980,
+    "avatar_url": "https://avatars.githubusercontent.com/u/68980",
+    "gravatar_id": "4e51de553ebfe258f038e040c8b25892",
+    "url": "https://api.github.com/users/JHaals",
+    "html_url": "https://github.com/JHaals",
+    "followers_url": "https://api.github.com/users/JHaals/followers",
+    "following_url": "https://api.github.com/users/JHaals/following{/other_user}",
+    "gists_url": "https://api.github.com/users/JHaals/gists{/gist_id}",
+    "starred_url": "https://api.github.com/users/JHaals/starred{/owner}{/repo}",
+    "subscriptions_url": "https://api.github.com/users/JHaals/subscriptions",
+    "organizations_url": "https://api.github.com/users/JHaals/orgs",
+    "repos_url": "https://api.github.com/users/JHaals/repos",
+    "events_url": "https://api.github.com/users/JHaals/events{/privacy}",
+    "received_events_url": "https://api.github.com/users/JHaals/received_events",
+    "type": "User",
+    "site_admin": false
+  }
+}

data/spec/fixtures/tutter.yaml

@@ -0,0 +1,10 @@
+projects:
+  - name: 'JHaals/testing'
+    access_token: ''
+    github_site: 'https://github.com'
+    github_api_endpoint: 'https://api.github.com'
+
+    action: 'sppuppet'
+    action_settings:
+      post_instructions: true
+      plus_ones_required: 3

data/spec/spec_helper.rb

@@ -0,0 +1,16 @@
+ENV['TUTTER_CONFIG_PATH'] = 'spec/fixtures/tutter.yaml'
+require 'tutter'
+require 'sinatra'
+require 'rack/test'
+Tutter.environment = :development
+set :run, false
+set :raise_errors, true
+set :logging, true
+
+def app
+  Tutter
+end
+
+RSpec.configure do |config|
+  config.include Rack::Test::Methods
+end

data/spec/unit/tutter/action/sppuppet_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+require 'json'
+
+describe 'tutter Sppuppet action' do
+  it 'should post a comment with instructions' do
+    data = IO.read('spec/fixtures/new_issue.json')
+    post '/', data, 'HTTP_X_GITHUB_EVENT' => 'opened'
+    expect { last_response.body match(/To merge at least/) }
+    expect { last_response.status == 200 }
+  end
+end

data/{.gemspec → tutter-sppuppet.gemspec}

@@ -1,7 +1,7 @@
 # -*- encoding: utf-8 -*-
 Gem::Specification.new do |s|
   s.name        = 'tutter-sppuppet'
-  s.version     = '0.0.4'
+  s.version     = '0.0.6'
   s.author      = 'Johan Haals'
   s.email       = ['johan.haals@gmail.com']
   s.homepage    = 'https://github.com/jhaals/tutter-sppuppet'

metadata

@@ -1,27 +1,27 @@
 --- !ruby/object:Gem::Specification
 name: tutter-sppuppet
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.6
 platform: ruby
 authors:
 - Johan Haals
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-28 00:00:00.000000000 Z
+date: 2015-01-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: tutter
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
 description: This tutter action let non collaborators review and merge code without
@@ -32,9 +32,16 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gemspec"
+- .travis.yml
+- Gemfile
 - README.md
+- Rakefile
 - lib/tutter/action/sppuppet.rb
+- spec/fixtures/new_issue.json
+- spec/fixtures/tutter.yaml
+- spec/spec_helper.rb
+- spec/unit/tutter/action/sppuppet_spec.rb
+- tutter-sppuppet.gemspec
 homepage: https://github.com/jhaals/tutter-sppuppet
 licenses:
 - Apache 2.0
@@ -45,17 +52,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">="
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: 1.8.7
 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.4.5
 signing_key: 
 specification_version: 4
 summary: Github code review without collaborator access