pra 0.1.1 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 990d69dd58accffc7a9a697b26589334785b0eed
-  data.tar.gz: e075ae03f303e947c04b7275d22cb3b58a72e03f
+  metadata.gz: a2be92bf55869e2dd6b07043ef8bd1826698895d
+  data.tar.gz: 105bfd005ecf099bb7b25a8b61131a4155b0a724
 SHA512:
-  metadata.gz: e6fc9ec8e865363b814e8e453a8918aeb1a483dae88ac5ad3fc37d64b5c410e7c7eb0cda259ef90f71e293eb518fd579f4a6f2369601f08c57b4f8e1f6ecdf33
-  data.tar.gz: f8b78ff247073c23f30e38df0a3601a6493a3c4490748572dc7d9eec0fefdee879fa9ec4293a0ac34094bddaa4b52435f10222064c4a6f900f673f84d0084c9c
+  metadata.gz: f83912cc359bfd70f36747b7c7493ed06906073fc2715b91dcede0fb9b42eb6be1212b8c519c42f4ba4acaa4445217705cc747011815543d3dc2b8f5514350c4
+  data.tar.gz: 85833ebe12ef250ed4c355d183ae2aced925c4e6e8b7fbaea22f804e698133d398eca30f8a0ba7f8174cd81b3f80520a24b7a7d25e001ff10a32f196447b52fd

data/ChangeLog.md

@@ -6,6 +6,15 @@ versions as well as provide a rough history.
 
 #### Next Release
 
+#### v1.0.0
+
+* Added connection failure handling so it stays running
+  ([\#1](https://github.com/reachlocal/pra/issues/1))
+* Added user notification of connection failures
+  ([\#1](https://github.com/reachlocal/pra/issues/1))
+* Added error logging to `~/.pra.error.log`
+  ([\#1](https://github.com/reachlocal/pra/issues/1))
+
 #### v0.1.1
 
 * Fixed up the gemspec to include homepage on rubygems.org

data/README.md

@@ -57,6 +57,37 @@ file to get you started. Then simply replace the appropriate fields and the
 **repositories** sections for all the pull sources with the repository
 information for the repositories you want to watch for open pull requests.
 
+### GitHub Authentication
+
+#### Multi-Factor Authentication
+
+Sadly, at the moment `pra` doesn't support GitHub's Multi-Factor
+Authentication. There is a ticket for this
+( [\#5](https://github.com/reachlocal/pra/issues/5) ).
+
+#### OAuth
+
+It is also lacking support for GitHub's OAuth mechanism. There is a ticket for
+this ( [\#6](https://github.com/reachlocal/pra/issues/6) ).
+
+#### HTTP Basic Auth
+
+The HTTP Basic Auth will work as long as you don't have multi-factor
+authentication enabled for your account.
+
+#### Personal Access Token
+
+Personal Access Token authentication is currently supported and this is the
+recommended authentication mechanism to use right now. It is the only
+authentication mechanism you can use at the moment if you have multi-factor
+authentication enabled.
+
+Simply go to your GitHub **Account Settings**, select **Applications**, click
+the **Create new token** button in the **Personal Access Token** section. Give
+it the name "Pra" and submit. This will generate your personal access token.
+Then simply put your personal access token in the `~/.pra.json` as your GitHub
+username and "x-oauth-basic" as your GitHub password.
+
 ## Usage
 
 Once you have configured `pra` as described above you can launch it by simply

data/lib/pra/app.rb

@@ -2,6 +2,7 @@ require 'thread'
 
 require 'pra/window_system_factory'
 require 'pra/pull_request_service'
+require 'pra/error_log'
 
 Thread.abort_on_exception=true
 
@@ -22,8 +23,21 @@ module Pra
 
     def fetch_and_refresh_pull_requests
       @window_system.fetching_pull_requests
-      pull_requests = Pra::PullRequestService.fetch_pull_requests
-      @window_system.refresh_pull_requests(pull_requests)
+      new_pull_requests = []
+
+      Pra::PullRequestService.fetch_pull_requests do |fetch|
+        fetch.on_success do |pull_requests|
+          new_pull_requests += pull_requests
+        end
+
+        fetch.on_error do |error|
+          Pra::ErrorLog.log(error)
+          @window_system.fetch_failed
+        end
+      end
+
+      @window_system.refresh_pull_requests(new_pull_requests)
+      
       Kernel.sleep(5 * 60)
     end
 

data/lib/pra/config.rb

@@ -25,6 +25,10 @@ module Pra
       return File.join(self.users_home_directory, '.pra.json')
     end
 
+    def self.error_log_path
+      return File.join(self.users_home_directory, '.pra.errors.log')
+    end
+
     def self.users_home_directory
       return ENV['HOME']
     end

data/lib/pra/curses_window_system.rb

@@ -1,4 +1,5 @@
 require 'pra/window_system'
+require 'pra/config'
 require 'launchy'
 require 'curses'
 require 'thread'
@@ -22,6 +23,12 @@ module Pra
 
     def fetching_pull_requests
       output_string(3, 0, "Fetching pull requests...")
+      Curses.setpos(4,0)
+      Curses.clrtoeol
+    end
+
+    def fetch_failed
+      output_string(4, 0, "Failed to fetch pull requests on 1 or more pull sources. Check #{Pra::Config.error_log_path} for details.")
     end
 
     def refresh_pull_requests(pull_requests)
@@ -99,13 +106,16 @@ module Pra
       }
     end
 
+    HEADER_LINE = 6
+    LIST_START_LINE = HEADER_LINE + 2
+
     def draw_current_pull_requests
       @state_lock.synchronize {
         output_string(3, 0, "#{@current_pull_requests.length} Pull Requests")
-        output_string(5, 0, "repository      title                   from_reference          to_reference            author                  service")
-        output_string(6, 0, "--------------------------------------------------------------------------------------------------------------------------------")
+        output_string(HEADER_LINE, 0, "repository      title                   from_reference          to_reference            author                  service")
+        output_string(HEADER_LINE + 1, 0, "--------------------------------------------------------------------------------------------------------------------------------")
         
-        (7...7+@previous_number_of_pull_requests).each do |i|
+        (LIST_START_LINE...LIST_START_LINE+@previous_number_of_pull_requests).each do |i|
           Curses.setpos(i,0)
           Curses.clrtoeol
           Curses.refresh
@@ -113,9 +123,9 @@ module Pra
 
         @current_pull_requests.each_with_index do |pull_request, index|
           if index == @selected_pull_request_index
-            output_highlighted_string(7 + index, 0, "#{pull_request.repository.ljust(15)[0..14]}\t#{pull_request.title.ljust(20)[0..19]}\t#{pull_request.from_reference.ljust(20)[0..19]}\t#{pull_request.to_reference.ljust(20)[0..19]}\t#{pull_request.author.ljust(20)[0..19]}\t#{pull_request.service_id.ljust(10)[0..9]}")
+            output_highlighted_string(LIST_START_LINE + index, 0, "#{pull_request.repository.ljust(15)[0..14]}\t#{pull_request.title.ljust(20)[0..19]}\t#{pull_request.from_reference.ljust(20)[0..19]}\t#{pull_request.to_reference.ljust(20)[0..19]}\t#{pull_request.author.ljust(20)[0..19]}\t#{pull_request.service_id.ljust(10)[0..9]}")
           else
-            output_string(7 + index, 0, "#{pull_request.repository.ljust(15)[0..14]}\t#{pull_request.title.ljust(20)[0..19]}\t#{pull_request.from_reference.ljust(20)[0..19]}\t#{pull_request.to_reference.ljust(20)[0..19]}\t#{pull_request.author.ljust(20)[0..19]}\t#{pull_request.service_id.ljust(10)[0..9]}")
+            output_string(LIST_START_LINE + index, 0, "#{pull_request.repository.ljust(15)[0..14]}\t#{pull_request.title.ljust(20)[0..19]}\t#{pull_request.from_reference.ljust(20)[0..19]}\t#{pull_request.to_reference.ljust(20)[0..19]}\t#{pull_request.author.ljust(20)[0..19]}\t#{pull_request.service_id.ljust(10)[0..9]}")
           end
         end
       }

data/lib/pra/error_log.rb

@@ -0,0 +1,12 @@
+require 'pra/config'
+
+module Pra
+  class ErrorLog
+    def self.log(error)
+      File.open(Pra::Config.error_log_path, 'a') do |f|
+        f.puts(error.message)
+        error.backtrace.each { |line| f.puts(line) }
+      end
+    end
+  end
+end

data/lib/pra/pull_request_service/fetch_status.rb

@@ -0,0 +1,37 @@
+module Pra
+  module PullRequestService
+    class FetchStatus
+      attr_reader :status, :pull_requests, :error
+
+      def self.success(pull_requests)
+        new(:success, pull_requests)
+      end
+      
+      def self.error(error)
+        new(:error, :no_pull_requests, error)
+      end
+
+      def initialize(status, pull_requests, error = nil)
+        @status = status
+        @pull_requests = pull_requests
+        @error = error
+      end
+
+      def on_success &block
+        yield(@pull_requests) if success?
+      end
+
+      def on_error &block
+        yield(@error) if error?
+      end
+
+      def success?
+        status == :success
+      end
+
+      def error?
+        status == :error
+      end
+    end
+  end
+end

data/lib/pra/pull_request_service.rb

@@ -1,14 +1,20 @@
 require 'pra/config'
 require 'pra/pull_source_factory'
+require 'pra/pull_request_service/fetch_status'
 
 module Pra
   module PullRequestService
     def self.fetch_pull_requests
-      pull_requests = []
       pull_sources.each do |pull_source|
-        pull_requests.concat(pull_source.pull_requests)
+        yield fetch_with_status(pull_source)
       end
-      return pull_requests
+    end
+
+    def self.fetch_with_status(pull_source)
+      pull_requests = pull_source.pull_requests
+      FetchStatus.success(pull_requests)
+    rescue Exception => error
+      FetchStatus.error(error)
     end
 
     def self.pull_sources

data/lib/pra/version.rb

@@ -1,3 +1,3 @@
 module Pra
-  VERSION = "0.1.1"
+  VERSION = "1.0.0"
 end

data/lib/pra/window_system.rb

@@ -18,6 +18,14 @@ module Pra
       raise PureVirtualMethodNotImplemented, "the 'fetching_pull_requests' method needs to be implemented."
     end
 
+    # This method is a pure virtual method and is called by the system
+    # when an error occurs while fetching pull requests. It is intended to
+    # be a notification to the window system so that it can notify the user
+    # that fetching pull requests failed.
+    def fetch_failed
+      raise PureVirtualMethodNotImplemented, "the 'fetch_failed' method needs to be implemented."
+    end
+
     # This method is a pure virtual method and is intendend for the inheriting
     # class to implement it to handle refreshing the pull requests within in
     # the window system. This method is called evertime the pull request

data/spec/lib/pra/app_spec.rb

@@ -34,30 +34,49 @@ describe Pra::App do
   end
 
   describe "#fetch_and_refresh_pull_requests" do
-    it "notifies the window system it is starting to fetch pull requests" do
-      Kernel.stub(:sleep)
-      Pra::PullRequestService.stub(:fetch_pull_requests)
-      window_system_double = double('window system', refresh_pull_requests: nil)
+    let(:pull_request_one) { double }
+    let(:pull_request_two) { double }
+    let(:error) { double('error fetching pull requests', message: double('error message'), backtrace: double('backtrace')) }
+    let(:good_source_one) { double('good pull source one', :pull_requests => [pull_request_one]) }
+    let(:good_source_two) { double('good pull source two', :pull_requests => [pull_request_two])}
+    let(:bad_source) { double('bad pull source') }
+    let(:success_status_one) { Pra::PullRequestService::FetchStatus.success([pull_request_one]) }
+    let(:success_status_two) { Pra::PullRequestService::FetchStatus.success([pull_request_two])}
+    let(:error_status) { Pra::PullRequestService::FetchStatus.error(error) }
+    let(:window_system_double) { double('window system', refresh_pull_requests: nil, fetch_failed: nil, fetching_pull_requests: nil) }
+
+    before do
+      allow(Kernel).to receive(:sleep)
+      allow(Pra::PullRequestService).to receive(:fetch_pull_requests).
+        and_yield(success_status_one).
+        and_yield(error_status).
+        and_yield(success_status_two)
+      allow(Pra::ErrorLog).to receive(:log)
       subject.instance_variable_set(:@window_system, window_system_double)
+    end
+
+    it "notifies the window system it is starting to fetch pull requests" do
       expect(window_system_double).to receive(:fetching_pull_requests)
       subject.fetch_and_refresh_pull_requests
     end
 
     it "fetches the pull requests from all of the sources" do
-      Kernel.stub(:sleep)
-      window_system_double = double('window system', refresh_pull_requests: nil, fetching_pull_requests: nil)
-      subject.instance_variable_set(:@window_system, window_system_double)
       expect(Pra::PullRequestService).to receive(:fetch_pull_requests)
       subject.fetch_and_refresh_pull_requests
     end
 
-    it "tells the window system to refresh pull requests" do
-      Kernel.stub(:sleep)
-      pull_requests = double('fetched pull requests')
-      Pra::PullRequestService.stub(:fetch_pull_requests).and_return(pull_requests)
-      window_system_double = double('window system', fetching_pull_requests: nil)
-      subject.instance_variable_set(:@window_system, window_system_double)
-      expect(window_system_double).to receive(:refresh_pull_requests).with(pull_requests)
+    it "tells the window system to refresh with the fetched pull requests" do
+      expect(window_system_double).to receive(:refresh_pull_requests).with([pull_request_one, pull_request_two])
+      subject.fetch_and_refresh_pull_requests
+    end
+
+    it "tells the window system about failures" do
+      expect(window_system_double).to receive(:fetch_failed)
+      subject.fetch_and_refresh_pull_requests
+    end
+
+    it "logs the errors for pull sources that could not be fetched" do
+      expect(Pra::ErrorLog).to receive(:log).with(error)
       subject.fetch_and_refresh_pull_requests
     end
 

data/spec/lib/pra/config_spec.rb

@@ -93,6 +93,15 @@ describe Pra::Config do
     end
   end
 
+  describe ".error_log_path" do
+    subject { Pra::Config }
+
+    it "returns the joined users home directory and .pra.error.log to create the path" do
+      allow(subject).to receive(:users_home_directory).and_return('/home/someuser')
+      expect(subject.error_log_path).to eq('/home/someuser/.pra.errors.log')
+    end
+  end
+
   describe ".users_home_directory" do
     subject { Pra::Config }
 

data/spec/lib/pra/error_log_spec.rb

@@ -0,0 +1,30 @@
+require 'pra/error_log'
+
+describe Pra::ErrorLog do
+  describe '.log' do
+    let(:log_path) { double('log path') }
+    let(:file) { double('file object', puts: nil) }
+    let(:message) { double('error message') }
+    let(:backtrace) { ['backtrace line 1', 'backtrace line 2'] }
+    let(:error) { double('error', message: message, backtrace: backtrace) }
+
+    it 'opens the log file for appending' do
+      allow(Pra::Config).to receive(:error_log_path).and_return(log_path)
+      expect(File).to receive(:open).with(log_path, 'a')
+      Pra::ErrorLog.log(double)
+    end
+
+    it 'prints the error message to the file' do
+      allow(File).to receive(:open).and_yield(file)
+      expect(file).to receive(:puts).with(message)
+      Pra::ErrorLog.log(error)
+    end
+
+    it 'prints the backtrace to the file' do
+      allow(File).to receive(:open).and_yield(file)
+      expect(file).to receive(:puts).with('backtrace line 1')
+      expect(file).to receive(:puts).with('backtrace line 2')
+      Pra::ErrorLog.log(error)
+    end
+  end
+end

data/spec/lib/pra/pull_request_service/fetch_status_spec.rb

@@ -0,0 +1,127 @@
+require 'pra/pull_request_service/fetch_status'
+
+describe Pra::PullRequestService::FetchStatus do
+  describe '.new' do
+    it 'assigns the status' do
+      status = double
+      fetch = Pra::PullRequestService::FetchStatus.new(status, double)
+      expect(fetch.status).to eq(status)
+    end
+
+    it 'assigns the pull requests' do
+      pulls = [double]
+      fetch = Pra::PullRequestService::FetchStatus.new(double, pulls)
+      expect(fetch.pull_requests).to eq(pulls)
+    end
+
+    it 'assigns the error' do
+      error = double
+      fetch = Pra::PullRequestService::FetchStatus.new(double, double, error)
+      expect(fetch.error).to eq(error)
+    end
+  end
+
+  describe '.success' do
+    let(:pulls) { double }
+    subject { Pra::PullRequestService::FetchStatus.success(pulls) }
+
+    it 'sets the status to success' do
+      expect(subject.status).to eq(:success)
+    end
+
+    it 'assigns the pull requests' do
+      expect(subject.pull_requests).to eq(pulls)
+    end
+  end
+
+  describe '.error' do
+    let(:error) { double }
+    subject { Pra::PullRequestService::FetchStatus.error(error) }
+
+    it 'sets the status to error' do
+      expect(subject.status).to eq(:error)
+    end
+
+    it 'sets the pull requests to a dummy value' do
+      expect(subject.pull_requests).to eq(:no_pull_requests)
+    end
+  end
+
+  describe '#success?' do
+    context 'when status is :success' do
+      subject { Pra::PullRequestService::FetchStatus.new(:success, double) }
+
+      it 'returns true' do
+        expect(subject.success?).to be_true
+      end
+    end
+
+    context 'when status is not :success' do
+      subject { Pra::PullRequestService::FetchStatus.new(double, double) }
+
+      it 'returns false' do
+        expect(subject.success?).to be_false
+      end
+    end
+  end
+
+  describe '#error?' do
+    context 'when status is :error' do
+      subject { Pra::PullRequestService::FetchStatus.new(:error, double) }
+
+      it 'returns true' do
+        expect(subject.error?).to be_true
+      end
+    end
+
+    context 'when status is not :error' do
+      subject { Pra::PullRequestService::FetchStatus.new(double, double) }
+
+      it 'returns false' do
+        expect(subject.error?).to be_false
+      end
+    end
+  end
+
+  describe '#on_success' do
+    let(:pulls) { double }
+    let(:error) { double }
+
+    context 'when status is success' do
+      subject { Pra::PullRequestService::FetchStatus.success(pulls) }
+
+      it 'yields the pull requests to the block' do
+        expect { |success_block| subject.on_success(&success_block) }.to yield_with_args(pulls)
+      end
+    end
+
+    context 'when status is not success' do
+      subject { Pra::PullRequestService::FetchStatus.error(double) }
+
+      it 'does not yield' do
+        expect {|success_block| subject.on_success(&success_block) }.not_to yield_control
+      end
+    end
+  end
+
+  describe '#on_error' do
+    let(:pulls) { double }
+    let(:error) { double }
+
+    context 'when status is error' do
+      subject { Pra::PullRequestService::FetchStatus.error(error) }
+
+      it 'yields the error to the block' do
+        expect { |error_block| subject.on_error(&error_block) }.to yield_with_args(error)
+      end
+    end
+
+    context 'when status is not error' do
+      subject { Pra::PullRequestService::FetchStatus.success(double) }
+
+      it 'does not yield' do
+        expect {|error_block| subject.on_error(&error_block) }.not_to yield_control
+      end
+    end
+  end
+end

data/spec/lib/pra/pull_request_service_spec.rb

@@ -2,27 +2,67 @@ require_relative '../../../lib/pra/pull_request_service'
 
 describe Pra::PullRequestService do
   describe ".fetch_pull_requests" do
+    let(:pull_request_one) { double('pull request one') }
+    let(:pull_request_two) { double('pull request two') }
+    let(:pull_source_one) { double('good pull source one', :pull_requests => [pull_request_one]) }
+    let(:pull_source_two) { double('good pull source two', :pull_requests => [pull_request_two]) }
+
     it "gets all the pull-request sources" do
       subject.should_receive(:pull_sources).and_return([])
       subject.fetch_pull_requests
     end
 
     it "gets the pull requests from each pull-request source" do
-      pull_source_one = double('pull source one')
-      pull_source_two = double('pull source two')
-      subject.stub(:pull_sources).and_return([pull_source_one, pull_source_two])
-      pull_source_one.should_receive(:pull_requests).and_return([])
-      pull_source_two.should_receive(:pull_requests).and_return([])
-      subject.fetch_pull_requests
+      allow(subject).to receive(:pull_sources).and_return([pull_source_one, pull_source_two])
+      expect(subject).to receive(:fetch_with_status).with(pull_source_one)
+      expect(subject).to receive(:fetch_with_status).with(pull_source_two)
+      subject.fetch_pull_requests {}
+    end
+
+    it "yields each pull source with its fetch status object" do
+      status1 = double
+      status2 = double
+      allow(subject).to receive(:pull_sources).and_return([pull_source_one, pull_source_two])
+      allow(subject).to receive(:fetch_with_status).with(pull_source_one).and_return(status1)
+      allow(subject).to receive(:fetch_with_status).with(pull_source_two).and_return(status2)
+      expect { |b| subject.fetch_pull_requests(&b) }.to yield_successive_args(status1, status2)
+    end
+  end
+
+  describe '.fetch_with_status' do
+    let(:pulls) { double('pull requests') }
+    let(:error) { Exception.new('error fetching pull requests') }
+    let(:good_source) { double('good pull source', :pull_requests => pulls) }
+    let(:bad_source) { double('bad pull source') }
+
+    before do
+      allow(bad_source).to receive(:pull_requests).and_raise(error)
     end
 
-    it "returns an array of all the pull-requests from each of the sources" do
-      pull_request_one = double('pull request one')
-      pull_request_two = double('pull request two')
-      pull_source_one = double('pull source one', :pull_requests => [pull_request_one])
-      pull_source_two = double('pull source two', :pull_requests => [pull_request_two])
-      subject.stub(:pull_sources).and_return([pull_source_one, pull_source_two])
-      subject.fetch_pull_requests.should eq([pull_request_one, pull_request_two])
+    context 'when it fetches successfully' do
+      it 'builds a success status object for the requests from each pull source' do
+        expect(Pra::PullRequestService::FetchStatus).to receive(:success).with(pulls)
+        subject.fetch_with_status(good_source)
+      end
+
+      it 'returns the status object' do
+        status = double('success status object')
+        allow(Pra::PullRequestService::FetchStatus).to receive(:success).with(pulls).and_return(status)
+        expect(subject.fetch_with_status(good_source)).to eq(status)
+      end
+    end
+
+    context 'when fetching raises an exception' do
+      it 'builds an error status object with the error' do
+        expect(Pra::PullRequestService::FetchStatus).to receive(:error).with(error)
+        subject.fetch_with_status(bad_source)
+      end
+
+      it 'returns the status object' do
+        status = double('error status object')
+        allow(Pra::PullRequestService::FetchStatus).to receive(:error).and_return(status)
+        expect(subject.fetch_with_status(bad_source)).to eq(status)
+      end
     end
   end
 

data/spec/lib/pra/window_system_spec.rb

@@ -13,6 +13,12 @@ describe Pra::WindowSystem do
     end
   end
 
+  describe "#fetch_failed" do
+    it "raises a message stating that the pure virtual method has not been implemented" do
+      expect { subject.fetch_failed }.to raise_error(Pra::WindowSystem::PureVirtualMethodNotImplemented)
+    end
+  end
+
   describe "#refresh_pull_requests" do
     it "raises a message stating that the pure virtual method has not been implemented" do
       expect { subject.refresh_pull_requests(double('pull requests')) }.to raise_error(Pra::WindowSystem::PureVirtualMethodNotImplemented)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pra
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 1.0.0
 platform: ruby
 authors:
 - Andrew De Ponte
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-15 00:00:00.000000000 Z
+date: 2013-09-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -102,9 +102,11 @@ files:
 - lib/pra/app.rb
 - lib/pra/config.rb
 - lib/pra/curses_window_system.rb
+- lib/pra/error_log.rb
 - lib/pra/github_pull_source.rb
 - lib/pra/pull_request.rb
 - lib/pra/pull_request_service.rb
+- lib/pra/pull_request_service/fetch_status.rb
 - lib/pra/pull_source.rb
 - lib/pra/pull_source_factory.rb
 - lib/pra/stash_pull_source.rb
@@ -115,7 +117,9 @@ files:
 - spec/lib/pra/app_spec.rb
 - spec/lib/pra/config_spec.rb
 - spec/lib/pra/curses_window_system_spec.rb
+- spec/lib/pra/error_log_spec.rb
 - spec/lib/pra/github_pull_source_spec.rb
+- spec/lib/pra/pull_request_service/fetch_status_spec.rb
 - spec/lib/pra/pull_request_service_spec.rb
 - spec/lib/pra/pull_request_spec.rb
 - spec/lib/pra/pull_source_factory_spec.rb
@@ -152,7 +156,9 @@ test_files:
 - spec/lib/pra/app_spec.rb
 - spec/lib/pra/config_spec.rb
 - spec/lib/pra/curses_window_system_spec.rb
+- spec/lib/pra/error_log_spec.rb
 - spec/lib/pra/github_pull_source_spec.rb
+- spec/lib/pra/pull_request_service/fetch_status_spec.rb
 - spec/lib/pra/pull_request_service_spec.rb
 - spec/lib/pra/pull_request_spec.rb
 - spec/lib/pra/pull_source_factory_spec.rb