tracker_api 1.11.0 → 1.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: cce8ed0f32ae28c44915d1a2aa0dff3f7deb44f9
-  data.tar.gz: 6478d86c0e8e8e826e0ffd50b8111758f0dec9c0
+SHA256:
+  metadata.gz: 849c1d3fe22b5b812ed7144d3d0703a7a6f3701d575f5d794cf9f9dc8caf6643
+  data.tar.gz: d142dccb5a2eec0474e54262f351e4ea148651dd7c0c7a28fcb93acdbb9f3d25
 SHA512:
-  metadata.gz: 34b10827d2868cf6e0de4039ef23444b412fcb5d162e6fca2a20258f22d3ceede1e45cb4f7e829cb57ccd8a66a3506144e53eed8f7ae6954b2d2196c6244e0f1
-  data.tar.gz: 69eb1bbad1ea7a6ef35b84e36c9b856ebc300eda60369769c242a9ec77fd5cf3c436c7280101341f657d46beab1fc382e16bbb43c074bcb6b89c97b74d71105b
+  metadata.gz: 62a6cab610b4c3e4b43ff6b7ca007e477e0478e6c1e3821b2d49a1052a830d030dbf7bfcebb91158a2493cc589e8d7ca0155f3621a0e52119ff40fba560a243a
+  data.tar.gz: a030d836b14d017f57c2eaf91f12cb7b9202c23a0cabd6e2e08b425c5185f10cb477b1ec3e598c19cdf63889dcaf23ae5c3ede9dfdc28adc562aac4f9785ad17

data/.github/workflows/ruby-tests.yml

@@ -0,0 +1,43 @@
+name: Ruby Tests
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+  test:
+
+    name: Specs - Ruby ${{ matrix.ruby-version }} ${{matrix.gemfile}}
+    runs-on: ubuntu-latest
+    env:
+      CC_TEST_REPORTER_ID: 2a6849be8214739deef0090b810b945ce9a550a4d8279d242cb03242d1ad53c5
+      BUNDLE_GEMFILE: ${{ github.workspace }}/gemfiles/${{ matrix.gemfile }}.gemfile
+
+    strategy:
+      matrix:
+        gemfile:
+          - faraday-1
+          - faraday-2
+        ruby-version:
+          - "3.1"
+          - "3.0"
+          - "2.7"
+          - "2.6"
+          - jruby
+          - truffleruby
+
+    steps:
+    - uses: actions/checkout@v2
+    - name: Set up Ruby ${{ matrix.ruby-version }}
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rake test
+    - name: Upload Coverage
+      uses: paambaati/codeclimate-action@v3.0.0
+      if: matrix.ruby-version == '3.1' && matrix.gemfile == 'faraday-2'

data/Gemfile

@@ -2,6 +2,3 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in tracker_api.gemspec
 gemspec
-
-gem 'coveralls', group: :test, require: false
-

data/LICENSE.txt

@@ -1,22 +1,21 @@
-Copyright (c) 2014 Dash of Code
-
 MIT License
 
-Permission is hereby granted, free of charge, to any person obtaining
-a copy of this software and associated documentation files (the
-"Software"), to deal in the Software without restriction, including
-without limitation the rights to use, copy, modify, merge, publish,
-distribute, sublicense, and/or sell copies of the Software, and to
-permit persons to whom the Software is furnished to do so, subject to
-the following conditions:
+Copyright (c) [2021] [ProductPlan]
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
 
-The above copyright notice and this permission notice shall be
-included in all copies or substantial portions of the Software.
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
 
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
-EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
-MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
-NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
-LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
-OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
-WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

data/README.md

@@ -1,17 +1,19 @@
 # TrackerApi
 
-[![Gem Version](https://badge.fury.io/rb/tracker_api.png)](http://badge.fury.io/rb/tracker_api)
-[![Build Status](https://travis-ci.org/dashofcode/tracker_api.png?branch=master)](https://travis-ci.org/dashofcode/tracker_api)
-[![Code Climate](https://codeclimate.com/github/dashofcode/tracker_api.png)](https://codeclimate.com/github/dashofcode/tracker_api)
-[![Coverage Status](https://coveralls.io/repos/dashofcode/tracker_api/badge.png?branch=master)](https://coveralls.io/r/dashofcode/tracker_api?branch=master)
-[![Dependency Status](https://gemnasium.com/dashofcode/tracker_api.png)](https://gemnasium.com/dashofcode/tracker_api)
+[![Gem Version](https://badge.fury.io/rb/tracker_api.svg)](http://badge.fury.io/rb/tracker_api)
+[![Build Status](https://github.com/ProductPlan/tracker_api/actions/workflows/ruby-tests.yml/badge.svg?branch=master)](https://github.com/ProductPlan/tracker_api/actions)
+[![Maintainability](https://api.codeclimate.com/v1/badges/c4602f519cd748d53492/maintainability)](https://codeclimate.com/github/ProductPlan/tracker_api/maintainability)
+[![Test Coverage](https://api.codeclimate.com/v1/badges/c4602f519cd748d53492/test_coverage)](https://codeclimate.com/github/ProductPlan/tracker_api/test_coverage)
 
 This gem allows you to easily use the [Pivotal Tracker v5 API](https://www.pivotaltracker.com/help/api/rest/v5).
 
 It’s powered by [Faraday](https://github.com/lostisland/faraday) and [Virtus](https://github.com/solnic/virtus).
 
-## Demonstration
-[Dash of Agile](https://www.dashofagile.com) uses `tracker_api` to create agile dashboards from Pivotal Tracker projects.
+## Compatibility
+
+This gem is tested against the following officially supported Ruby versions:
+ - Minimum Ruby Version: 2.6.x
+ - Latest Ruby Supported: 3.1.x
 
 ## Installation
 
@@ -86,6 +88,10 @@ task = story.tasks.first                                                  # Get
 task.complete = true
 task.save                                                                 # Mark a task complete
 
+review = story.reviews.first                                              # Mark a review as complete
+review.status = 'pass'
+review.save
+
 epics = project.epics                                                     # Get all epics for a project
 epic  = epics.first
 label = epic.label                                                        # Get an epic's label

data/gemfiles/faraday-1.gemfile

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+
+gem 'faraday', '~> 1.10'
+
+gemspec path: '../'

data/gemfiles/faraday-2.gemfile

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+
+gem 'faraday', '~> 2.2'
+
+gemspec path: '../'

data/lib/tracker_api/client.rb

@@ -25,7 +25,7 @@ module TrackerApi
       @url               = Addressable::URI.parse(url).to_s
       @api_version       = options.fetch(:api_version, '/services/v5')
       @logger            = options.fetch(:logger, ::Logger.new(nil))
-      adapter            = options.fetch(:adapter) { defined?(JRUBY_VERSION) ? :net_http : :excon }
+      adapter            = options.fetch(:adapter, :net_http)
       connection_options = options.fetch(:connection_options, { ssl: { verify: true } })
       @auto_paginate     = options.fetch(:auto_paginate, true)
       @token             = options[:token]

data/lib/tracker_api/endpoints/attachment.rb

@@ -19,7 +19,9 @@ module TrackerApi
       # end
 
       def get(comment)
-        data = client.get("/projects/#{comment.project_id}/stories/#{comment.story_id}/comments/#{comment.id}?fields=file_attachments").body["file_attachments"]
+        comment_target_slug = !comment.story_id.nil? ? "stories/#{comment.story_id}" : "epics/#{comment.epic_id}"
+
+        data = client.get("/projects/#{comment.project_id}/#{comment_target_slug}/comments/#{comment.id}?fields=file_attachments").body["file_attachments"]
         raise Errors::UnexpectedData, 'Array of file attachments expected' unless data.is_a? Array
 
         data.map do |file_attachment|

data/lib/tracker_api/endpoints/comment.rb

@@ -7,16 +7,24 @@ module TrackerApi
         @client = client
       end
 
-      def create(project_id, story_id, params={})
-        data = client.post("/projects/#{project_id}/stories/#{story_id}/comments", params: params).body
+      def create(project_id, story_id: nil, epic_id: nil, params: {})
+        raise ArgumentError, 'One of story id or epic id must be provided.' if story_id.nil? && epic_id.nil?
+
+        comment_target_slug = !story_id.nil? ? "stories/#{story_id}" : "epics/#{epic_id}"
+
+        data = client.post("/projects/#{project_id}/#{comment_target_slug}/comments", params: params).body
         Resources::Comment.new({ client: client, project_id: project_id }.merge(data))
       end
 
       def update(comment, params={})
         raise ArgumentError, 'Valid comment required to update.' unless comment.instance_of?(Resources::Comment)
 
-        path = "/projects/#{comment.project_id}/stories/#{comment.story_id}/comments/#{comment.id}"
-        path += "?fields=:default,file_attachments" if params.represented.file_attachment_ids_to_add.present? || params.represented.file_attachment_ids_to_remove.present?
+        comment_target_slug = !comment.story_id.nil? ? "stories/#{comment.story_id}" : "epics/#{comment.epic_id}"
+
+        path = "/projects/#{comment.project_id}/#{comment_target_slug}/comments/#{comment.id}"
+        if params.represented.file_attachment_ids_to_add.present? || params.represented.file_attachment_ids_to_remove.present?
+          path += "?fields=:default,file_attachments"
+        end
         data = client.put(path, params: params).body
 
         comment.attributes = data
@@ -27,7 +35,9 @@ module TrackerApi
       def delete(comment)
         raise ArgumentError, 'Valid comment required to update.' unless comment.instance_of?(Resources::Comment)
 
-        client.delete("/projects/#{comment.project_id}/stories/#{comment.story_id}/comments/#{comment.id}").body
+        comment_target_slug = !comment.story_id.nil? ? "stories/#{comment.story_id}" : "epics/#{comment.epic_id}"
+
+        client.delete("/projects/#{comment.project_id}/#{comment_target_slug}/comments/#{comment.id}").body
       end
     end
   end

data/lib/tracker_api/endpoints/comments.rb

@@ -7,14 +7,19 @@ module TrackerApi
         @client = client
       end
 
-      def get(project_id, story_id, params={})
-        data = client.paginate("/projects/#{project_id}/stories/#{story_id}/comments", params: params)
+      def get(project_id, story_id: nil, epic_id: nil, params: {})
+        raise ArgumentError, 'One of story id or epic id must be provided.' if story_id.nil? && epic_id.nil?
+
+        comment_target_slug = !story_id.nil? ? "stories/#{story_id}" : "epics/#{epic_id}"
+
+        data = client.paginate("/projects/#{project_id}/#{comment_target_slug}/comments", params: params)
         raise Errors::UnexpectedData, 'Array of comments expected' unless data.is_a? Array
 
         data.map do |comment|
-          Resources::Comment.new({ client: client,
-                                   project_id: project_id,
-                                   story_id: story_id }.merge(comment))
+          Resources::Comment.new({
+            client: client,
+            project_id: project_id
+          }.merge(comment))
         end
       end
     end

data/lib/tracker_api/endpoints/review.rb

@@ -0,0 +1,21 @@
+module TrackerApi
+  module Endpoints
+    class Review
+      attr_accessor :client
+
+      def initialize(client)
+        @client = client
+      end
+
+      def update(review, params = {})
+        raise ArgumentError, 'Valid review required to update.' unless review.instance_of?(Resources::Review)
+
+        data = client.put("/projects/#{review.project_id}/stories/#{review.story_id}/reviews/#{review.id}", params: params).body
+
+        review.attributes = data
+        review.clean!
+        review
+      end
+    end
+  end
+end

data/lib/tracker_api/file_utility.rb

@@ -2,7 +2,7 @@ module TrackerApi
   class FileUtility
     class << self
       def get_file_upload(file)
-        mime_type = MimeMagic.by_path(file)
+        mime_type = MiniMime.lookup_by_filename(file)
         { :file => Faraday::UploadIO.new(file, mime_type) }
       end
 

data/lib/tracker_api/logger.rb

@@ -1,5 +1,5 @@
 module TrackerApi
-  class Logger < Faraday::Response::Middleware
+  class Logger < Faraday::Middleware
     extend Forwardable
 
     def initialize(app, logger = nil)

data/lib/tracker_api/resources/comment.rb

@@ -32,13 +32,13 @@ module TrackerApi
       end
 
       def save
-        raise ArgumentError, 'Cannot update a comment with an unknown story_id.' if story_id.nil?
+        raise ArgumentError, 'Cannot update a comment with an unknown story_id or epic_id.' if story_id.nil? && epic_id.nil?
 
         Endpoints::Comment.new(client).update(self, UpdateRepresenter.new(Comment.new(self.dirty_attributes)))
       end
 
       def delete
-        raise ArgumentError, 'Cannot delete a comment with an unknown story_id.' if story_id.nil?
+        raise ArgumentError, 'Cannot delete a comment with an unknown story_id or epic_id.' if story_id.nil? && epic_id.nil?
 
         Endpoints::Comment.new(client).delete(self)
       end

data/lib/tracker_api/resources/epic.rb

@@ -35,11 +35,20 @@ module TrackerApi
         Endpoints::Epic.new(client).update(self, UpdateRepresenter.new(self))
       end
 
+      # Provides a list of all the comments on the epic.
+      def comments(reload: false)
+        if !reload && @comments.present?
+          @comments
+        else
+          @comments = Endpoints::Comments.new(client).get(project_id, epic_id: id)
+        end
+      end
+
       # @param [Hash] params attributes to create the comment with
       # @return [Comment] newly created Comment
       def create_comment(params)
         files = params.delete(:files)
-        comment = Endpoints::Comment.new(client).create(project_id, id, params)
+        comment = Endpoints::Comment.new(client).create(project_id, epic_id: id, params: params)
         comment.create_attachments(files: files) if files.present?
         comment
       end

data/lib/tracker_api/resources/pull_request.rb

@@ -0,0 +1,22 @@
+module TrackerApi
+  module Resources
+    class PullRequest
+      include Shared::Base
+
+      attribute :client
+
+      attribute :id, Integer
+      attribute :story_id, Integer
+      attribute :epic_id, Integer
+      attribute :owner, String
+      attribute :repo, String
+      attribute :number, Integer
+      attribute :host_url, String
+      attribute :original_url, String
+      attribute :status, String
+      attribute :created_at, DateTime
+      attribute :updated_at, DateTime
+      attribute :kind, String
+    end
+  end
+end

data/lib/tracker_api/resources/review.rb

@@ -7,6 +7,7 @@ module TrackerApi
 
       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)
@@ -14,6 +15,21 @@ module TrackerApi
       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
+end

data/lib/tracker_api/resources/story.rb

@@ -30,6 +30,7 @@ module TrackerApi
       attribute :owners, [Person]
       attribute :planned_iteration_number, Integer
       attribute :project_id, Integer
+      attribute :pull_requests, [PullRequest]
       attribute :requested_by, Person
       attribute :requested_by_id, Integer
       attribute :reviews, [Review]
@@ -101,7 +102,7 @@ module TrackerApi
       #
       # @param [Person|Fixnum] owner
       def add_owner(owner)
-        owner_id = if owner.kind_of?(Fixnum)
+        owner_id = if owner.kind_of?(Integer)
           owner_id = owner
         else
           raise ArgumentError, 'Valid Person expected.' unless owner.instance_of?(Resources::Person)
@@ -132,7 +133,7 @@ module TrackerApi
         if !reload && @comments.present?
           @comments
         else
-          @comments = Endpoints::Comments.new(client).get(project_id, id)
+          @comments = Endpoints::Comments.new(client).get(project_id, story_id: id)
         end
       end
 
@@ -193,7 +194,7 @@ module TrackerApi
       # @return [Comment] newly created Comment
       def create_comment(params)
         files = params.delete(:files)
-        comment = Endpoints::Comment.new(client).create(project_id, id, params)
+        comment = Endpoints::Comment.new(client).create(project_id, story_id: id, params: params)
         comment.create_attachments(files: files) if files.present?
         comment
       end

data/lib/tracker_api/version.rb

@@ -1,3 +1,3 @@
 module TrackerApi
-  VERSION = '1.11.0'
+  VERSION = '1.14.0'
 end

data/lib/tracker_api.rb

@@ -2,9 +2,9 @@ require 'tracker_api/version'
 
 # dependencies
 require 'faraday'
-require 'faraday_middleware'
+require 'faraday/multipart'
 require 'pathname'
-require 'mimemagic'
+require 'mini_mime'
 
 if defined?(ActiveSupport)
   require 'active_support/core_ext/object/blank'
@@ -66,6 +66,7 @@ module TrackerApi
     autoload :Attachments, 'tracker_api/endpoints/attachments'
     autoload :Releases, 'tracker_api/endpoints/releases'
     autoload :Release, 'tracker_api/endpoints/release'
+    autoload :Review, 'tracker_api/endpoints/review'
     autoload :Reviews, 'tracker_api/endpoints/reviews'
   end
 
@@ -89,6 +90,7 @@ module TrackerApi
     autoload :PrimaryResource, 'tracker_api/resources/primary_resource'
     autoload :Project, 'tracker_api/resources/project'
     autoload :ProjectMembership, 'tracker_api/resources/project_membership'
+    autoload :PullRequest, 'tracker_api/resources/pull_request'
     autoload :Workspace, 'tracker_api/resources/workspace'
     autoload :SearchResultContainer, 'tracker_api/resources/search_result_container'
     autoload :StoriesSearchResult, 'tracker_api/resources/stories_search_result'

data/test/comment_test.rb

@@ -7,13 +7,17 @@ describe TrackerApi::Resources::Comment 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) } }
-  let(:comments) { VCR.use_cassette('get comments') { story.comments } }
-  let(:existing_comment) { comments.first }
+  let(:story_comments) { VCR.use_cassette('get story comments') { story.comments } }
+  let(:epic_id) { '4737194' }
+  let(:epic) { VCR.use_cassette('get epic') { project.epic(epic_id) } }
+  let(:epic_comments) { VCR.use_cassette('get epic comments') { epic.comments } }
+  let(:existing_story_comment) { story_comments.first }
+  let(:existing_epic_comment) { epic_comments.first }
 
   it 'can create a comment given a story' do
     text = "Test creating a comment"
     comment = nil
-    VCR.use_cassette('create comment', record: :new_episodes) do
+    VCR.use_cassette('create story comment', record: :new_episodes) do
       comment = story.create_comment(text: text)
     end
 
@@ -21,11 +25,22 @@ describe TrackerApi::Resources::Comment do
     _(comment.clean?).must_equal true
   end
 
-  it 'can create a comment with file attachment' do
+  it 'can create a comment given an epic' do
+    text = "Test creating a comment"
+    comment = nil
+    VCR.use_cassette('create epic comment', record: :new_episodes) do
+      comment = epic.create_comment(text: text)
+    end
+
+    _(comment.text).must_equal text
+    _(comment.clean?).must_equal true
+  end
+
+  it 'can create a story comment with file attachment' do
     text = "Test creating a comment"
     comment = nil
     files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
-    VCR.use_cassette('create comment with attachment', record: :new_episodes) do
+    VCR.use_cassette('create story comment with attachment', record: :new_episodes) do
       comment = story.create_comment(text: text, files: files)
     end
     _(comment.text).must_equal text
@@ -33,39 +48,82 @@ describe TrackerApi::Resources::Comment do
     _(comment.clean?).must_equal true
   end
 
-  it 'can update an existing comment' do
-    new_text = "#{existing_comment.text}+"
-    existing_comment.text = new_text
+  it 'can create an epic comment with file attachment' do
+    text = "Test creating a comment"
+    comment = nil
+    files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
+    VCR.use_cassette('create epic comment with attachment', record: :new_episodes) do
+      comment = epic.create_comment(text: text, files: files)
+    end
+    _(comment.text).must_equal text
+    _(comment.attachments.size).must_equal 1
+    _(comment.clean?).must_equal true
+  end
+
+  it 'can update an existing story comment' do
+    new_text = "#{existing_story_comment.text}+"
+    existing_story_comment.text = new_text
 
-    VCR.use_cassette('save comment', record: :new_episodes) do
-      existing_comment.save
+    VCR.use_cassette('save story comment', record: :new_episodes) do
+      existing_story_comment.save
     end
 
-    _(existing_comment.text).must_equal new_text
-    _(existing_comment.clean?).must_equal true
+    _(existing_story_comment.text).must_equal new_text
+    _(existing_story_comment.clean?).must_equal true
   end
 
-  it 'can create attachments in a comment' do
+  it 'can update an existing epic comment' do
+    new_text = "#{existing_epic_comment.text}+"
+    existing_epic_comment.text = new_text
+
+    VCR.use_cassette('save epic comment', record: :new_episodes) do
+      existing_epic_comment.save
+    end
+
+    _(existing_epic_comment.text).must_equal new_text
+    _(existing_epic_comment.clean?).must_equal true
+  end
+
+  it 'can create attachments in a story comment' do
+    files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
+    VCR.use_cassette('create story attachments', record: :new_episodes) do
+      existing_story_comment.create_attachments(files: files)
+      assert existing_story_comment.attachments.size > 0
+      _(existing_story_comment.clean?).must_equal true
+    end
+  end
+
+  it 'can create attachments in an epic comment' do
+    files = [File.expand_path('../Gemfile', File.dirname(__FILE__))]
+    VCR.use_cassette('create epic attachments', record: :new_episodes) do
+      existing_epic_comment.create_attachments(files: files)
+      assert existing_epic_comment.attachments.size > 0
+      _(existing_epic_comment.clean?).must_equal true
+    end
+  end
+
+  it 'can delete attachments in a story 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
+    VCR.use_cassette('delete story attachments', record: :new_episodes) do
+      existing_story_comment.create_attachments(files: files)
+      assert existing_story_comment.attachments.size > 0
+      existing_story_comment.delete_attachments
+      _(existing_story_comment.attachments.size).must_equal 0
     end
   end
 
-  it 'can delete attachments in a comment' do
+  it 'can delete attachments in an epic 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.delete_attachments
-      _(existing_comment.attachments.size).must_equal 0
+    VCR.use_cassette('delete epic attachments', record: :new_episodes) do
+      existing_epic_comment.create_attachments(files: files)
+      assert existing_epic_comment.attachments.size > 0
+      existing_epic_comment.delete_attachments
+      _(existing_epic_comment.attachments.size).must_equal 0
     end
   end
 
-  it 'can delete a comment' do
-    VCR.use_cassette('delete comment', record: :new_episodes) do
+  it 'can delete a story comment' do
+    VCR.use_cassette('delete story 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
@@ -74,4 +132,15 @@ describe TrackerApi::Resources::Comment do
       _(current_story.comments.last.id).wont_equal new_comment_id
     end
   end
+
+  it 'can delete an epic comment' do
+    VCR.use_cassette('delete epic comment', record: :new_episodes) do
+      current_epic = project.epic(epic_id)
+      new_comment_id = current_epic.create_comment(text: "test comment").id
+      _(current_epic.comments.last.id).must_equal new_comment_id
+      current_epic.comments.last.delete
+      current_epic = project.epic(epic_id)
+      _(current_epic.comments.last.id).wont_equal new_comment_id
+    end
+  end
 end

data/test/iteration_test.rb

@@ -12,9 +12,9 @@ describe TrackerApi::Resources::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_details).wont_be_empty
         cycle_time_detail = cycle_time_details.first
-        cycle_time_detail.must_be_instance_of TrackerApi::Resources::CycleTimeDetails
+        _(cycle_time_detail).must_be_instance_of TrackerApi::Resources::CycleTimeDetails
       end
     end
   end
@@ -24,7 +24,7 @@ describe TrackerApi::Resources::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
+        _(daily_history_container).must_be_instance_of TrackerApi::Resources::DailyHistoryContainer
       end
     end
   end