tracker_api 1.8.0 → 1.12.0

data/test/minitest_helper.rb

@@ -6,7 +6,10 @@ Coveralls.wear!
 
 require 'minitest/byebug' if ENV['DEBUG']
 require 'minitest/autorun'
-require 'mocha/mini_test'
+
+require 'mocha'
+require('mocha/minitest')
+
 require 'awesome_print'
 require 'multi_json'
 require 'vcr'
@@ -20,7 +23,7 @@ VCR.configure do |c|
   c.ignore_localhost         = true
   c.cassette_library_dir     = File.expand_path('../vcr/cassettes', __FILE__).to_s
   c.default_cassette_options = { serialize_with: :json }
-  c.hook_into :excon
+  c.hook_into :faraday
   c.allow_http_connections_when_no_cassette = false
 end
 

data/test/project_test.rb

@@ -11,9 +11,9 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get epics', record: :new_episodes) do
         epics = project.epics
 
-        epics.wont_be_empty
+        _(epics).wont_be_empty
         epic = epics.first
-        epic.must_be_instance_of TrackerApi::Resources::Epic
+        _(epic).must_be_instance_of TrackerApi::Resources::Epic
       end
     end
 
@@ -28,9 +28,9 @@ describe TrackerApi::Resources::Project do
       it 'does not make an extra request' do
         epics = project_with_epics.epics
 
-        epics.wont_be_empty
+        _(epics).wont_be_empty
         epic = epics.first
-        epic.must_be_instance_of TrackerApi::Resources::Epic
+        _(epic).must_be_instance_of TrackerApi::Resources::Epic
       end
     end
   end
@@ -46,9 +46,9 @@ describe TrackerApi::Resources::Project do
       it 'gets all labels for this project' do
         labels = project_with_labels.labels
 
-        labels.wont_be_empty
+        _(labels).wont_be_empty
         label = labels.first
-        label.must_be_instance_of TrackerApi::Resources::Label
+        _(label).must_be_instance_of TrackerApi::Resources::Label
       end
     end
 
@@ -56,9 +56,9 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get labels', record: :new_episodes) do
         labels = project.labels
 
-        labels.wont_be_empty
+        _(labels).wont_be_empty
         label = labels.first
-        label.must_be_instance_of TrackerApi::Resources::Label
+        _(label).must_be_instance_of TrackerApi::Resources::Label
       end
     end
   end
@@ -69,11 +69,11 @@ describe TrackerApi::Resources::Project do
         offset          = -project.number_of_done_iterations_to_show.to_i
         done_iterations = project.iterations(scope: :done, offset: offset)
 
-        done_iterations.wont_be_empty
-        done_iterations.length.must_be :<=, project.number_of_done_iterations_to_show
+        _(done_iterations).wont_be_empty
+        _(done_iterations.length).must_be :<=, project.number_of_done_iterations_to_show
 
         iteration = done_iterations.first
-        iteration.must_be_instance_of TrackerApi::Resources::Iteration
+        _(iteration).must_be_instance_of TrackerApi::Resources::Iteration
       end
     end
 
@@ -81,14 +81,14 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get current iteration', record: :new_episodes) do
         iterations = project.iterations(scope: :current)
 
-        iterations.wont_be_empty
+        _(iterations).wont_be_empty
 
         current = iterations.first
-        current.must_be_instance_of TrackerApi::Resources::Iteration
-        current.stories.wont_be_empty
+        _(current).must_be_instance_of TrackerApi::Resources::Iteration
+        _(current.stories).wont_be_empty
 
         story = current.stories.first
-        story.must_be_instance_of TrackerApi::Resources::Story
+        _(story).must_be_instance_of TrackerApi::Resources::Story
       end
     end
 
@@ -96,13 +96,13 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get current iteration', record: :new_episodes) do
         iterations = project.iterations(scope: :current, fields: ":default,velocity,points,accepted_points,effective_points")
 
-        iterations.wont_be_empty
+        _(iterations).wont_be_empty
 
         current = iterations.first
-        current.velocity.must_equal 10.0
-        current.points.must_equal 9.0
-        current.accepted_points.must_equal 0
-        current.effective_points.must_equal 9.0
+        _(current.velocity).must_equal 10.0
+        _(current.points).must_equal 9.0
+        _(current.accepted_points).must_equal 0
+        _(current.effective_points).must_equal 9.0
       end
     end
 
@@ -110,26 +110,26 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get iteration by number', record: :new_episodes) do
         iterations = project.iterations(number: 2)
 
-        iterations.size.must_equal 1
-        iterations.first.must_be_instance_of TrackerApi::Resources::Iteration
-        iterations.first.number.must_equal 2
+        _(iterations.size).must_equal 1
+        _(iterations.first).must_be_instance_of TrackerApi::Resources::Iteration
+        _(iterations.first.number).must_equal 2
 
         iterations = project.iterations(number: 1)
 
-        iterations.size.must_equal 1
-        iterations.first.must_be_instance_of TrackerApi::Resources::Iteration
-        iterations.first.number.must_equal 1
+        _(iterations.size).must_equal 1
+        _(iterations.first).must_be_instance_of TrackerApi::Resources::Iteration
+        _(iterations.first.number).must_equal 1
 
         iterations = project.iterations(number: 10_000)
 
-        iterations.must_be_empty
+        _(iterations).must_be_empty
       end
     end
 
     it 'requires an iteration number > 0' do
       VCR.use_cassette('get iteration by number', record: :new_episodes) do
-        -> { project.iterations(number: 0) }.must_raise(ArgumentError, /> 0/)
-        -> { project.iterations(number: -1) }.must_raise(ArgumentError, /> 0/)
+        _(-> { project.iterations(number: 0) }).must_raise(ArgumentError, /> 0/)
+        _(-> { project.iterations(number: -1) }).must_raise(ArgumentError, /> 0/)
       end
     end
   end
@@ -139,11 +139,11 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get unscheduled stories', record: :new_episodes) do
         stories = project.stories(with_state: :unscheduled)
 
-        stories.wont_be_empty
+        _(stories).wont_be_empty
 
         story = stories.first
-        story.must_be_instance_of TrackerApi::Resources::Story
-        story.current_state.must_equal 'unscheduled'
+        _(story).must_be_instance_of TrackerApi::Resources::Story
+        _(story.current_state).must_equal 'unscheduled'
       end
     end
 
@@ -151,10 +151,10 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('create story') do
         story = project.create_story(name: 'Test story')
 
-        story.must_be_instance_of TrackerApi::Resources::Story
-        story.id.wont_be_nil
-        story.id.must_be :>, 0
-        story.name.must_equal 'Test story'
+        _(story).must_be_instance_of TrackerApi::Resources::Story
+        _(_(story.id)).wont_be_nil
+        _(story.id).must_be :>, 0
+        _(story.name).must_equal 'Test story'
       end
     end
 
@@ -162,10 +162,10 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('create story with lengthy params') do
         story = project.create_story(name: 'Test story', description: ('Test description ' * 500))
 
-        story.must_be_instance_of TrackerApi::Resources::Story
-        story.id.wont_be_nil
-        story.id.must_be :>, 0
-        story.description.must_equal ('Test description ' * 500)
+        _(story).must_be_instance_of TrackerApi::Resources::Story
+        _(_(story.id)).wont_be_nil
+        _(story.id).must_be :>, 0
+        _(story.description).must_equal ('Test description ' * 500)
       end
     end
   end
@@ -186,9 +186,9 @@ describe TrackerApi::Resources::Project do
       VCR.use_cassette('get project activity', record: :new_episodes) do
         activity = project.activity
 
-        activity.wont_be_empty
+        _(activity).wont_be_empty
         event = activity.first
-        event.must_be_instance_of TrackerApi::Resources::Activity
+        _(event).must_be_instance_of TrackerApi::Resources::Activity
       end
     end
   end
@@ -201,11 +201,23 @@ describe TrackerApi::Resources::Project do
         project = client.project(pt_user[:project_id])
         search_container = project.search('name:"story to test search"')
 
-        search_container.wont_be_nil
-        search_container.must_be_instance_of TrackerApi::Resources::SearchResultContainer
-        search_container.epics.must_be_instance_of TrackerApi::Resources::EpicsSearchResult
-        search_container.stories.must_be_instance_of TrackerApi::Resources::StoriesSearchResult
-        search_container.stories.stories.first[:id].must_equal 143444685
+        _(search_container).wont_be_nil
+        _(search_container).must_be_instance_of TrackerApi::Resources::SearchResultContainer
+        _(search_container.epics).must_be_instance_of TrackerApi::Resources::EpicsSearchResult
+        _(search_container.stories).must_be_instance_of TrackerApi::Resources::StoriesSearchResult
+        _(search_container.stories.stories.first[:id]).must_equal 143444685
+      end
+    end
+  end
+
+  describe '.releases' do
+    it 'gets all of the releases for the project' do
+      VCR.use_cassette('get releases', record: :new_episodes) do
+        releases = project.releases
+
+        _(releases).wont_be_empty
+        _(releases.size).must_equal 3
+        _(releases.first).must_be_instance_of TrackerApi::Resources::Release
       end
     end
   end

data/test/release_test.rb

@@ -0,0 +1,22 @@
+require_relative 'minitest_helper'
+
+describe TrackerApi::Resources::Release 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) } }
+
+  describe '.stories' do
+    it 'returns all the stories related to a release' do
+      releases = VCR.use_cassette('get releases') { project.releases }
+      release = releases.find { |release| release.name == 'Beta launch' }
+
+      VCR.use_cassette('release stories', record: :new_episodes) do
+        stories = release.stories
+
+        _(stories.size).must_equal 9
+        _(stories.first).must_be_instance_of TrackerApi::Resources::Story
+      end
+    end
+  end
+end

data/test/review_test.rb

@@ -0,0 +1,27 @@
+require_relative 'minitest_helper'
+
+describe TrackerApi::Resources::Review do
+  let(:pt_user) { PT_USER_1 }
+  let(:client) { TrackerApi::Client.new token: pt_user[:token] }
+  let(:story) do
+    TrackerApi::Resources::Story.new(
+      client: client,
+      project_id: pt_user[:project_id],
+      id: '66728004'
+    )
+  end
+  let(:reviews) { VCR.use_cassette('get story reviews') { story.reviews } }
+  let(:existing_review) { reviews.first }
+
+  it 'can update an existing review' do
+    new_state = 'pass'
+    existing_review.status = new_state
+
+    VCR.use_cassette('save review', record: :new_episodes) do
+      existing_review.save
+    end
+
+    _(existing_review.status).must_equal new_state
+    _(existing_review.clean?).must_equal true
+  end
+end

data/test/story_test.rb

@@ -22,8 +22,8 @@ describe TrackerApi::Resources::Story do
       story.save
     end
 
-    story.name.must_equal new_name
-    story.clean?.must_equal true
+    _(story.name).must_equal new_name
+    _(story.clean?).must_equal true
   end
 
   it 'can update multiple attributes of an existing story at once' do
@@ -36,14 +36,14 @@ describe TrackerApi::Resources::Story do
       story.save
     end
 
-    story.name.must_equal new_name
-    story.description.must_equal new_desc
+    _(story.name).must_equal new_name
+    _(story.description).must_equal new_desc
   end
 
   it 'can add new labels to an existing story' do
     new_label_name = "super-special-label"
 
-    story.labels.map(&:name).wont_include new_label_name
+    _(story.labels.map(&:name)).wont_include new_label_name
 
     story.add_label(new_label_name)
 
@@ -51,13 +51,13 @@ describe TrackerApi::Resources::Story do
       story.save
     end
 
-    story.labels.wont_be_empty
-    story.labels.map(&:name).must_include new_label_name
+    _(story.labels).wont_be_empty
+    _(story.labels.map(&:name)).must_include new_label_name
   end
 
   it 'can add new labels to an existing story without existing labels' do
     story = VCR.use_cassette('get story no existing labels') { project.story(story_id_no_existing_labels) }
-    story.labels.must_be_nil
+    _(story.labels).must_be_nil
 
     new_label_name = "super-special-label"
     story.add_label(new_label_name)
@@ -66,8 +66,8 @@ describe TrackerApi::Resources::Story do
       story.save
     end
 
-    story.labels.wont_be_empty
-    story.labels.map(&:name).must_include new_label_name
+    _(story.labels).wont_be_empty
+    _(story.labels.map(&:name)).must_include new_label_name
   end
 
   it 'does not remove existing labels when updating story fields' do
@@ -86,15 +86,15 @@ describe TrackerApi::Resources::Story do
       story_in_epic.save
     end
 
-    story_in_epic.labels.must_equal original_labels
-    story_in_epic.label_list.must_equal original_label_list
+    _(story_in_epic.labels).must_equal original_labels
+    _(story_in_epic.label_list).must_equal original_label_list
   end
 
   it 'does not send unmodified fields when saving' do
     story_with_one_change = TrackerApi::Resources::Story::UpdateRepresenter.new(TrackerApi::Resources::Story.new(name: "new_name"))
     expected_json = MultiJson.dump({name: "new_name"})
 
-    expected_json.must_equal story_with_one_change.to_json
+    _(expected_json).must_equal story_with_one_change.to_json
   end
 
   it 'objects are equal based on id' do
@@ -102,15 +102,15 @@ describe TrackerApi::Resources::Story do
     story_b = VCR.use_cassette('get story') { project.story(story_id) }
     story_c = VCR.use_cassette('get another story') { project.story(another_story_id) }
 
-    story_a.must_equal story_b
-    story_a.hash.must_equal story_b.hash
-    story_a.eql?(story_b).must_equal true
-    story_a.equal?(story_b).must_equal false
+    _(story_a).must_equal story_b
+    _(story_a.hash).must_equal story_b.hash
+    _(story_a.eql?(story_b)).must_equal true
+    _(story_a.equal?(story_b)).must_equal false
 
-    story_a.wont_equal story_c
-    story_a.hash.wont_equal story_c.hash
-    story_a.eql?(story_c).must_equal false
-    story_a.equal?(story_c).must_equal false
+    _(story_a).wont_equal story_c
+    _(story_a.hash).wont_equal story_c.hash
+    _(story_a.eql?(story_c)).must_equal false
+    _(story_a.equal?(story_c)).must_equal false
   end
 
   describe '.owners' do
@@ -123,9 +123,9 @@ describe TrackerApi::Resources::Story do
       # it should not be making another HTTP request.
       owners = story.owners
 
-      owners.wont_be_empty
+      _(owners).wont_be_empty
       owner = owners.first
-      owner.must_be_instance_of TrackerApi::Resources::Person
+      _(owner).must_be_instance_of TrackerApi::Resources::Person
     end
 
     it 'gets all owners for this story' do
@@ -133,9 +133,9 @@ describe TrackerApi::Resources::Story do
         story = project.story(story_id)
         owners = VCR.use_cassette('get owners for story') { story.owners }
 
-        owners.wont_be_empty
+        _(owners).wont_be_empty
         owner = owners.first
-        owner.must_be_instance_of TrackerApi::Resources::Person
+        _(owner).must_be_instance_of TrackerApi::Resources::Person
       end
     end
   end
@@ -146,8 +146,8 @@ describe TrackerApi::Resources::Story do
         story = project.story(story_id)
         owner_ids = story.owner_ids
 
-        owner_ids.wont_be_empty
-        owner_ids.first.must_be_instance_of Fixnum
+        _(owner_ids).wont_be_empty
+        _(owner_ids.first).must_be_instance_of Fixnum
       end
     end
 
@@ -160,16 +160,16 @@ describe TrackerApi::Resources::Story do
         story.owner_ids = one_owner
         VCR.use_cassette('save story with one owner') { story.save }
 
-        story.owner_ids.wont_be_empty
-        story.owner_ids.must_equal one_owner
+        _(story.owner_ids).wont_be_empty
+        _(story.owner_ids).must_equal one_owner
 
         # save with two owners
         two_owners = [pt_user_1_id, pt_user_2_id]
         story.owner_ids = two_owners
         VCR.use_cassette('save story with two owners') { story.save }
 
-        story.owner_ids.wont_be_empty
-        story.owner_ids.must_equal two_owners
+        _(story.owner_ids).wont_be_empty
+        _(story.owner_ids).must_equal two_owners
       end
     end
   end
@@ -184,8 +184,8 @@ describe TrackerApi::Resources::Story do
 
         story.add_owner(TrackerApi::Resources::Person.new(id: 123))
 
-        story.owner_ids.wont_be_empty
-        story.owner_ids.must_equal [123]
+        _(story.owner_ids).wont_be_empty
+        _(story.owner_ids).must_equal [123]
       end
     end
 
@@ -201,8 +201,8 @@ describe TrackerApi::Resources::Story do
         # test dups are not added
         story.add_owner(123)
 
-        story.owner_ids.wont_be_empty
-        story.owner_ids.must_equal [123, 456]
+        _(story.owner_ids).wont_be_empty
+        _(story.owner_ids).must_equal [123, 456]
       end
     end
   end
@@ -212,9 +212,9 @@ describe TrackerApi::Resources::Story do
       VCR.use_cassette('get story with tasks', record: :new_episodes) do
         tasks = project.story(story_id, fields: ':default,tasks').tasks
 
-        tasks.wont_be_empty
+        _(tasks).wont_be_empty
         task = tasks.first
-        task.must_be_instance_of TrackerApi::Resources::Task
+        _(task).must_be_instance_of TrackerApi::Resources::Task
       end
     end
 
@@ -223,9 +223,9 @@ describe TrackerApi::Resources::Story do
         story = project.story(story_id)
         tasks = VCR.use_cassette('get tasks for story') { story.tasks }
 
-        tasks.wont_be_empty
+        _(tasks).wont_be_empty
         task = tasks.first
-        task.must_be_instance_of TrackerApi::Resources::Task
+        _(task).must_be_instance_of TrackerApi::Resources::Task
       end
     end
 
@@ -236,7 +236,7 @@ describe TrackerApi::Resources::Story do
           tasks = story.tasks
           unless tasks.empty?
             task = tasks.first
-            task.must_be_instance_of TrackerApi::Resources::Task
+            _(task).must_be_instance_of TrackerApi::Resources::Task
           end
         end
       end
@@ -246,10 +246,10 @@ describe TrackerApi::Resources::Story do
       VCR.use_cassette('create task') do
         task = project.story(story_id).create_task(description: 'Test task')
 
-        task.must_be_instance_of TrackerApi::Resources::Task
-        task.id.wont_be_nil
-        task.id.must_be :>, 0
-        task.description.must_equal 'Test task'
+        _(task).must_be_instance_of TrackerApi::Resources::Task
+        _(task.id).wont_be_nil
+        _(task.id).must_be :>, 0
+        _(task.description).must_equal 'Test task'
       end
     end
   end
@@ -268,9 +268,9 @@ describe TrackerApi::Resources::Story do
       VCR.use_cassette('get story activity', record: :new_episodes) do
         activity = story.activity
 
-        activity.wont_be_empty
+        _(activity).wont_be_empty
         event = activity.first
-        event.must_be_instance_of TrackerApi::Resources::Activity
+        _(event).must_be_instance_of TrackerApi::Resources::Activity
       end
     end
   end
@@ -284,7 +284,7 @@ describe TrackerApi::Resources::Story do
 
         comments = story.comments
         comment = comments.first
-        comment.must_be_instance_of TrackerApi::Resources::Comment
+        _(comment).must_be_instance_of TrackerApi::Resources::Comment
       end
     end
   end
@@ -298,7 +298,24 @@ describe TrackerApi::Resources::Story do
 
         transitions = story.transitions
         transition = transitions.first
-        transition.must_be_instance_of TrackerApi::Resources::StoryTransition
+        _(transition).must_be_instance_of TrackerApi::Resources::StoryTransition
+      end
+    end
+  end
+
+  describe '.reviews' do
+    it 'gets all reviews (and review_types field by default) for the story' do
+      VCR.use_cassette('get story reviews', record: :new_episodes) do
+        story = TrackerApi::Resources::Story.new( client:     client,
+                                                  project_id: project_id,
+                                                  id:         story_id)
+
+        reviews = story.reviews
+        review = reviews.first
+        _(review).must_be_instance_of TrackerApi::Resources::Review
+        _(review.review_type).must_be_instance_of TrackerApi::Resources::ReviewType
+        _(review.review_type.name).must_equal 'Test (QA)'
+        _(review.status).must_equal 'unstarted'
       end
     end
   end