Add notifications api single thread retrieval.

piotrmurach · piotrmurach · commit 9dd5d509654a · 2012-11-03T20:19:53.000Z
diff --git a/lib/github_api/activity/notifications.rb b/lib/github_api/activity/notifications.rb
@@ -48,5 +48,21 @@ def list(*args)
     end
     alias :all :list
 
+    # View a single thread
+    #
+    # = Examples
+    #  github = Github.new oauth_token: 'token'
+    #  github.activity.notifications.get 'thread_id'
+    #  github.activity.notifications.get 'thread_id' { |thread| ... }
+    #
+    def get(thread_id, params={})
+      assert_presence_of thread_id
+      normalize! params
+      response = get_request("/notifications/threads/#{thread_id}", params)
+      return response unless block_given?
+      response.each { |el| yield el }
+    end
+    alias :find :get
+
   end # Activity::Notifications
 end # Github
diff --git a/spec/fixtures/activity/threads.json b/spec/fixtures/activity/threads.json
@@ -0,0 +1,32 @@
+[
+  {
+    "id": 1,
+      "repository": {
+        "id": 1296269,
+        "owner": {
+          "login": "octocat",
+          "id": 1,
+          "avatar_url": "https://github.com/images/error/octocat_happy.gif",
+          "gravatar_id": "somehexcode",
+          "url": "https://api.github.com/users/octocat"
+        },
+      "name": "Hello-World",
+      "full_name": "octocat/Hello-World",
+      "description": "This your first repo!",
+      "private": false,
+      "fork": false,
+      "url": "https://api.github.com/repos/octocat/Hello-World",
+      "html_url": "https://github.com/octocat/Hello-World"
+    },
+    "subject": {
+      "title": "Greetings",
+      "url": "https://api.github.com/repos/pengwynn/octokit/issues/123",
+      "latest_comment_url": "https://api.github.com/repos/pengwynn/octokit/issues/comments/123"
+    },
+    "reason": "subscribed",
+    "unread": true,
+    "updated_at": "2012-09-25T07:54:41-07:00",
+    "last_read_at": "2012-09-25T07:54:41-07:00",
+    "url": "https://api.github.com/notifications/threads/1"
+  }
+]
diff --git a/spec/github/activity/notifications/get_spec.rb b/spec/github/activity/notifications/get_spec.rb
@@ -0,0 +1,58 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+describe Github::Activity::Notifications, '#get' do
+  let(:thread_id) { 1 }
+  let(:request_path) { "/notifications/threads/#{thread_id}" }
+
+  before {
+    stub_get(request_path).to_return(:body => body, :status => status,
+      :headers => {:content_type => "application/json; charset=utf-8"})
+  }
+
+  after { reset_authentication_for subject }
+
+  context "resource found" do
+    let(:body)   { fixture('activity/threads.json') }
+    let(:status) { 200 }
+
+    it { should respond_to(:find) }
+
+    it "should raise error when no thread-id parameter" do
+      expect { subject.get nil }.to raise_error(ArgumentError)
+    end
+
+    it "should find resources" do
+      subject.get thread_id
+      a_get(request_path).should have_been_made
+    end
+
+    it "should return repository mash" do
+      threads = subject.get thread_id
+      threads.should be_an Array
+      threads.should have(1).items
+    end
+
+    it "should get repository information" do
+      threads = subject.get thread_id
+      threads.first.repository.name.should == 'Hello-World'
+    end
+
+    it "should yield repositories to a block" do
+      subject.should_receive(:get).and_yield('octocat')
+      subject.get(thread_id) { |repo| 'octocat' }
+    end
+  end
+
+  context "resource not found" do
+    let(:body)   { '' }
+    let(:status) { 404 }
+
+    it "should fail to get resource" do
+      expect {
+        subject.get thread_id
+      }.to raise_error(Github::Error::NotFound)
+    end
+  end
+end # get
