tracker_api 1.8.0 → 1.12.0

data/lib/tracker_api/resources/review.rb

@@ -0,0 +1,35 @@
+module TrackerApi
+  module Resources
+    class Review
+      include Shared::Base
+
+      attribute :client
+
+      attribute :id, Integer
+      attribute :story_id, Integer
+      attribute :project_id, Integer
+      attribute :review_type_id, Integer
+      attribute :reviewer_id, Integer
+      attribute :status, String # (unstarted, in_review, pass, revise)
+      attribute :created_at, DateTime
+      attribute :updated_at, DateTime
+      attribute :kind, String
+      attribute :review_type, ReviewType
+
+      class UpdateRepresenter < Representable::Decorator
+        include Representable::JSON
+
+        property :id
+        property :review_type_id
+        property :reviewer_id
+        property :status
+      end
+
+      def save
+        raise ArgumentError, 'Cannot update a review with an unknown story_id.' if story_id.nil?
+
+        Endpoints::Review.new(client).update(self, UpdateRepresenter.new(Review.new(dirty_attributes)))
+      end
+    end
+  end
+end

data/lib/tracker_api/resources/review_type.rb

@@ -0,0 +1,15 @@
+module TrackerApi
+  module Resources
+    class ReviewType
+      include Shared::Base
+
+      attribute :id, Integer
+      attribute :project_id, Integer
+      attribute :name, String
+      attribute :hidden, Boolean
+      attribute :created_at, DateTime
+      attribute :updated_at, DateTime
+      attribute :kind, String
+    end
+  end
+end

data/lib/tracker_api/resources/story.rb

@@ -8,6 +8,7 @@ module TrackerApi
       attribute :accepted_at, DateTime
       attribute :after_id, Integer
       attribute :before_id, Integer
+      attribute :blockers, [Blocker]
       attribute :comment_ids, [Integer]
       attribute :comments, [Comment]
       attribute :created_at, DateTime
@@ -31,6 +32,7 @@ module TrackerApi
       attribute :project_id, Integer
       attribute :requested_by, Person
       attribute :requested_by_id, Integer
+      attribute :reviews, [Review]
       attribute :story_type, String # (feature, bug, chore, release)
       attribute :task_ids, [Integer]
       attribute :tasks, [Task]
@@ -54,6 +56,7 @@ module TrackerApi
         property :deadline
         property :requested_by_id
         property :owner_ids, if: ->(_) { !owner_ids.blank? }
+        property :project_id
 
         # Use render_empty: false to address: https://github.com/dashofcode/tracker_api/issues/110
         # - The default value of the labels attribute in Resources::Story is an empty array.
@@ -117,6 +120,10 @@ module TrackerApi
         Endpoints::Activity.new(client).get_story(project_id, id, params)
       end
 
+      def blockers(params = {})
+        Endpoints::Blockers.new(client).get(project_id, id, params)
+      end
+
       # Provides a list of all the comments on the story.
       #
       # @param [Hash] params
@@ -165,6 +172,17 @@ module TrackerApi
         end
       end
 
+      # Returns the story's original ("undirtied") project_id
+      #
+      # @return Integer
+      def project_id
+        if dirty_attributes.key?(:project_id)
+          original_attributes[:project_id]
+        else
+          @project_id
+        end
+      end
+
       # @param [Hash] params attributes to create the task with
       # @return [Task] newly created Task
       def create_task(params)
@@ -187,6 +205,14 @@ module TrackerApi
 
         Endpoints::Story.new(client).update(self, UpdateRepresenter.new(Story.new(self.dirty_attributes)))
       end
+
+      def reviews(params = {})
+          if params.blank? && @reviews.present?
+            @reviews
+          else
+            @reviews = Endpoints::Reviews.new(client).get(project_id, id, params)
+          end
+      end
     end
   end
 end

data/lib/tracker_api/version.rb

@@ -1,3 +1,3 @@
 module TrackerApi
-  VERSION = '1.8.0'
+  VERSION = '1.12.0'
 end

data/test/client_test.rb

@@ -2,7 +2,7 @@ require_relative 'minitest_helper'
 
 describe TrackerApi do
   it 'has a version' do
-    ::TrackerApi::VERSION.wont_be_nil
+    _(::TrackerApi::VERSION).wont_be_nil
   end
 end
 
@@ -13,10 +13,10 @@ describe TrackerApi::Client do
                                     token:       '12345',
                                     logger:      LOGGER)
 
-    client.url.must_equal 'http://test.com'
-    client.api_version.must_equal '/foo-bar/1'
-    client.token.must_equal '12345'
-    client.logger.must_equal LOGGER
+    _(client.url).must_equal 'http://test.com'
+    _(client.api_version).must_equal '/foo-bar/1'
+    _(client.token).must_equal '12345'
+    _(client.logger).must_equal LOGGER
   end
 
   describe '.projects' do
@@ -27,18 +27,18 @@ describe TrackerApi::Client do
       VCR.use_cassette('get all projects', record: :new_episodes) do
         projects = client.projects(fields: ':default,account,current_velocity,labels(name),epics(:default,label(name))')
 
-        projects.wont_be_empty
+        _(projects).wont_be_empty
         project = projects.first
-        project.must_be_instance_of TrackerApi::Resources::Project
-        project.id.must_equal pt_user[:project_id]
+        _(project).must_be_instance_of TrackerApi::Resources::Project
+        _(project.id).must_equal pt_user[:project_id]
 
-        project.account.must_be_instance_of TrackerApi::Resources::Account
+        _(project.account).must_be_instance_of TrackerApi::Resources::Account
 
-        project.labels.wont_be_empty
-        project.labels.first.must_be_instance_of TrackerApi::Resources::Label
+        _(project.labels).wont_be_empty
+        _(project.labels.first).must_be_instance_of TrackerApi::Resources::Label
 
-        project.epics.wont_be_empty
-        project.epics.first.must_be_instance_of TrackerApi::Resources::Epic
+        _(project.epics).wont_be_empty
+        _(project.epics.first).must_be_instance_of TrackerApi::Resources::Epic
       end
     end
   end
@@ -52,11 +52,11 @@ describe TrackerApi::Client do
       VCR.use_cassette('get project', record: :new_episodes) do
         project = client.project(project_id)
 
-        project.must_be_instance_of TrackerApi::Resources::Project
-        project.id.must_equal project_id
+        _(project).must_be_instance_of TrackerApi::Resources::Project
+        _(project.id).must_equal project_id
 
-        project.account.must_be_nil
-        project.account_id.wont_be_nil
+        _(project.account).must_be_nil
+        _(project.account_id).wont_be_nil
       end
     end
   end
@@ -69,9 +69,9 @@ describe TrackerApi::Client do
       VCR.use_cassette('get workspace', record: :new_episodes) do
         workspace = client.workspace(pt_user[:workspace_id])
 
-        workspace.must_be_instance_of TrackerApi::Resources::Workspace
-        workspace.id.must_equal pt_user[:workspace_id]
-        workspace.name.wont_be_empty
+        _(workspace).must_be_instance_of TrackerApi::Resources::Workspace
+        _(workspace.id).must_equal pt_user[:workspace_id]
+        _(workspace.name).wont_be_empty
       end
     end
   end
@@ -85,10 +85,10 @@ describe TrackerApi::Client do
       VCR.use_cassette('get all workspaces', record: :new_episodes) do
         workspaces = client.workspaces(fields: ':default,projects(id,name)')
 
-        workspaces.wont_be_empty
+        _(workspaces).wont_be_empty
         workspace = workspaces.first
-        workspace.must_be_instance_of TrackerApi::Resources::Workspace
-        workspace.id.must_equal pt_user[:workspace_id]
+        _(workspace).must_be_instance_of TrackerApi::Resources::Workspace
+        _(workspace.id).must_equal pt_user[:workspace_id]
       end
     end
   end
@@ -105,10 +105,10 @@ describe TrackerApi::Client do
       VCR.use_cassette('get me', record: :new_episodes) do
         me = client.me
 
-        me.must_be_instance_of TrackerApi::Resources::Me
-        me.username.must_equal username
+        _(me).must_be_instance_of TrackerApi::Resources::Me
+        _(me.username).must_equal username
 
-        me.projects.map(&:project_id).must_include project_id
+        _(me.projects.map(&:project_id)).must_include project_id
       end
     end
   end
@@ -124,14 +124,14 @@ describe TrackerApi::Client do
 
         # skip pagination with a hugh limit
         unpaged_stories = project.stories(limit: 300)
-        unpaged_stories.wont_be_empty
-        unpaged_stories.length.must_be :>, 7
+        _(unpaged_stories).wont_be_empty
+        _(unpaged_stories.length).must_be :>, 7
 
         # force pagination with a small limit
         paged_stories = project.stories(limit: 7)
-        paged_stories.wont_be_empty
-        paged_stories.length.must_equal unpaged_stories.length
-        paged_stories.map(&:id).sort.uniq.must_equal unpaged_stories.map(&:id).sort.uniq
+        _(paged_stories).wont_be_empty
+        _(paged_stories.length).must_equal unpaged_stories.length
+        _(paged_stories.map(&:id).sort.uniq).must_equal unpaged_stories.map(&:id).sort.uniq
       end
     end
 
@@ -141,8 +141,8 @@ describe TrackerApi::Client do
 
         # force no pagination
         stories = project.stories(limit: 7, auto_paginate: false)
-        stories.wont_be_empty
-        stories.length.must_equal 7
+        _(stories).wont_be_empty
+        _(stories.length).must_equal 7
       end
     end
 
@@ -152,8 +152,8 @@ describe TrackerApi::Client do
 
         done_iterations = project.iterations(scope: :done, offset: -12, limit: 5)
 
-        done_iterations.wont_be_empty
-        done_iterations.length.must_be :<=, 12
+        _(done_iterations).wont_be_empty
+        _(done_iterations.length).must_be :<=, 12
       end
     end
   end
@@ -166,8 +166,8 @@ describe TrackerApi::Client do
       VCR.use_cassette('client: get single story by story id', record: :new_episodes) do
         story = client.story('66728004', fields: ':default,owned_by')
 
-        story.must_be_instance_of TrackerApi::Resources::Story
-        story.owned_by.wont_be_nil
+        _(story).must_be_instance_of TrackerApi::Resources::Story
+        _(story.owned_by).wont_be_nil
       end
     end
   end
@@ -180,8 +180,8 @@ describe TrackerApi::Client do
       VCR.use_cassette('client: get single epic by epic id', record: :new_episodes) do
         epic = client.epic('1087314', fields: ':default,label_id')
 
-        epic.must_be_instance_of TrackerApi::Resources::Epic
-        epic.label_id.wont_be_nil
+        _(epic).must_be_instance_of TrackerApi::Resources::Epic
+        _(epic.label_id).wont_be_nil
       end
     end
   end
@@ -194,13 +194,13 @@ describe TrackerApi::Client do
       VCR.use_cassette('get all notifications', record: :new_episodes) do
         notifications = client.notifications
 
-        notifications.wont_be_empty
+        _(notifications).wont_be_empty
         notification = notifications.first
-        notification.must_be_instance_of TrackerApi::Resources::Notification
+        _(notification).must_be_instance_of TrackerApi::Resources::Notification
 
-        notification.project.id.must_equal pt_user[:project_id]
-        notification.story.must_be_instance_of TrackerApi::Resources::Story
-        notification.performer.must_be_instance_of TrackerApi::Resources::Person
+        _(notification.project.id).must_equal pt_user[:project_id]
+        _(notification.story).must_be_instance_of TrackerApi::Resources::Story
+        _(notification.performer).must_be_instance_of TrackerApi::Resources::Person
       end
     end
   end
@@ -213,19 +213,19 @@ describe TrackerApi::Client do
       VCR.use_cassette('get my activities', record: :new_episodes) do
         activities = client.activity(fields: ':default')
 
-        activities.wont_be_empty
+        _(activities).wont_be_empty
         activity = activities.first
-        activity.must_be_instance_of TrackerApi::Resources::Activity
+        _(activity).must_be_instance_of TrackerApi::Resources::Activity
 
-        activity.changes.wont_be_empty
-        activity.changes.first.must_be_instance_of TrackerApi::Resources::Change
+        _(activity.changes).wont_be_empty
+        _(activity.changes.first).must_be_instance_of TrackerApi::Resources::Change
 
-        activity.primary_resources.wont_be_empty
-        activity.primary_resources.first.must_be_instance_of TrackerApi::Resources::PrimaryResource
+        _(activity.primary_resources).wont_be_empty
+        _(activity.primary_resources.first).must_be_instance_of TrackerApi::Resources::PrimaryResource
 
-        activity.project.must_be_instance_of TrackerApi::Resources::Project
+        _(activity.project).must_be_instance_of TrackerApi::Resources::Project
 
-        activity.performed_by.must_be_instance_of TrackerApi::Resources::Person
+        _(activity.performed_by).must_be_instance_of TrackerApi::Resources::Person
       end
     end
   end

data/test/comment_test.rb

@@ -17,8 +17,8 @@ describe TrackerApi::Resources::Comment do
       comment = story.create_comment(text: text)
     end
 
-    comment.text.must_equal text
-    comment.clean?.must_equal true
+    _(comment.text).must_equal text
+    _(comment.clean?).must_equal true
   end
 
   it 'can create a comment with file attachment' do
@@ -28,9 +28,9 @@ describe TrackerApi::Resources::Comment do
     VCR.use_cassette('create comment with attachment', record: :new_episodes) do
       comment = story.create_comment(text: text, files: files)
     end
-    comment.text.must_equal text
-    comment.attachments.size.must_equal 1
-    comment.clean?.must_equal true
+    _(comment.text).must_equal text
+    _(comment.attachments.size).must_equal 1
+    _(comment.clean?).must_equal true
   end
 
   it 'can update an existing comment' do
@@ -41,16 +41,16 @@ describe TrackerApi::Resources::Comment do
       existing_comment.save
     end
 
-    existing_comment.text.must_equal new_text
-    existing_comment.clean?.must_equal true
+    _(existing_comment.text).must_equal new_text
+    _(existing_comment.clean?).must_equal true
   end
 
   it 'can create attachments in a comment' do
     files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
     VCR.use_cassette('create attachments', record: :new_episodes) do
       existing_comment.create_attachments(files: files)
-      existing_comment.attachments.size.must_equal 1
-      existing_comment.clean?.must_equal true
+      _(existing_comment.attachments.size).must_equal 1
+      _(existing_comment.clean?).must_equal true
     end
   end
 
@@ -58,9 +58,9 @@ describe TrackerApi::Resources::Comment do
     files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
     VCR.use_cassette('delete attachments', record: :new_episodes) do
       existing_comment.create_attachments(files: files)
-      existing_comment.attachments.size.must_equal 1
+      _(existing_comment.attachments.size).must_equal 1
       existing_comment.delete_attachments
-      existing_comment.attachments.size.must_equal 0
+      _(existing_comment.attachments.size).must_equal 0
     end
   end
 
@@ -68,10 +68,10 @@ describe TrackerApi::Resources::Comment do
     VCR.use_cassette('delete comment', record: :new_episodes) do
       current_story = project.story(story_id)
       new_comment_id = current_story.create_comment(text: "test comment").id
-      current_story.comments.last.id.must_equal new_comment_id
+      _(current_story.comments.last.id).must_equal new_comment_id
       current_story.comments.last.delete
       current_story = project.story(story_id)
-      current_story.comments.last.id.wont_equal new_comment_id
+      _(current_story.comments.last.id).wont_equal new_comment_id
     end
   end
 end

data/test/error_test.rb

@@ -22,7 +22,7 @@ describe TrackerApi::Error do
       end
     end
   end
-  
+
   it 'raises RuntimeError for HTTP status codes < 400 and > 500' do
     [399, 600].each do |status_code|
       mock_faraday_error(status_code)
@@ -35,7 +35,13 @@ describe TrackerApi::Error do
   # Simulate the error Faraday will raise with a specific HTTP status code so
   # we can test our rescuing of those errors
   def mock_faraday_error(status_code)
+    mocked_error_class = if (500..599).include?(status_code) && Faraday::VERSION.to_f >= 16.0
+      Faraday::ServerError
+    else
+      Faraday::ClientError
+    end
+
     ::Faraday::Connection.any_instance.stubs(:get).
-      raises(::Faraday::Error::ClientError.new(nil, { status: status_code}))
+      raises(mocked_error_class.new(nil, { status: status_code}))
   end
 end

data/test/file_attachment_test.rb

@@ -7,13 +7,13 @@ describe TrackerApi::Resources::FileAttachment do
   let(:project) { VCR.use_cassette('get project') { client.project(project_id) } }
   let(:story_id) { '66728004' }
   let(:story) { VCR.use_cassette('get story') { project.story(story_id) } }
-      
+
   it 'can be deleted' do
     VCR.use_cassette('delete an attachment', record: :new_episodes) do
       comment_with_attachments = story.create_comment(text: "test comment", files: [File.expand_path('../Gemfile', File.dirname(__FILE__))])
-      comment_with_attachments.attachments(reload: true).size.must_equal 1
+      _(comment_with_attachments.attachments(reload: true).size).must_equal 1
       comment_with_attachments.attachments.first.delete
-      comment_with_attachments.attachments(reload: true).size.must_equal 1
+      _(comment_with_attachments.attachments(reload: true).size).must_equal 0
     end
   end
-end
+end

data/test/iteration_test.rb

@@ -0,0 +1,31 @@
+require_relative 'minitest_helper'
+
+describe TrackerApi::Resources::Iteration do
+  let(:pt_user) { PT_USER_1 }
+  let(:client) { TrackerApi::Client.new token: pt_user[:token] }
+  let(:project_id) { pt_user[:project_id] }
+  let(:project) { VCR.use_cassette('get project') { client.project(project_id) } }
+  let(:iteration) { VCR.use_cassette('get current iteration') { project.iterations(scope: "current").first } }
+
+  describe "#cycle_time_details" do
+    it "gets all cycle_time_details for this iteration" do
+      VCR.use_cassette('get cycle time details', record: :new_episodes) do
+        cycle_time_details = iteration.cycle_time_details
+
+        cycle_time_details.wont_be_empty
+        cycle_time_detail = cycle_time_details.first
+        cycle_time_detail.must_be_instance_of TrackerApi::Resources::CycleTimeDetails
+      end
+    end
+  end
+
+  describe "#get_history" do
+    it "gets all history for a particular iteration" do
+      VCR.use_cassette('get daily history container', record: :new_episodes) do
+        daily_history_container = iteration.get_history
+
+        daily_history_container.must_be_instance_of TrackerApi::Resources::DailyHistoryContainer
+      end
+    end
+  end
+end