hipchat_searcher 0.0.3 → 1.0.0

data/spec/data/item-list-empty.json

@@ -0,0 +1,7 @@
+{
+    "items": [],
+    "links": {"self": "https://api.hipchat.com/v2/room/12345/history"},
+    "maxResults": 100,
+    "startIndex": 0
+}
+

data/spec/hipchat_searcher_spec.rb

@@ -1,7 +1,6 @@
-require 'spec_helper'
 
 describe HipchatSearcher do
   it 'has a version number' do
-    HipchatSearcher::VERSION.should == '0.0.3'
+    HipchatSearcher::VERSION.should == '1.0.0'
   end
 end

data/spec/lib/hipchat_searcher/message_spec.rb

@@ -0,0 +1,79 @@
+require 'webmock/rspec'
+
+describe HipchatSearcher::Message do
+  def stub_request_with_headers(uri)
+    stub_request(:get, uri).with(headers: headers)
+  end
+
+  let(:headers) do
+    {
+      'Accept'       => 'application/json',
+      'Content-Type' => 'application/json',
+    }
+  end
+
+  describe '#get_history' do
+    context 'with no options' do
+      before do
+        stub_request_with_headers('https://api.hipchat.com/v2/room/id-12345/history?auth_token=abcd&date=recent&format=JSON&room_id=id-12345&timezone=UTC')
+      end
+
+      it 'should request to hipchat-api with default option of hipchat api' do
+        message = described_class.new('abcd')
+        message.get_history('id-12345', {})
+      end
+    end
+
+    context 'with options' do
+      before do
+        stub_request_with_headers('https://api.hipchat.com/v2/room/id-12345/history?auth_token=abcd&date=2014-07-12&format=JSON&room_id=id-12345&timezone=JST')
+      end
+
+      it 'should request to hipchat-api with option overridden of hipchat api' do
+        message = described_class.new('abcd')
+        message.get_history('id-12345', {date: '2014-07-12', timezone: 'JST'})
+      end
+    end
+
+    context 'when id must be escaped' do
+      before do
+        stub_request_with_headers('https://api.hipchat.com/v2/room/id%2012345/history?auth_token=abcd&date=recent&format=JSON&room_id=id%2012345&timezone=UTC')
+      end
+
+      it 'should request to url that is escaped' do
+        message = described_class.new('abcd')
+        message.get_history('id 12345', {})
+      end
+    end
+  end
+
+  describe '#history' do
+    before do
+      stub_request_with_headers('https://api.hipchat.com/v2/room/id-12345/history?auth_token=abcd&date=recent&format=JSON&room_id=id-12345&timezone=UTC').to_return(body: response)
+    end
+
+    let(:response) do
+      path = File.join('spec', 'data', 'item-list.json')
+      File.read(path)
+    end
+
+    it 'should return HipchatSearcher::Result instance' do
+      message  = described_class.new('abcd')
+      response = message.history('id-12345')
+      response.should be_instance_of HipchatSearcher::Result
+    end
+
+    it 'when arguments is String should return the room name the same string of arguments' do
+      message  = described_class.new('abcd')
+      response = message.history('id-12345')
+      response.room.should == 'id-12345'
+    end
+
+    it 'when arguments is Hashie::Mash object should return the room name objects given' do
+      room     = Hashie::Mash.new(id: 'id-12345', name: 'room1')
+      message  = described_class.new('abcd')
+      response = message.history(room)
+      response.room.should == 'room1'
+    end
+  end
+end

data/spec/lib/hipchat_searcher/result_spec.rb

@@ -1,5 +1,3 @@
-require 'spec_helper'
-
 describe HipchatSearcher::Result do
   def result(response)
     described_class.new(response)
@@ -61,29 +59,6 @@ describe HipchatSearcher::Result do
     end
   end
 
-  describe '#rooms' do
-    context 'the value' do
-      subject { result(response).rooms }
-
-      let(:response) { eval File.read(path) }
-      let(:path)     { File.join('spec', 'data', 'room-list.txt') }
-
-      it { should be_instance_of Array }
-    end
-
-    context 'the element' do
-      let(:rooms)    { result(response).rooms.first }
-      let(:response) { eval File.read(path) }
-      let(:path)     { File.join('spec', 'data', 'room-list.txt') }
-
-      it { rooms.should be_instance_of ::Hashie::Mash }
-
-      it 'should have keys "id", "links", "name"' do
-        rooms.keys.should == ['id', 'links', 'name']
-      end
-    end
-  end
-
   describe '#items' do
     context 'the value' do
       subject { result(response).items }
@@ -144,4 +119,51 @@ describe HipchatSearcher::Result do
       end
     end
   end
+
+  describe '#oldest_date' do
+    context 'when items is not empty' do
+      subject { result(response).oldest_date }
+
+      let(:response) { File.read(path) }
+      let(:path)     { File.join('spec', 'data', 'item-list.json') }
+
+      it { should be_instance_of String }
+
+      it 'should return the oldest date in any items' do
+        should == "2014-05-30T01:38:16.741565+00:00"
+      end
+    end
+
+    context 'when item is empty' do
+      subject { result(response).oldest_date }
+
+      let(:response) { File.read(path) }
+      let(:path)     { File.join('spec', 'data', 'item-list-empty.json') }
+
+      it { should be_nil }
+    end
+  end
+
+  describe '#rooms' do
+    context 'the value' do
+      subject { result(response).rooms }
+
+      let(:response) { eval File.read(path) }
+      let(:path)     { File.join('spec', 'data', 'room-list.txt') }
+
+      it { should be_instance_of Array }
+    end
+
+    context 'the element' do
+      let(:rooms)    { result(response).rooms.first }
+      let(:response) { eval File.read(path) }
+      let(:path)     { File.join('spec', 'data', 'room-list.txt') }
+
+      it { rooms.should be_instance_of ::Hashie::Mash }
+
+      it 'should have keys "id", "links", "name"' do
+        rooms.keys.should == ['id', 'links', 'name']
+      end
+    end
+  end
 end

data/spec/lib/hipchat_searcher/room_spec.rb

@@ -0,0 +1,58 @@
+require 'webmock/rspec'
+
+describe HipchatSearcher::Room do
+  describe '#get_all_room' do
+    context 'with no options' do
+      before do
+        stub_request(:get, 'https://api.hipchat.com/v2/room?auth_token=abcde')
+      end
+
+      it 'should request to hipchat api' do
+        room = described_class.new('abcde')
+        room.get_all_room
+      end
+    end
+
+    context 'when response code 400 from hipchat api' do
+      before do
+        stub_request(:get, 'https://api.hipchat.com/v2/room?auth_token=abcde')
+          .to_return(status: 401)
+      end
+
+      it 'should raise error HipChat::Unauthorized' do
+        room = described_class.new('abcde')
+        expect do
+          room.get_all_room
+        end.to raise_error HipChat::Unauthorized
+      end
+    end
+
+    context 'when response code 404 from hipchat api' do
+      before do
+        stub_request(:get, 'https://api.hipchat.com/v2/room?auth_token=abcde')
+          .to_return(status: 404)
+      end
+
+      it 'should raise error HipChat::UnknownRoom' do
+        room = described_class.new('abcde')
+        expect do
+          room.get_all_room
+        end.to raise_error HipChat::UnknownRoom
+      end
+    end
+
+    context 'when response code other type from hipchat api' do
+      before do
+        stub_request(:get, 'https://api.hipchat.com/v2/room?auth_token=abcde')
+          .to_return(status: 400)
+      end
+
+      it 'should raise error HipChat::UnknownResponseCode' do
+        room = described_class.new('abcde')
+        expect do
+          room.get_all_room
+        end.to raise_error HipChat::UnknownResponseCode
+      end
+    end
+  end
+end

data/spec/lib/hipchat_searcher/{searcher_spec.rb → search_proxy/grep_spec.rb}

@@ -1,6 +1,4 @@
-require 'spec_helper'
-
-describe HipchatSearcher::Searcher do
+describe HipchatSearcher::SearchProxy::Grep do
   def searcher(pattern, result, options={})
     described_class.new(pattern, result, options)
   end
@@ -22,13 +20,13 @@ describe HipchatSearcher::Searcher do
     end
   end
 
-  describe '#extended_items' do
+  describe '#around_items' do
     describe 'you speciy :before_context option' do
       context 'when specify the range is included in the array' do
-        subject { searcher(pattern, result, before_context: '3').extended_items(3) }
+        subject { searcher(pattern, result, before_context: '3').around_items(3) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array of range to include index you specify' do
           should == %w|1 2 3 4|
@@ -36,10 +34,10 @@ describe HipchatSearcher::Searcher do
       end
 
       context 'when specify the range is exluded in the array' do
-        subject { searcher(pattern, result, before_context: '3').extended_items(1) }
+        subject { searcher(pattern, result, before_context: '3').around_items(1) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array includes range until index from first' do
           should == %w|1 2|
@@ -49,21 +47,21 @@ describe HipchatSearcher::Searcher do
 
     describe 'you specify :after_context option' do
       context 'when specify the range is included in the array' do
-        subject { searcher(pattern, result, after_context: '3').extended_items(1) }
+        subject { searcher(pattern, result, after_context: '3').around_items(3) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array of range to include index you specify' do
-          should == %w|2 3 4 5|
+          should == %w|4 5 6|
         end
       end
 
       context 'when specify the range is exluded in the array' do
-        subject { searcher(pattern, result, after_context: '3').extended_items(4) }
+        subject { searcher(pattern, result, after_context: '3').around_items(4) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array includes range until the end from index' do
           should == %w|5 6|
@@ -73,10 +71,10 @@ describe HipchatSearcher::Searcher do
 
     describe 'you specify :context option' do
       context 'when specify the range is included in the array' do
-        subject { searcher(pattern, result, context: '2').extended_items(3) }
+        subject { searcher(pattern, result, context: '2').around_items(3) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array of range to surround index you specify' do
           should == %w|2 3 4 5 6|
@@ -84,10 +82,10 @@ describe HipchatSearcher::Searcher do
       end
 
       context 'when specify the range is exluded in the array' do
-        subject { searcher(pattern, result, context: '5').extended_items(3) }
+        subject { searcher(pattern, result, context: '5').around_items(3) }
 
         let(:pattern) { 'fuga' }
-        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|) }
+        let(:result)  { double(:result, items: %w|1 2 3 4 5 6|.reverse) }
 
         it 'should return the array includes range to surround as much as possible' do
           should == %w|1 2 3 4 5 6|
@@ -97,61 +95,10 @@ describe HipchatSearcher::Searcher do
   end
 
   describe '#search' do
-    context 'when matches pattern in messages' do
-      subject { searcher(pattern, result).search }
-
-      let(:pattern)  { 'yare' }
-      let(:result) do
-        response = File.read(File.join('spec', 'data', 'item-list.json'))
-        HipchatSearcher::Result.new(response).tap do |r|
-          r.room = "Joestars"
-        end
-      end
-      let(:search_result) do
-        "\e[4;39;49mJoestars\e[0m" + "\n" + \
-        "  Date: 2014-05-30T01:38:16.741565+00:00" + "\n" + \
-        "  @jotaro: \e[0;31;49myare\e[0m\e[0;31;49myare\e[0m daze" + "\n" + \
-        "\n"
-      end
-
-      it 'should print roomname & search result' do
-        expect do
-          subject
-        end.to output(search_result).to_stdout
-      end
-    end
-
-    context 'when matches pattern in many messages' do
-      subject { searcher(pattern, result).search }
-
-      let(:pattern)  { 'ze' }
-      let(:result) do
-        response = File.read(File.join('spec', 'data', 'item-list.json'))
-        HipchatSearcher::Result.new(response).tap do |r|
-          r.room = "Joestars"
-        end
-      end
-      let(:search_result) do
-        "\e[4;39;49mJoestars\e[0m" + "\n" + \
-        "  Date: 2014-05-30T01:38:16.741565+00:00" + "\n" + \
-        "  @jotaro: yareyare da\e[0;31;49mze\e[0m" + "\n" \
-        "\n" + \
-        "  Date: 2014-06-09T11:29:10.209014+00:00" + "\n" + \
-        "  @polnareff: a... arinomama ima okotta koto wo hanasu \e[0;31;49mze\e[0m" + "\n" + \
-        "\n"
-      end
-
-      it 'should print roomname & search result' do
-        expect do
-          subject
-        end.to output(search_result).to_stdout
-      end
-    end
-
     context 'when search_option --before_context' do
       subject { searcher(pattern, result, before_context: '1').search }
 
-      let(:pattern) { 'inu' }
+      let(:pattern) { 'chirp' }
       let(:result) do
         response = File.read(File.join('spec', 'data', 'item-list-with-overlap.json'))
         HipchatSearcher::Result.new(response).tap do |r|
@@ -160,11 +107,11 @@ describe HipchatSearcher::Searcher do
       end
       let(:search_result) do
         "\e[4;39;49mJoestars\e[0m" + "\n" + \
-        "  Date: 2014-06-10T08:36:09.281643+00:00" + "\n" + \
-        "  @abdul: chirp chirp chirp" + "\n" + \
-        "\n" + \
         "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
-        "  @lggy: \e[0;31;49minu\e[0m zuki no kodomo ha migoroshiniha dekine-ze" + "\n" + \
+        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-ze" + "\n" + \
+        "\n" + \
+        "  Date: 2014-06-10T08:36:09.281643+00:00" + "\n" + \
+        "  @abdul: \e[0;31;49mchirp\e[0m \e[0;31;49mchirp\e[0m \e[0;31;49mchirp\e[0m" + "\n" + \
         "\n"
       end
 
@@ -190,8 +137,8 @@ describe HipchatSearcher::Searcher do
         "  Date: 2014-05-30T01:39:02.186319+00:00" + "\n" + \
         "  @noriaki: \e[0;31;49mrero\e[0m\e[0;31;49mrero\e[0m" + "\n" + \
         "\n" + \
-        "  Date: 2014-06-09T11:29:10.209014+00:00" + "\n" + \
-        "  @polnareff: a... arinomama ima okotta koto wo hanasu ze" + "\n" + \
+        "  Date: 2014-05-30T01:38:16.741565+00:00" + "\n" + \
+        "  @jotaro: yareyare daze" + "\n" + \
         "\n"
       end
 
@@ -203,7 +150,7 @@ describe HipchatSearcher::Searcher do
     end
 
     context 'when search_option --context' do
-      subject { searcher(pattern, result, context: '2').search }
+      subject { searcher(pattern, result, context: '1').search }
 
       let(:pattern) { 'chirp' }
       let(:result) do
@@ -214,17 +161,14 @@ describe HipchatSearcher::Searcher do
       end
       let(:search_result) do
         "\e[4;39;49mJoestars\e[0m" + "\n" + \
-        "  Date: 2014-05-30T01:39:02.186319+00:00" + "\n" + \
-        "  @noriaki: rerorero" + "\n" + \
-        "\n" + \
-        "  Date: 2014-06-09T11:29:10.209014+00:00" + "\n" + \
-        "  @polnareff: a... arinomama ima okotta koto wo hanasu ze" + "\n" + \
+        "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
+        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-ze" + "\n" + \
         "\n" + \
         "  Date: 2014-06-10T08:36:09.281643+00:00" + "\n" + \
         "  @abdul: \e[0;31;49mchirp\e[0m \e[0;31;49mchirp\e[0m \e[0;31;49mchirp\e[0m" + "\n" + \
         "\n" + \
-        "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
-        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-ze" + "\n" + \
+        "  Date: 2014-06-09T11:29:10.209014+00:00" + "\n" + \
+        "  @polnareff: a... arinomama ima okotta koto wo hanasu ze" + "\n" + \
         "\n"
       end
 
@@ -247,20 +191,20 @@ describe HipchatSearcher::Searcher do
       end
       let(:search_result) do
         "\e[4;39;49mJoestars\e[0m" + "\n" + \
-        "  Date: 2014-05-30T01:38:16.741565+00:00" + "\n" + \
-        "  @jotaro: yareyare da\e[0;31;49mze\e[0m" + "\n" + \
+        "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
+        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-\e[0;31;49mze\e[0m" + "\n" + \
         "\n" + \
-        "  Date: 2014-05-30T01:39:02.186319+00:00" + "\n" + \
-        "  @noriaki: rerorero" + "\n" + \
+        "  Date: 2014-06-10T08:36:09.281643+00:00" + "\n" + \
+        "  @abdul: chirp chirp chirp" + "\n" + \
         "\n" + \
         "  Date: 2014-06-09T11:29:10.209014+00:00" + "\n" + \
         "  @polnareff: a... arinomama ima okotta koto wo hanasu \e[0;31;49mze\e[0m" + "\n" + \
         "\n" + \
-        "  Date: 2014-06-10T08:36:09.281643+00:00" + "\n" + \
-        "  @abdul: chirp chirp chirp" + "\n" + \
+        "  Date: 2014-05-30T01:39:02.186319+00:00" + "\n" + \
+        "  @noriaki: rerorero" + "\n" + \
         "\n" + \
-        "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
-        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-\e[0;31;49mze\e[0m" + "\n" + \
+        "  Date: 2014-05-30T01:38:16.741565+00:00" + "\n" + \
+        "  @jotaro: yareyare da\e[0;31;49mze\e[0m" + "\n" + \
         "\n"
       end
 
@@ -271,37 +215,28 @@ describe HipchatSearcher::Searcher do
       end
     end
 
-    context "when don't match pattern in messages" do
-      subject { searcher(pattern, result).search }
+    context 'when you specify option date and include item date no later than this date' do
+      subject { searcher(pattern, result, date: '2014-06-10', before_context: 1).search }
 
-      let(:pattern) { 'abcd' }
+      let(:pattern) { 'ze' }
       let(:result) do
-        response = File.read(File.join('spec', 'data', 'item-list.json'))
-        HipchatSearcher::Result.new(response)
-      end
-
-      it { should be_nil }
-
-      it 'should no output to stdout' do
-        expect do
-          subject
-        end.to output('').to_stdout
+        response = File.read(File.join('spec', 'data', 'item-list-with-overlap.json'))
+        HipchatSearcher::Result.new(response).tap do |r|
+          r.room = "Joestars"
+        end
       end
-    end
-
-    context "when pattern can't convert regexp" do
-      subject { searcher(pattern, result).search }
 
-      let(:pattern) { nil }
-      let(:result) do
-        response = File.read(File.join('spec', 'data', 'item-list.json'))
-        HipchatSearcher::Result.new(response)
+      let(:search_result) do
+        "\e[4;39;49mJoestars\e[0m" + "\n" + \
+        "  Date: 2014-06-11T19:20:47.726182+00:00" + "\n" + \
+        "  @lggy: inu zuki no kodomo ha migoroshiniha dekine-\e[0;31;49mze\e[0m" + "\n" + \
+        "\n"
       end
 
-      it 'should raise exception' do
+      it 'should print the matched message since option date' do
         expect do
           subject
-        end.to raise_error
+        end.to output(search_result).to_stdout
       end
     end
   end
@@ -311,7 +246,7 @@ describe HipchatSearcher::Searcher do
       hash  = JSON.parse(File.read(path))
       items = ::Hashie::Mash.new(hash).items
       items.map do |itm|
-        itm.extend(HipchatSearcher::Searcher::ItemExtention).tap do |i|
+        itm.extend(HipchatSearcher::ItemExtention).tap do |i|
           i.pattern = Regexp.new(pattern)
         end
       end