my_todo 3.0.0 → 3.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 093228595b114cced5a32a36ad34238f403ed5dc
-  data.tar.gz: 6f1c939f68f4f6249235f6520a42b038420431d6
+  metadata.gz: 70b1f135ecffdf6707b18aa7a916493221bce327
+  data.tar.gz: e8fcb736c11ac62879afdc96b33f2f98a23f3875
 SHA512:
-  metadata.gz: cae431c8ee769fece3442cb705bc60871a6067306ee9aa48ffd323325995fa4dbb9c8f80556cbb6cfaaf81dcbfb27088dd4a3f31a182a96d2d3b07e0ccef3337
-  data.tar.gz: fed9ceeb3af9c1043d389b878fa1a26b7174ecc88d2e2acc7713ee726cc6d6e23f93f48e6b7666ca3e11770f043e4e84ee45908f65eb67375f64222605e9e909
+  metadata.gz: 01ec4a1b26546905b02194a39d4ccd84f9b74defdb49046183ea9ef06cc129f9a7100148ab105d72744f05bc79b9256ae1a8a8fe1c320a9f030ce3cb7504ec68
+  data.tar.gz: a6887b4de5c89b24e7e56ce2910941f9824658f7f27939081f3f03412a72271510f2f712cd636087daee721c8fdd4332c073e1495607a27d719f14fdc2fd0055

data/README.md

@@ -24,7 +24,7 @@ gem install my_todo
 ```
 
 ## setup
-Create and migrate the DB
+Create and migrate the DB - This always needs to be run after every deployment to set the proper paths at the very least.
 
 `my_todo rake db:migrate`
 
@@ -43,10 +43,10 @@ will display
 ToDo CREATED!
 
 
-ID: 4
-ToDo: hello world
-Tags: default
-Complete: false
+ID: 1 | Created On: 2016-10-04 | Tags: default | Status: In Progress | Complete: false | Notes: 0
+
+hello world
+****************************************************************************************************
 ```
 
 Example of listing pending todos
@@ -58,13 +58,17 @@ my_todo list
 will display
 
 ```
-ToDos FOUND: 1
+ToDos FOUND: 2
+
+ID: 1 | Created On: 2016-10-04 | Tags: default | Status: In Progress  | Complete: false | Notes: 0
+
+hello world
+****************************************************************************************************
 
+ID: 2 | Created On: 2016-10-05 | Tags: default | Status: In Progress | Complete: false | Notes: 0
 
-ID: 3
-ToDo: hello world
-Tags: default
-Complete: false
+hello world 2
+****************************************************************************************************
 ```
 
 Use aliasing to shorten the syntax:
@@ -77,9 +81,10 @@ alias mupdate='my_todo update'
 alias mdelete='my_todo delete'
 alias mtag='my_todo tag'
 alias mrmtag='my_todo rm_tag'
-alias mnote='my_todo note'
+alias mnotes='my_todo notes'
 alias mrmnote='my_todo rm_note'
 alias msearch='my_todo search'
+alias mnote='my_todo note'
 ```
 
 Functions can be created around these actions to possibly shorten the syntax that much more

data/lib/db/migrate/20161220143613_add_default_to_item_done.rb

@@ -0,0 +1,5 @@
+class AddDefaultToItemDone < ActiveRecord::Migration[5.0]
+  def change
+    change_column :items, :done, :boolean, default: false
+  end
+end

data/lib/db/schema.rb

@@ -10,16 +10,24 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20161005133023) do
+ActiveRecord::Schema.define(version: 20161220143613) do
 
   create_table "items", force: :cascade do |t|
     t.string   "body"
-    t.boolean  "done"
-    t.datetime "created_at",      null: false
-    t.datetime "updated_at",      null: false
+    t.boolean  "done",            default: false
+    t.datetime "created_at",                      null: false
+    t.datetime "updated_at",                      null: false
+    t.string   "status"
     t.string   "detailed_status"
   end
 
+  create_table "lists", force: :cascade do |t|
+    t.string   "name"
+    t.string   "value"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+  end
+
   create_table "notes", force: :cascade do |t|
     t.integer  "item_id"
     t.text     "body"

data/lib/my_todo/models/item.rb

@@ -1,18 +1,30 @@
 # @author Lovell McIlwain#
 # Handles business logic for todo item
 class Item < ActiveRecord::Base
-  DETAILED_STATUSES = ['None', 'In Progress', 'Complete', 'Punted', 'Waiting Feedback']
+  INCOMPLETE_STATUSES = ['None', 'In Progress', 'Waiting Feedback']
+  COMPLETE_STATUSES = ['Complete', 'Punted']
+  DETAILED_STATUSES = INCOMPLETE_STATUSES + COMPLETE_STATUSES
   # ActiveRecord association to stubs
   # @note Item.first.stubs
   has_many :stubs
+
   # ActiveRecord association to tags
   # @note Item.first.tags
   # @note destroys associated stubs/tags when deleted
   has_many :tags, through: :stubs, dependent: :destroy
+
   # ActiveRecord association to notes
   # @note Item.first.notes
   # @note destroys associated notes when deleted
   has_many :notes, dependent: :destroy
+
   # ActiveModel validation to ensure body is present
   validates :body, presence: true
+
+  before_save :update_done
+
+  #set done attribute based on detailed_status set
+  def update_done
+    self.done = true if COMPLETE_STATUSES.include? self.detailed_status
+  end
 end

data/lib/my_todo/models/note.rb

@@ -1,6 +1,28 @@
 # @author Lovell McIlwain#
 # Handles business logic for todo item
 class Note < ActiveRecord::Base
+  # ActiveModel validation to ensure body has content
   validates :body, presence: true
+
+  # ActiveRecord association to parent item
+  # @note Note.first.item
   belongs_to :item
+
+  # Add date to created at when record is created
+  # (see #tag_created_at)
+  before_save :tag_created_at, on: :create
+
+  # Add date to updated at when record is updated
+  # (see #tag_updated_at)
+  before_save :tag_updated_at, on: :update
+
+  # set created at to the current date
+  def tag_created_at
+    self.created_at = Date.today
+  end
+
+  # set updated at to the current date
+  def tag_updated_at
+    self.updated_at = Date.today if changes.size > 0
+  end
 end

data/lib/my_todo/modules/my_todo_actions.rb

@@ -7,7 +7,7 @@ module MyTodoActions
   def create_item(options)
     ask_status
     @item = Item.create!(options.merge({detailed_status: detailed_statuses[@status.to_i]}).except(:tags))
-    options[:tags].split(' ').each{|tag| item.tags.create(name: tag) }
+    options[:tags].split(' ').each{|tag| item.tags.create(name: tag) } if options[:tags]
   end
 
   def update_item(options)

data/lib/my_todo/modules/templates.rb

@@ -1,5 +1,4 @@
 module Templates
-  p __dir__
   def print_list
     say ERB.new(File.read("#{__dir__}/../templates/list.erb"), nil, '-').result(binding)
   end

data/lib/my_todo/templates/list.erb

@@ -1,6 +1,6 @@
 
 <% @items.each do |item| -%>
-ID: <%= item.id %> | Created On: <%= item.created_at.strftime("%Y-%m-%d") %> | Tags: <%= item.tags.map(&:name).join(', ') %> | Status: <%= item.detailed_status %> | Complete: <%= item.done %>
+ID: <%= item.id %> | Created On: <%= item.created_at.strftime("%Y-%m-%d") %> | Tags: <%= item.tags.map(&:name).join(', ') %> | Status: <%= item.detailed_status %> | Complete: <%= item.done %> | Notes: <%= item.notes.size %>
 
 <%= item.body %>
 <%= '*' * 100 %>

data/lib/my_todo/templates/results.erb

@@ -1,6 +1,6 @@
 
 <% @items.each do |item| -%>
-ID: <%= item.id %> | Created On: <%= item.created_at.strftime("%Y-%m-%d") %> | Tags: <%= item.tags.map(&:name).join(', ') %> | Status: <%= item.detailed_status %> | Complete: <%= item.done %>
+ID: <%= item.id %> | Created On: <%= item.created_at.strftime("%Y-%m-%d") %> | Tags: <%= item.tags.map(&:name).join(', ') %> | Status: <%= item.detailed_status %> | Complete: <%= item.done %> | Notes: <%= item.notes.count %>
 <%= item.body %>
 
 <% if item.notes.any? -%>

data/lib/my_todo/version.rb

@@ -1,3 +1,3 @@
 module MyTodo
-  VERSION = "3.0.0"
+  VERSION = "3.1.0"
 end

data/lib/my_todo.rb

@@ -38,9 +38,7 @@ module MyTodo
 
     desc "create --body='some text' [--done=true] [--tags='tag1 tag2']", 'Create a todo'
     option :body
-    option :done, default: false
     option :tags, default: 'default'
-    option :created_at, default: DateTime.now
     def create
       begin
         say 'ToDo CREATED!'
@@ -54,8 +52,6 @@ module MyTodo
     desc "update --id=TODO_ID --body='some text' [--done=true]", 'Change an existing todo'
     option :id
     option :body
-    option :done
-    option :updated_at, default: DateTime.now
     def update
       begin
         update_item(options)
@@ -110,10 +106,10 @@ module MyTodo
       end
     end
 
-    desc "add_note --id=TODO_ID --body='text'", 'Adds note to existing item'
+    desc "note --id=TODO_ID --body='text'", 'Adds note to existing item'
     option :id
     option :body
-    def add_note
+    def note
       begin
         item.notes.create(body: options[:body])
         print_notes

data/spec/actions/create_spec.rb

@@ -33,7 +33,7 @@ describe MyTodo do
         end
 
         it 'displays the created todo item' do
-          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nID: 1 | Created On: #{Date.today} | Tags: default | Status: In Progress | Complete: false\nwierdness_of_text\n").to_stdout
+          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nID: 1 | Created On: #{Date.today} | Tags: default | Status: In Progress | Complete: false\nwierdness_of_text\n").to_stdout
         end
       end
 
@@ -43,27 +43,20 @@ describe MyTodo do
         end
 
         it 'displays the created todo item with tag' do
-          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text --tags=tag1))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status: In Progress | Complete: false\nwierdness_of_text\n").to_stdout
+          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text --tags=tag1))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status: In Progress | Complete: false\nwierdness_of_text\n").to_stdout
         end
       end
 
-      describe 'create with tags and setting done attribute' do
-
-        it 'creates todo item with flag set to done' do
-          MyTodo::Todo.start(%w[create --body=wierdness_of_text --done=true])
-          todo = Item.last
-          expect(todo.done).to eq true
-        end
-
+      describe 'create with tags' do
         it 'displays the created to item with complete set to true' do
-          expect{MyTodo::Todo.start(%w[create --body=wierdness_of_text --done=true])}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nID: 1 | Created On: #{Date.today} | Tags: default | Status: In Progress | Complete: true\nwierdness_of_text\n").to_stdout
+          expect{MyTodo::Todo.start(%w[create --body=wierdness_of_text])}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nID: 1 | Created On: #{Date.today} | Tags: default | Status: In Progress | Complete: false\nwierdness_of_text\n").to_stdout
         end
       end
     end
 
     context 'unsuccessful creation' do
       it 'returns error message when body is missing' do
-        expect{MyTodo::Todo.start(%w(create))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nValidation failed: Body can't be blank\n").to_stdout
+        expect{MyTodo::Todo.start(%w(create))}.to output("ToDo CREATED!\n0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nValidation failed: Body can't be blank\n").to_stdout
       end
     end
   end

data/spec/actions/list_spec.rb

@@ -4,19 +4,19 @@ describe MyTodo do
   describe 'list' do
     before do
       @todo1 = FactoryGirl.create :item, done: true
-      @todo2 = FactoryGirl.create :item, done: false
+      @todo2 = FactoryGirl.create :item
     end
 
     it 'displays items with done set to true' do
-      expect{MyTodo::Todo.start(%w[list --status=done])}.to output("ToDos FOUND: 1\n\nID: 1 | Created On: #{Date.today} | Tags:  | Status:  | Complete: true\n\nSome Body\n****************************************************************************************************\n").to_stdout
+      expect{MyTodo::Todo.start(%w[list --status=done])}.to output("ToDos FOUND: 1\n\nID: 1 | Created On: #{Date.today} | Tags:  | Status:  | Complete: true | Notes: 0\n\nSome Body\n****************************************************************************************************\n").to_stdout
     end
 
     it 'displays items with done set to false by default' do
-      expect{MyTodo::Todo.start(%w[list])}.to output("ToDos FOUND: 1\n\nID: 2 | Created On: #{Date.today} | Tags:  | Status:  | Complete: false\n\nSome Body\n****************************************************************************************************\n").to_stdout
+      expect{MyTodo::Todo.start(%w[list])}.to output("ToDos FOUND: 1\n\nID: 2 | Created On: #{Date.today} | Tags:  | Status:  | Complete: false | Notes: 0\n\nSome Body\n****************************************************************************************************\n").to_stdout
     end
 
     it 'displays all items' do
-      expect{MyTodo::Todo.start(%w[list --status=all])}.to output("ToDos FOUND: 2\n\nID: 1 | Created On: #{Date.today} | Tags:  | Status:  | Complete: true\n\nSome Body\n****************************************************************************************************\nID: 2 | Created On: #{Date.today} | Tags:  | Status:  | Complete: false\n\nSome Body\n****************************************************************************************************\n").to_stdout
+      expect{MyTodo::Todo.start(%w[list --status=all])}.to output("ToDos FOUND: 2\n\nID: 1 | Created On: #{Date.today} | Tags:  | Status:  | Complete: true | Notes: 0\n\nSome Body\n****************************************************************************************************\nID: 2 | Created On: #{Date.today} | Tags:  | Status:  | Complete: false | Notes: 0\n\nSome Body\n****************************************************************************************************\n").to_stdout
     end
   end
 end

data/spec/actions/note_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe MyTodo do
+  describe 'add_note' do
+    before {@todo = FactoryGirl.create(:item)}
+    context 'successful creation' do
+      it 'creates a note' do
+        expect{MyTodo::Todo.start(%W[note --id=#{@todo.id} --body=note_text])}.to change{Note.count}.by(1)
+      end
+
+      it 'associates note to item' do
+        MyTodo::Todo.start(%W[note --id=#{@todo.id} --body=note_text])
+        expect(@todo.notes.size).to eq 1
+      end
+
+      it 'displays item with notes' do
+        expect{MyTodo::Todo.start(%W[note --id=#{@todo.id} --body=note_text])}.to output("Notes for 1: Some Body\nID: 1 | Created On: #{Date.today}\nnote_text\n\n\n").to_stdout
+      end
+    end
+
+    context 'unsuccessful creation' do
+      it 'returns exception if partent item is not found' do
+        expect{MyTodo::Todo.start(%W[note --id=#{@todo.id + 1}])}.to output("undefined method `notes' for nil:NilClass\n").to_stdout
+      end
+    end
+  end
+end

data/spec/actions/notes_spec.rb

@@ -8,7 +8,7 @@ describe MyTodo do
       before {@todo.notes.create(body: 'What a world!')}
 
       it 'displays note' do
-        expect{MyTodo::Todo.start(%W[notes --id=#{@todo.id}])}.to output("Notes for 1: Some Body\nID: 1 | Created On: 2016-12-12\nWhat a world!\n\n\n").to_stdout
+        expect{MyTodo::Todo.start(%W[notes --id=#{@todo.id}])}.to output("Notes for 1: Some Body\nID: 1 | Created On: #{Date.today}\nWhat a world!\n\n\n").to_stdout
       end
     end
 
@@ -19,8 +19,8 @@ describe MyTodo do
     end
 
     context 'unsuccessful creation' do
-      it 'returns exception if partent item is not found' do
-        expect{MyTodo::Todo.start(%W[add_note --id=#{@todo.id + 1}])}.to output("undefined method `notes' for nil:NilClass\n").to_stdout
+      it 'returns exception if parent item is not found' do
+        expect{MyTodo::Todo.start(%W[note --id=#{@todo.id + 1}])}.to output("undefined method `notes' for nil:NilClass\n").to_stdout
       end
     end
   end

data/spec/actions/search_spec.rb

@@ -12,20 +12,20 @@ describe MyTodo do
       end
 
       it 'finds todo item by body' do
-        expect{MyTodo::Todo.start( %w[search --text=nfl])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status:  | Complete: \nnfl\n\n****************************************************************************************************\n").to_stdout
+        expect{MyTodo::Todo.start( %w[search --text=nfl])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status:  | Complete: false | Notes: 0\nnfl\n\n****************************************************************************************************\n").to_stdout
       end
 
       it 'finds todo items by associated tag' do
-        expect{MyTodo::Todo.start( %w[search --text=tag1])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status:  | Complete: \nnfl\n\n****************************************************************************************************\n").to_stdout
+        expect{MyTodo::Todo.start( %w[search --text=tag1])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 1 | Created On: #{Date.today} | Tags: tag1 | Status:  | Complete: false | Notes: 0\nnfl\n\n****************************************************************************************************\n").to_stdout
       end
 
       it 'finds todo items by detailed status' do
         @todo2.update!(detailed_status: 'None')
-        expect{MyTodo::Todo.start( %w[search --text=None])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 2 | Created On: #{Date.today} | Tags:  | Status: None | Complete: \nrocks\n\n****************************************************************************************************\n").to_stdout
+        expect{MyTodo::Todo.start( %w[search --text=None])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 2 | Created On: #{Date.today} | Tags:  | Status: None | Complete: false | Notes: 0\nrocks\n\n****************************************************************************************************\n").to_stdout
       end
 
       it 'finds todo items by notes' do
-        expect{MyTodo::Todo.start( %w[search --text=note1])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 3 | Created On: 2016-12-12 | Tags:  | Status:  | Complete: \nalways\n\n  [1] note1\n  ----------------------------------------------------------------------------------------------------\n****************************************************************************************************\n").to_stdout
+        expect{MyTodo::Todo.start( %w[search --text=note1])}.to output("ToDos FOUND: 1\nSearch based on ransack search: body_or_detailed_status_or_tags_name_or_notes_body_cont\n\nID: 3 | Created On: #{Date.today} | Tags:  | Status:  | Complete: false | Notes: 1\nalways\n\n  [1] note1\n  ----------------------------------------------------------------------------------------------------\n****************************************************************************************************\n").to_stdout
       end
     end
 

data/spec/actions/update_spec.rb

@@ -9,13 +9,13 @@ describe MyTodo do
 
     context 'successful update' do
       it 'displays the update' do
-        expect{MyTodo::Todo.start(%W[update --id=#{@todo.id} --body=hello])}.to output("0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nToDo UPDATED!\nID: 1 | Created On: #{Date.today} | Tags:  | Status: None | Complete: \nhello\n").to_stdout
+        expect{MyTodo::Todo.start(%W[update --id=#{@todo.id} --body=hello])}.to output("0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nToDo UPDATED!\nID: 1 | Created On: #{Date.today} | Tags:  | Status: None | Complete: false\nhello\n").to_stdout
       end
     end
 
     context 'unsuccessful update' do
       it 'returns validation error when body is missing' do
-        expect{MyTodo::Todo.start(%W(update --id=#{@todo.id} --body=))}.to output("0: None\n1: In Progress\n2: Complete\n3: Punted\n4: Waiting Feedback\nValidation failed: Body can't be blank\n").to_stdout
+        expect{MyTodo::Todo.start(%W(update --id=#{@todo.id} --body=))}.to output("0: None\n1: In Progress\n2: Waiting Feedback\n3: Complete\n4: Punted\nValidation failed: Body can't be blank\n").to_stdout
       end
     end
   end

data/spec/models/item_spec.rb

@@ -1,8 +1,20 @@
 require 'spec_helper'
 
 describe Item, type: :model do
+  let(:item) {FactoryGirl.create :item}
+
   it {should have_many :stubs}
   it {should have_many :tags}
   it {should have_many :notes}
   it {should validate_presence_of :body}
+
+  describe 'update_done' do
+    it 'sets to true when detailed status is of a complete status' do
+      item.update(detailed_status: 'Complete')
+      expect(item.reload.done).to eq true
+    end
+    it 'sets to false when detailed status is of an incomplete status' do
+      expect(item.done).to eq false
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -1,7 +1,7 @@
 # code climent must be loaded and started before RAILS_ENV is declared
 require "codeclimate-test-reporter"
+ENV['CODECLIMATE_REPO_TOKEN'] = ENV['MYTODO_CC']
 CodeClimate::TestReporter.start
-ENV['CODECLIMATE_REPO_TOKEN'] = ENV['MYTOD_CC']
 ENV["RAILS_ENV"] = 'test'
 $LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
 require 'database_cleaner'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: my_todo
 version: !ruby/object:Gem::Version
-  version: 3.0.0
+  version: 3.1.0
 platform: ruby
 authors:
 - Vell
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-12 00:00:00.000000000 Z
+date: 2016-12-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -258,6 +258,7 @@ files:
 - lib/db/migrate/20160913134610_create_stubs.rb
 - lib/db/migrate/20161003121448_create_notes.rb
 - lib/db/migrate/20161005133023_add_detailed_status_to_items.rb
+- lib/db/migrate/20161220143613_add_default_to_item_done.rb
 - lib/db/schema.rb
 - lib/db/todos_development.sqlite3
 - lib/db/todos_test.sqlite3
@@ -281,10 +282,10 @@ files:
 - lib/my_todo/version.rb
 - lib/setup.rb
 - my_todo.gemspec
-- spec/actions/add_note_spec.rb
 - spec/actions/create_spec.rb
 - spec/actions/delete_spec.rb
 - spec/actions/list_spec.rb
+- spec/actions/note_spec.rb
 - spec/actions/notes_spec.rb
 - spec/actions/rm_note_spec.rb
 - spec/actions/rm_tag_spec.rb
@@ -323,10 +324,10 @@ signing_key:
 specification_version: 4
 summary: A basic todo application with tags.
 test_files:
-- spec/actions/add_note_spec.rb
 - spec/actions/create_spec.rb
 - spec/actions/delete_spec.rb
 - spec/actions/list_spec.rb
+- spec/actions/note_spec.rb
 - spec/actions/notes_spec.rb
 - spec/actions/rm_note_spec.rb
 - spec/actions/rm_tag_spec.rb

data/spec/actions/add_note_spec.rb

@@ -1,27 +0,0 @@
-require 'spec_helper'
-
-describe MyTodo do
-  describe 'add_note' do
-    before {@todo = FactoryGirl.create(:item)}
-    context 'successful creation' do
-      it 'creates a note' do
-        expect{MyTodo::Todo.start(%W[add_note --id=#{@todo.id} --body=note_text])}.to change{Note.count}.by(1)
-      end
-
-      it 'associates note to item' do
-        MyTodo::Todo.start(%W[add_note --id=#{@todo.id} --body=note_text])
-        expect(@todo.notes.size).to eq 1
-      end
-
-      it 'displays item with notes' do
-        expect{MyTodo::Todo.start(%W[add_note --id=#{@todo.id} --body=note_text])}.to output("Notes for 1: Some Body\nID: 1 | Created On: 2016-12-12\nnote_text\n\n\n").to_stdout
-      end
-    end
-
-    context 'unsuccessful creation' do
-      it 'returns exception if partent item is not found' do
-        expect{MyTodo::Todo.start(%W[add_note --id=#{@todo.id + 1}])}.to output("undefined method `notes' for nil:NilClass\n").to_stdout
-      end
-    end
-  end
-end