Prevents new issue assignments for inactive Repos

Author: hbontempo-br

app/models/repo.rb

@@ -226,6 +226,10 @@ def repo_path
     File.join 'repos', path
   end
 
+  def active?
+    removed_from_github == false && archived == false
+  end
+
   def self.find_by_full_name(full_name)
     Repo.find_by!(full_name: full_name)
   end

app/models/user.rb

@@ -201,6 +201,7 @@ def issue_assignments_to_deliver(assign: true)
   private
 
   def issue_assigner
-    @issue_assigner ||= IssueAssigner.new(self, repo_subscriptions)
+    subs = repo_subscriptions.select { |s| s.repo.active? }
+    @issue_assigner ||= IssueAssigner.new(self, subs)
   end
 end

test/fixtures/repo_subscriptions.yml

@@ -26,6 +26,14 @@ jroes_to_rails:
   last_sent_at:
   email_limit: 5
 
+empty_to_archived:
+  repo: archived_repo
+  user: empty
+  created_at: 2013-10-29 21:09:48.351554000 Z
+  updated_at: 2013-10-29 21:09:48.351554000 Z
+  last_sent_at:
+  email_limit: 5
+
 read_doc_only:
   repo: issue_triage_sandbox
   user: foo_user

test/unit/user_test.rb

@@ -160,4 +160,8 @@ class UserTest < ActiveSupport::TestCase
   test "#repos_fetcher" do
     assert users(:mockstar).repos_fetcher(GithubFetcher::Repos::OWNED).is_a? GithubFetcher::Repos
   end
+
+  test "#issue_assignments_to_deliver should ignore repos not active" do
+    assert_empty users(:empty).issue_assignments_to_deliver
+  end
 end