hubstats 0.3.0 → 0.3.1

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YmYyMTJlM2ZlOWJhNDFjYjcwNTI3Yjg4OWJkYmVmZTQ3MTg2ZDZiNw==
+    NzRkMWM1NzQyOWY2YWMxZDdmZGZlMGIyZTgzMGU2YjQ0MjQ2OTFhZg==
   data.tar.gz: !binary |-
-    ZmQzYjZlNjMwYWJiZjBmYWY1NTBmMWY4YTFiN2FiOGY2NWNlNmQwMA==
+    ZjVhMDgwNTRmYmNlMGM1M2I0M2IwNDQ3NTQ0ODg5ZjRmNDk5MzhlYw==
 SHA512:
   metadata.gz: !binary |-
-    YWQyN2QwMDAyNzBkYzg4MWZmZjJlMjM1NDkzMTU3ZjM5YzcyNDc2NGFjNzBh
-    MGI3OWFiZGUyMTA5ODg0NWE2ZWJhNTE1OTk5Mzk3MGIzZWQ2YmY3NTBjYjZi
-    MWMxMTg0ODA0MGNlMjU5MmQ5NzBhYTRhNGZmMzRmNWQwZTA0MTM=
+    MzZjMzZhYmM2ODg3ODQ0OTYxYzMyMWRmOTlmNTlmMTAyZGU0MjRmMzExMWY2
+    MjMxNzNkMTFhODE0YmYwYTIyNzkyYjcyMTAyYTlhYmE0ZTA1MzMyZjU3ZTU4
+    NTM2NzBjZTgwNDhlYTcyNTY2NzFkZWUzOThmZmVkNThlYzU1ZmM=
   data.tar.gz: !binary |-
-    NDJhNGQ5OTk3MTY3YzZkNTY4ZWE0ZGRhZjhiMzAzOWRjM2ExYTMwNzYyNzcx
-    Y2ZjYWQ5YWM4ODUyNTRhYWQ4NWE2ZWVmY2I2ZjM4NDE1MDMxOGNkZjgwMjhk
-    MmQ1NzhiYmIzMTlmYjEzNGJmM2RhODRmOTgzNDg2YmZlOGUzOGE=
+    OGNhMDc2ZGNhYzg4NWQ5NzZhODc2MWU4MzIwMmUyOTQ2NTUwZDI5Y2JhYmVh
+    ZjY0MGJjMGEwYzY0NjZkMzk3YjRiMzdjZDUwNDFjMzVjZjBjMGQwNjZkM2U1
+    YTQ5ZmUzY2M5MzYyMWVhMDNhMjY3MTZjOTY2YmE5NjdiNTQ0NDI=

data/CHANGELOG.markdown

@@ -1,3 +1,6 @@
+#### v0.4.2
+#### v0.4.1
+#### v0.4.0
 #### v0.3.0
 #### v0.2.3
 #### v0.2.2

data/README.md

@@ -46,7 +46,7 @@ Set the endpoint to be:
 
 Hubstats needs to know what repos for it to watch. You can set it to watch either an entire organization or a list of specific repos in octokit.yml.
 
-### Testing
+## Testing
 
 All of the automated tests are written in RSpec. Since hubstats is a plugin, not an application, we need to install the plugin into a Rails application to run. The `test` directory is a dummy rails application for manually testing the UI by serving hubstats locally. When developing and using the `test/dummy` locally, then the test will automatically sync with any updated code, so it doesn't need to be re-served when changes are made with the normal Rails application. When in the development process, one just needs to run `rails s` once from the command line (while in `test/dummy`) to serve the plugin. http://guides.rubyonrails.org/plugins.html will give more information on the implementation of a Rails plugin. 
 

data/app/controllers/hubstats/deploys_controller.rb

@@ -12,7 +12,7 @@ module Hubstats
       #  .map(&:id)
 
       # sets to include user and repo, and sorts data
-      @deploys = Hubstats::Deploy.includes(:user).includes(:repo)
+      @deploys = Hubstats::Deploy.includes(:repo)
         .order_with_timespan(@timespan, params[:order])
       #  .belonging_to_users(params[:users]).belonging_to_repos(params[:repos])
       #  .paginate(:page => params[:page], :per_page => 15).order("deployed_at DESC")
@@ -30,11 +30,26 @@ module Hubstats
     end
 
     def create
-      @deploy = Deploy.new(params[:deploy])
-      if @deploy.save
-        redirect_to :action => 'index'
+      if params[:deployed_by].nil? || params[:git_revision].nil? || params[:repo_name].nil?
+        render text: "Missing a necessary parameter: deployer, git revision, or repository name.", :status => 400 and return
       else
-        render :nothing => true, :status => 400
+        @deploy = Deploy.new
+        @deploy.deployed_at = params[:deployed_at]
+        @deploy.deployed_by = params[:deployed_by]
+        @deploy.git_revision = params[:git_revision]
+        @repo = Hubstats::Repo.where(full_name: params[:repo_name])
+        
+        if @repo.empty?
+          render text: "Repository name is invalid.", :status => 400 and return
+        else
+          @deploy.repo_id = @repo.first.id.to_i
+        end
+        
+        if @deploy.save
+          render :nothing =>true, :status => 200 and return
+        else
+          render :nothing => true, :status => 400 and return
+        end
       end
     end
 

data/app/models/hubstats/deploy.rb

@@ -1,6 +1,14 @@
 module Hubstats
   class Deploy < ActiveRecord::Base
 
+    before_validation :check_time, on: :create
+    validates :git_revision, :deployed_at, :deployed_by, :repo_id, presence: true
+    #validates_associated :repo
+
+    def check_time
+        self.deployed_at = Time.now.getutc if deployed_at.nil?
+    end
+
     scope :deployed_since, lambda {|time| where("hubstats_deploys.deployed_at > ?", time) }
     scope :belonging_to_repo, lambda {|repo_id| where(repo_id: repo_id)}
     scope :belonging_to_user, lambda {|user_id| where(user_id: user_id)}

data/app/models/hubstats/label.rb

@@ -21,7 +21,7 @@ module Hubstats
       if exists = Hubstats::Label.where(name: label[:name]).first
         return exists
       else
-        Hubstats::Label.new(name: label[:name], url: label[:url], color: label[:color])
+        Label.new(name: label[:name], url: label[:url], color: label[:color])
       end
     end
 

data/config/routes.rb

@@ -1,7 +1,7 @@
 Hubstats::Engine.routes.draw do
   root to: "pull_requests#index" #sets default root to be the pulls page
   post "/handler" => "events#handler", :as => :handler
-  resources :deploys, :only => [:index, :create] #routes to both index and to create method
+  resources :deploys, :only => [:create, :index] #routes to both index and to create method
   get "/metrics" => "repos#dashboard", :as => :metrics #routes to list of repos and stats
   get "/pulls" => "pull_requests#index", :as => :pulls #routes to list of pulls
   get "/users" => "users#index", :as => :users #routes to list of users

data/lib/hubstats/version.rb

@@ -1,3 +1,3 @@
 module Hubstats
-  VERSION = "0.3.0"
+  VERSION = "0.3.1"
 end

data/spec/controllers/hubstats/deploys_controller_spec.rb

@@ -0,0 +1,95 @@
+require 'spec_helper'
+
+module Hubstats
+  describe DeploysController, :type => :controller do
+    routes { Hubstats::Engine.routes }
+    describe "#create" do
+
+      before :each do
+        create(:repo, :full_name => "sportngin/ngin")
+      end
+
+      it 'should create a deploy' do
+        post(:create, {"git_revision" => "c1a2b37",
+                       "repo_name" => "sportngin/ngin",
+                       "deployed_at" => "2009-02-03 03:00:00 -0500",
+                       "deployed_by" => "emmasax1"})
+        expect(assigns(:deploy).git_revision).to eq("c1a2b37")
+        expect(assigns(:deploy).deployed_at).to eq("2009-02-03 03:00:00 -0500")
+        expect(assigns(:deploy).deployed_by).to eq("emmasax1")
+        expect(assigns(:deploy).repo_id).to eq(101010)
+        expect(response).to have_http_status(200)
+      end
+
+      it 'should create a deploy without a deployed_at because nil time turns into current time' do
+        post(:create, {"git_revision" => "c1a2b37",
+                       "repo_name" => "sportngin/ngin",
+                       "deployed_at" => nil,
+                       "deployed_by" => "emmasax1"})
+        expect(assigns(:deploy).git_revision).to eq("c1a2b37")
+        expect(assigns(:deploy).deployed_by).to eq("emmasax1")
+        expect(assigns(:deploy).repo_id).to eq(101010)
+        expect(response).to have_http_status(200)
+      end
+
+      it 'should NOT create a deploy without a git_revision' do
+        post(:create, {"git_revision" => nil,
+                       "repo_name" => "sportngin/ngin",
+                       "deployed_at" => "2009-02-03 03:00:00 -0500",
+                       "deployed_by" => "emmasax1"})
+        expect(response).to have_http_status(400)
+      end
+
+      it 'should NOT create a deploy without a deployed_by' do
+        post(:create, {"git_revision" => "c1a2b37",
+                       "repo_name" => "sportngin/ngin",
+                       "deployed_at" => "2009-02-03 03:00:00 -0500",
+                       "deployed_by" => nil})
+        expect(response).to have_http_status(400)
+      end
+
+      it 'should NOT create a deploy without a repo_name' do
+        post(:create, {"git_revision" => "c1a2b37",
+                       "repo_name" => nil,
+                       "deployed_at" => "2009-02-03 03:00:00 -0500",
+                       "deployed_by" => "emmasax1"})
+        expect(response).to have_http_status(400)
+      end
+
+      it 'should NOT create a deploy when given a non existing repo_name' do
+        post(:create, {"git_revision" => "c1a2b37",
+                       "repo_name" => "sportngin/make_resourceful",
+                       "deployed_at" => "2009-02-03 03:00:00 -0500",
+                       "deployed_by" => "emmasax1"})
+        expect(response).to have_http_status(400)
+      end
+    end
+
+    describe "#index" do
+
+      before :each do
+        create(:repo, :full_name => "sportngin/ngin")
+      end
+
+      it "should correctly order all of the deploys" do
+        deploy1 = create(:deploy, :git_revision => "c1a2b37",
+                                  :repo_id => 101010,
+                                  :deployed_at => "2009-02-03 03:00:00 -0500",
+                                  :deployed_by => "emmasax1")
+        deploy2 = create(:deploy, :git_revision => "kd9c102",
+                                  :repo_id => 101010,
+                                  :deployed_at => "2015-02-03 03:00:00 -0500",
+                                  :deployed_by => "emmasax1")
+        deploy3 = create(:deploy, :git_revision => "owk19sf",
+                                  :repo_id => 101010,
+                                  :deployed_at => "2011-02-03 03:00:00 -0500",
+                                  :deployed_by => "emmasax1")
+        deploys_ordered = [deploy2, deploy3, deploy1]
+        expect(Hubstats::Deploy).to receive(:order_with_timespan).and_return(deploys_ordered)
+        get :index
+        expect(response).to have_http_status(200)
+        expect(response).to render_template(:index)
+      end
+    end
+  end
+end

data/spec/controllers/hubstats/events_controller_spec.rb

@@ -2,6 +2,6 @@ require 'spec_helper'
 
 module Hubstats
   RSpec.describe EventsController, :type => :controller do
-  
+  	
   end
 end

data/spec/factories/deploys.rb

@@ -0,0 +1,8 @@
+FactoryGirl.define do
+  factory :deploy, :class => Hubstats::Deploy do
+    git_revision "3dis01"
+    repo_id 101010
+    deployed_at "2015-02-03 03:00:00 -0500"
+    deployed_by "emmasax1"
+  end
+end

data/spec/factories/repo.rb

@@ -12,5 +12,4 @@ FactoryGirl.define do
 
     initialize_with { attributes } 
   end
-
-end
+end

data/spec/lib/hubstats/events_handler_spec.rb

@@ -8,30 +8,30 @@ module Hubstats
       let(:payload) {build(:pull_request_payload_hash)}
 
       subject {Hubstats::EventsHandler.new()}
-      it 'successfully routes the event' do
+      it 'should successfully route the event' do
         expect(subject).to receive(:pull_processor)
         subject.route(payload,payload[:type])
       end
 
-      it 'adds labels to pull request' do
-        allow(Hubstats::PullRequest).to receive(:create_or_update) {pull}
-        allow(Hubstats::Repo).to receive(:where) {[repo,repo]}
-        allow(Hubstats::GithubAPI).to receive(:get_labels_for_pull) {['low','high']}
+      it 'should add labels to pull request' do
+        allow(PullRequest).to receive(:create_or_update) {pull}
+        allow(Repo).to receive(:where) {[repo,repo]}
+        allow(GithubAPI).to receive(:get_labels_for_pull) {['low','high']}
         expect(pull).to receive(:add_labels).with(['low','high'])
         subject.route(payload,payload[:type])
       end
     end
 
     context "CommentEvent" do
-      it 'successfully routes the event' do
-        ehandler = Hubstats::EventsHandler.new()
+      it 'should successfully route the event' do
+        ehandler = EventsHandler.new()
         payload = build(:comment_payload_hash)
         expect(ehandler).to receive(:comment_processor)
 
         ehandler.route(payload,payload[:type])
       end
 
-      it 'successfully processes the event' do
+      it 'should successfully process the event' do
         ehandler = Hubstats::EventsHandler.new()
         payload = build(:comment_payload_hash)
         expect(Hubstats::Comment).to receive(:create_or_update)
@@ -39,7 +39,7 @@ module Hubstats
         ehandler.route(payload,payload[:type])
       end
 
-      it 'successfully creates_or_updates the event' do
+      it 'should successfully creates_or_updates the event' do
         ehandler = Hubstats::EventsHandler.new()
         payload = build(:comment_payload_hash)
         expect(ehandler.route(payload,payload[:type]).class).to eq(Hubstats::Comment)
@@ -47,4 +47,4 @@ module Hubstats
     end
 
   end
-end
+end

data/spec/lib/hubstats/github_api_spec.rb

@@ -20,7 +20,7 @@ module Hubstats
           allow(ENV).to receive(:[]).and_return(nil)
         end
 
-        it 'initializes client with options param' do
+        it 'should initialize client with options param' do
           Hubstats::GithubAPI.configure({"access_token" => access_token})
           expect(Octokit::Client).to receive(:new).with(access_token: access_token).and_return(client)
           expect(Hubstats::GithubAPI.client).to eq(client)
@@ -32,7 +32,7 @@ module Hubstats
           allow(ENV).to receive(:[]).with("GITHUB_API_TOKEN").and_return("github_api_token")
         end
 
-        it 'initializes client with environment variables' do
+        it 'should initialize client with environment variables' do
           Hubstats::GithubAPI.configure({"access_token" => access_token})
           expect(Octokit::Client).to receive(:new).with(access_token: "github_api_token").and_return(client)
           expect(Hubstats::GithubAPI.client()).to eq(client)
@@ -46,7 +46,7 @@ module Hubstats
           allow(ENV).to receive(:[]).with("CLIENT_SECRET").and_return("client_secret")
         end 
 
-        it 'intializes client with client-id environment variables' do
+        it 'should intialize client with client-id environment variables' do
           Hubstats::GithubAPI.configure()
           expect(Octokit::Client).to receive(:new).with(client_id: "client_id", client_secret: "client_secret").and_return(client)
           expect(Hubstats::GithubAPI.client).to eq(client)
@@ -59,9 +59,9 @@ module Hubstats
           allow(ENV).to receive(:[]).and_return(nil)
         end
 
-        it 'fails to initialize at all' do
+        it 'should fail to initialize at all' do
           Hubstats::GithubAPI.configure()
-          expect(lambda { Hubstats::GithubAPI.client}).to raise_error Octokit::Unauthorized
+          expect{Hubstats::GithubAPI.client}.to raise_error Octokit::Unauthorized
         end
       end
     end
@@ -71,7 +71,7 @@ module Hubstats
       let(:repo) {'hubstats'}
       context "with old_endpoint" do
         let(:old_endpoint) {'www.hubstats.com'}
-        it 'calls delete_hook' do
+        it 'should call delete_hook' do
           allow(subject).to receive(:create_hook)
           expect(subject).to receive(:delete_hook).with(repo,old_endpoint)
           subject.update_hook('hubstats','www.hubstats.com')
@@ -79,7 +79,7 @@ module Hubstats
       end
 
       context "without old_point" do
-        it 'does not call delete_hook' do
+        it 'should not call delete_hook' do
           allow(subject).to receive(:create_hook)
           expect(subject).to_not receive(:delete_hook).with(repo)
           subject.update_hook('hubstats')
@@ -97,12 +97,12 @@ module Hubstats
         allow(subject).to receive(:client) {client}
       end
 
-      it "calls octokit create_hook" do
+      it "should call octokit create_hook" do
         expect(client).to receive(:create_hook)
         subject.create_hook(repo)
       end
 
-      it "rescues unprocessable entity" do
+      it "should rescue unprocessable entity" do
         allow(client).to receive(:create_hook) { raise Octokit::UnprocessableEntity }
         subject.create_hook(repo)
       end

data/spec/models/hubstats/comment_spec.rb

@@ -2,35 +2,22 @@ require 'spec_helper'
 
 module Hubstats
   describe Comment, :type => :model do
-    it 'creates and returns a comment and user' do
-      github_user = {
-        :login => "elliothursh",
-        :id => 10,
-        :type => "User"
-      }
-
-      github_comment = {
-        :user => github_user,
-        :pull_request_url => "www.pull.com",
-        :id => 100
-      }
-
-      github_repo = {
-        :id => 151,
-        :name => "HellWorld"
-      }
-
-      pull_request = {
-        :id => 1000,
-        :url => "www.pull.com",
-        :repository => github_repo,
-        :user => github_user
-      }
-
-      repo = Hubstats::Repo.create_or_update(github_repo)
-      pull = Hubstats::PullRequest.create_or_update(pull_request)
-      comment = Hubstats::Comment.create_or_update(github_comment)
-
+    it 'should create and return a comment and user' do
+      github_user = {:login => "elliothursh",
+                     :id => 10,
+                    :type => "User"}
+      github_comment = {:user => github_user,
+                        :pull_request_url => "www.pull.com",
+                        :id => 100}
+      github_repo = {:id => 151,
+                     :name => "HelloWorld"}
+      pull_request = {:id => 1000,
+                      :url => "www.pull.com",
+                      :repository => github_repo,
+                      :user => github_user}
+      repo = Repo.create_or_update(github_repo)
+      pull = PullRequest.create_or_update(pull_request)
+      comment = Comment.create_or_update(github_comment)
       expect(comment.id).to eq(github_comment[:id])
       expect(comment.user_id).to eq(github_user[:id])
     end

data/spec/models/hubstats/deploy_spec.rb

@@ -0,0 +1,110 @@
+require 'spec_helper'
+
+module Hubstats
+  describe Deploy, :type => :model do
+    it 'should create a deploy and check validations' do
+      repo = build(:repo)
+      deploy_hash = {git_revision: "c1a2b37", 
+                     repo_id: repo.id,
+                     deployed_at: "2009-02-03 03:00:00 -0500", 
+                     deployed_by: "emmasax1"}
+      deploy = Deploy.create(deploy_hash)
+      expect(Deploy.count).to eq(1)
+      expect(deploy.git_revision).to eq(deploy_hash[:git_revision])
+      expect(deploy.repo_id).to eq(deploy_hash[:repo_id])
+      expect(deploy.deployed_at).to eq(deploy_hash[:deployed_at])
+      expect(deploy.deployed_by).to eq(deploy_hash[:deployed_by])
+    end
+
+    it 'should order deploys based on timespan ASC' do
+      repo = build(:repo)
+      deploy1 = Deploy.create(git_revision: "c1a2b37",
+                              repo_id: repo.id, 
+                              deployed_at: "2009-12-03 23:00:00", 
+                              deployed_by: "emmasax1")
+      deploy2 = Deploy.create(git_revision: "kd92h10",
+                              repo_id: repo.id, 
+                              deployed_at: "2010-05-26 22:00:00", 
+                              deployed_by: "odelltuttle")
+      deploy3 = Deploy.create(git_revision: "k10d8as",
+                              repo_id: repo.id, 
+                              deployed_at: "2015-08-21 12:00:00", 
+                              deployed_by: "EvaMartinuzzi")
+      deploy4 = Deploy.create(git_revision: "917d9ss",
+                              repo_id: repo.id, 
+                              deployed_at: "2014-12-19 08:00:00", 
+                              deployed_by: "panderson74")
+      deploys_ordered = [deploy1, deploy2, deploy4, deploy3]
+      new_ordered_deploys = Deploy.order_with_timespan(520.weeks, "ASC")
+      expect(Deploy.count).to eq(4)
+      expect(new_ordered_deploys).to eq(deploys_ordered)
+    end
+
+    it 'should order deploys based on timespan DESC' do
+      repo = build(:repo)
+      deploy1 = Deploy.create(git_revision: "c1a2b37",
+                              repo_id: repo.id, 
+                              deployed_at: "2009-12-03 23:00:00", 
+                              deployed_by: "emmasax1")
+      deploy2 = Deploy.create(git_revision: "kd92h10",
+                              repo_id: repo.id, 
+                              deployed_at: "2010-05-26 22:00:00", 
+                              deployed_by: "odelltuttle")
+      deploy3 = Deploy.create(git_revision: "k10d8as",
+                              repo_id: repo.id, 
+                              deployed_at: "2015-08-21 12:00:00", 
+                              deployed_by: "EvaMartinuzzi")
+      deploy4 = Deploy.create(git_revision: "917d9ss",
+                              repo_id: repo.id, 
+                              deployed_at: "2014-12-19 08:00:00", 
+                              deployed_by: "panderson74")
+      deploys_ordered = [deploy3, deploy4, deploy2, deploy1]
+      new_ordered_deploys = Deploy.order_with_timespan(520.weeks, "DESC")
+      expect(Deploy.count).to eq(4)
+      expect(new_ordered_deploys).to eq(deploys_ordered)
+    end
+
+    it 'should NOT create a deploy without a git_revision' do
+      repo = build(:repo)
+      expect(Deploy.create(git_revision: nil, 
+                           repo_id: repo.id, 
+                           deployed_at: "2009-02-03 03:00:00 -0500", 
+                           deployed_by: "emmasax1")).not_to be_valid
+    end
+
+    it 'should NOT create a deploy without a repo_id' do
+      repo = build(:repo)
+      expect(Deploy.create(git_revision: "c1a2b37", 
+                           repo_id: nil, 
+                           deployed_at: "2009-02-03 03:00:00 -0500", 
+                           deployed_by: "emmasax1")).not_to be_valid
+    end
+
+
+    it 'should NOT create a deploy without a deployed_by' do
+      repo = build(:repo)
+      expect(Deploy.create(git_revision: "c1a2b37", 
+                           repo_id: repo.id, 
+                           deployed_at: "2009-02-03 03:00:00 -0500", 
+                           deployed_by: nil)).not_to be_valid
+    end
+
+    it 'should create a deploy wihtout a deployed_at because nil time turns into current time' do
+      repo = build(:repo)
+      expect(Deploy.create(git_revision: "c1a2b37", 
+                           repo_id: repo.id, 
+                           deployed_at: nil, 
+                           deployed_by: "emmasax1")).to be_valid
+    end
+
+    it 'should NOT create a deploy without an already valid repo_id' do
+      repo = build(:repo, :id => nil,
+                          :name => nil,
+                          :full_name => nil)
+      expect(Deploy.create(git_revision: "c1a2b37", 
+                           repo_id: repo.id, 
+                           deployed_at: "2009-02-03 03:00:00 -0500", 
+                           deployed_by: "emmasax1")).not_to be_valid
+    end
+  end
+end

data/spec/models/hubstats/label_spec.rb

@@ -1,6 +1,16 @@
 require 'spec_helper'
 
 module Hubstats
-  RSpec.describe Label, :type => :model do
+  describe Label, :type => :model do
+    it 'should create a label' do
+      label_hash = {name: "Feature request",
+                    color: "FCEC7F",
+                    url: "https://api.github.com/repos/sportngin/ngin/labels/feature-request"}
+      label = Label.create(label_hash)
+      expect(Label.count).to eq(1)
+      expect(label.name).to eq(label_hash[:name])
+      expect(label.color).to eq(label_hash[:color])
+      expect(label.url).to eq(label_hash[:url])
+    end
   end
 end

data/spec/models/hubstats/pull_request_spec.rb

@@ -2,12 +2,13 @@ require 'spec_helper'
 
 module Hubstats
   describe PullRequest, :type => :model do
-    it 'creates and returns a PullRequest' do
+    it 'should create and return a Pull Request' do
       pull = build(:pull_request_hash)
-      pull_request = Hubstats::PullRequest.create_or_update(pull)
+      pull_request = PullRequest.create_or_update(pull)
       expect(pull_request.id).to eq(pull[:id])
       expect(pull_request.user_id).to eq(pull[:user][:id])
       expect(pull_request.repo_id).to eq(pull[:repository][:id])
+      expect(pull_request.number).to eq(pull[:number])
     end
   end
 end

data/spec/models/hubstats/repo_spec.rb

@@ -1,7 +1,23 @@
 require 'spec_helper'
 
 module Hubstats
-  # describe Repo, :type => :model do
-  #   pending "add some examples to (or delete) #{__FILE__}"
-  # end
+  describe Repo, :type => :model do
+  	it 'should create a repo' do
+  		repo_hash = {id: 131242,
+  		             owner_id: 3728684,
+  		             name: "example_name",
+  		             full_name: "sportngin/example_name",
+  		             pushed_at: "2015-05-27 13:51:21 -0500",
+  		             created_at: "2009-05-27 06:27:04 -0500",
+  		             updated_at: "2015-03-26 16:17:19 -0500"}
+  		repo = Repo.create(repo_hash)
+  		expect(repo.id).to eq(repo_hash[:id])
+  		expect(repo.owner).to eq(repo_hash[:owner])
+  		expect(repo.name).to eq(repo_hash[:name])
+  		expect(repo.full_name).to eq(repo_hash[:full_name])
+  		expect(repo.pushed_at).to eq(repo_hash[:pushed_at])
+  		expect(repo.created_at).to eq(repo_hash[:created_at])
+  		expect(repo.updated_at).to eq(repo_hash[:updated_at])
+    end
+  end
 end

data/spec/models/hubstats/user_spec.rb

@@ -2,22 +2,21 @@ require 'spec_helper'
 
 module Hubstats
   describe User, :type => :model do
-
     before do
-      Hubstats::User.destroy_all()
+      User.destroy_all()
     end
 
-
-    it 'creates and returns a user' do
+    it 'should create and return a user' do
       user = build(:user_hash, id: 10)
-      expect(Hubstats::User.create_or_update(user).id).to eq(10)
+      expect(User.create_or_update(user).id).to eq(10)
     end
 
-    it 'it updates a user based off id' do
-      user1 = Hubstats::User.create_or_update(build(:user_hash, login: 'johnappleseed', id: 10))
-      user2 = Hubstats::User.create_or_update(build(:user_hash, login: 'elliothursh', id: 10))
+    it 'should update a user based off id' do
+      user1 = User.create_or_update(build(:user_hash, login: 'johnappleseed', id: 10))
+      user2 = User.create_or_update(build(:user_hash, login: 'johndoe', id: 10))
       expect(user1).to eq(user2)
+      expect(user2.login).to eq("johndoe")
+      expect(user2.login).not_to eq("johnapplesdeed")
     end
-
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hubstats
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.3.1
 platform: ruby
 authors:
 - Elliot Hursh
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-05-28 00:00:00.000000000 Z
+date: 2015-06-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -232,11 +232,13 @@ files:
 - lib/tasks/hubstats_tasks.rake
 - lib/tasks/populate_task.rake
 - script/rails
+- spec/controllers/hubstats/deploys_controller_spec.rb
 - spec/controllers/hubstats/events_controller_spec.rb
 - spec/controllers/hubstats/pull_requests_controller_spec.rb
 - spec/controllers/hubstats/repos_controller_spec.rb
 - spec/controllers/hubstats/users_controller_spec.rb
 - spec/factories/comment.rb
+- spec/factories/deploys.rb
 - spec/factories/pull_requests.rb
 - spec/factories/repo.rb
 - spec/factories/users.rb
@@ -247,6 +249,7 @@ files:
 - spec/lib/hubstats/events_handler_spec.rb
 - spec/lib/hubstats/github_api_spec.rb
 - spec/models/hubstats/comment_spec.rb
+- spec/models/hubstats/deploy_spec.rb
 - spec/models/hubstats/label_spec.rb
 - spec/models/hubstats/pull_request_spec.rb
 - spec/models/hubstats/repo_spec.rb
@@ -312,11 +315,13 @@ signing_key:
 specification_version: 4
 summary: Github Statistics
 test_files:
+- spec/controllers/hubstats/deploys_controller_spec.rb
 - spec/controllers/hubstats/events_controller_spec.rb
 - spec/controllers/hubstats/pull_requests_controller_spec.rb
 - spec/controllers/hubstats/repos_controller_spec.rb
 - spec/controllers/hubstats/users_controller_spec.rb
 - spec/factories/comment.rb
+- spec/factories/deploys.rb
 - spec/factories/pull_requests.rb
 - spec/factories/repo.rb
 - spec/factories/users.rb
@@ -327,6 +332,7 @@ test_files:
 - spec/lib/hubstats/events_handler_spec.rb
 - spec/lib/hubstats/github_api_spec.rb
 - spec/models/hubstats/comment_spec.rb
+- spec/models/hubstats/deploy_spec.rb
 - spec/models/hubstats/label_spec.rb
 - spec/models/hubstats/pull_request_spec.rb
 - spec/models/hubstats/repo_spec.rb