my_todo 2.0.0 → 2.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 76a111465e17fce1d2f964cb77123c2f2497102d
-  data.tar.gz: 9e3e5bdba8b59aaf441e0964c89de22edffc5f38
+  metadata.gz: 7d43fa975defc5843e115f0d8258086d03bebf62
+  data.tar.gz: 8f8d18f7d47fcceade42a7d8bd22fcfce99e81f5
 SHA512:
-  metadata.gz: 8c47ce3d3f0d87c5930de8aae7e32996dbe2d0b2f0002001041b255dbbeed46e04046e87cefb16414895b8cf1eb0275c5a9a05962bb86fb034376c8c579c407f
-  data.tar.gz: ff1f261f50d0bd60f0d8c20d536cb2d0bc48a542c7c24a3b1220e2972440733350a7cafd7d8369536ae3d6ffa9fd4573c5ee0d2e3d4cb97f3b3c716fe961c6f9
+  metadata.gz: ed29e73bb98c43586922c64f549607d8fcff148a708cb5a91c3e5e74e6b7d5fa442d8300138b65ad4d598c1e22684983ea6ce851d21d48c2a6d62c5e10b14347
+  data.tar.gz: 5258f53446840229714aec2d8d394753e2587353f56e324978977373d8193efd822fea920d2a68b72b1a75e8baf464a8411c83bab81521e294a9867806ee4ca4

data/lib/db/migrate/20161003121448_create_notes.rb

@@ -0,0 +1,9 @@
+class CreateNotes < ActiveRecord::Migration[5.0]
+  def change
+    create_table :notes do |t|
+      t.belongs_to :item
+      t.text :body
+      t.timestamps null: false
+    end
+  end
+end

data/lib/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160913134610) do
+ActiveRecord::Schema.define(version: 20161003121448) do
 
   create_table "items", force: :cascade do |t|
     t.string   "body"
@@ -19,6 +19,14 @@ ActiveRecord::Schema.define(version: 20160913134610) do
     t.datetime "updated_at", null: false
   end
 
+  create_table "notes", force: :cascade do |t|
+    t.integer  "item_id"
+    t.text     "body"
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["item_id"], name: "index_notes_on_item_id"
+  end
+
   create_table "stubs", force: :cascade do |t|
     t.integer "item_id"
     t.integer "tag_id"

data/lib/item.rb

@@ -8,6 +8,10 @@ class Item < ActiveRecord::Base
   # @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
 end

data/lib/my_todo/templates/output.erb

@@ -3,3 +3,9 @@ ID: <%= item.id %>
 ToDo: <%=item.body %>
 Tags: <%=item.tags.map(&:name).join(', ') %>
 Complete: <%=item.done %>
+<% if item.notes.any? %>
+Notes:
+<% item.notes.each do |note| %>
+<%= note.id%>: <%= note.body %>
+<% end %>
+<% end %>

data/lib/my_todo/version.rb

@@ -1,3 +1,3 @@
 module MyTodo
-  VERSION = "2.0.0"
+  VERSION = "2.1.0"
 end

data/lib/my_todo.rb

@@ -11,6 +11,7 @@ require_relative 'ar_base'
 require_relative 'item'
 require_relative 'stub'
 require_relative 'tag'
+require_relative 'note'
 
 module MyTodo
   # Todo tasks using thor gem
@@ -58,7 +59,7 @@ module MyTodo
       end
     end
 
-    desc "update --id=1 --body='some text' [--done=true]", 'update an existing todo'
+    desc "update --id=TODO_ID --body='some text' [--done=true]", 'update an existing todo'
     option :id
     option :body
     option :done
@@ -74,7 +75,7 @@ module MyTodo
       end
     end
 
-    desc 'delete(ID)', 'destroy a todo'
+    desc 'delete(TODO_ID)', 'destroy a todo'
     def delete(id)
       begin
         item = Item.find_by_id(id)
@@ -93,10 +94,10 @@ module MyTodo
       items.each {|i| output i}
     end
 
-    desc "add_tag --todo-id=1 --tag=tag1", 'add a tag to an existing todo'
+    desc "tag --id=TODO_ID --tag=TAG_NAME", 'add a tag to an existing todo'
     option :id
     option :tag
-    def add_tag
+    def tag
       begin
         item = Item.where(id: options[:id]).first
         item.tags.create!(name: options[:tag])
@@ -105,7 +106,7 @@ module MyTodo
       end
     end
 
-    desc 'rm_tag --todo-id=1 --tag=tag1', 'remove tag from an existing todo'
+    desc 'rm_tag --id=TODO_ID --tag=TAG_NAME', 'remove tag from an existing todo'
     option :id
     option :tag
     def rm_tag
@@ -117,6 +118,33 @@ module MyTodo
         say e.message
       end
     end
+
+    desc "note --id=TODO_ID --body='text'", 'adds note to existing item'
+    option :id
+    option :body
+    def note
+      begin
+        item = Item.where(id: options[:id]).first
+        item.notes.create(body: options[:body])
+        output item.reload
+      rescue Exception => e
+        say e.message
+      end
+    end
+
+    desc 'rm_note --id=TODO_ID --noteid=NOTE_ID', 'remove note for exsiting item'
+    option :id
+    option :noteid
+    def rm_note
+      begin
+        item = Item.where(id: options[:id]).first
+        item.notes.where(id: options[:noteid]).first.destroy!
+        output item.reload
+      rescue Exception => e
+        say e.message
+      end
+    end
   end
 end
+
 MyTodo::Todo.start(ARGV)

data/lib/note.rb

@@ -0,0 +1,6 @@
+# @author Lovell McIlwain#
+# Handles business logic for todo item
+class Note < ActiveRecord::Base
+  validates :body, presence: true
+  belongs_to :item
+end

data/spec/actions/create_spec.rb

@@ -21,7 +21,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!\n\nID: 1\nToDo: wierdness_of_text\nTags: default\nComplete: false\n").to_stdout
+          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text))}.to output("ToDo CREATED!\n\nID: 1\nToDo: wierdness_of_text\nTags: default\nComplete: false\n\n").to_stdout
         end
       end
 
@@ -31,7 +31,7 @@ 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!\n\nID: 1\nToDo: wierdness_of_text\nTags: tag1\nComplete: false\n").to_stdout
+          expect{MyTodo::Todo.start(%w(create --body=wierdness_of_text --tags=tag1))}.to output("ToDo CREATED!\n\nID: 1\nToDo: wierdness_of_text\nTags: tag1\nComplete: false\n\n").to_stdout
         end
       end
 
@@ -44,7 +44,7 @@ describe MyTodo do
         end
 
         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!\n\nID: 1\nToDo: wierdness_of_text\nTags: default\nComplete: true\n").to_stdout
+          expect{MyTodo::Todo.start(%w[create --body=wierdness_of_text --done=true])}.to output("ToDo CREATED!\n\nID: 1\nToDo: wierdness_of_text\nTags: default\nComplete: true\n\n").to_stdout
         end
       end
     end

data/spec/actions/note_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe MyTodo do
+  describe '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("\nID: 1\nToDo: Some Body\nTags: \nComplete: \n\nNotes:\n\n1: note_text\n\n\n").to_stdout
+      end
+    end
+
+    context 'unsuccessful creation' do
+      it 'returns exception if note 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/rm_note_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe MyTodo do
+  describe 'rm_note' do
+    before do
+      @todo = FactoryGirl.create :item
+      @todo.notes.create(body: 'Body of text')
+    end
+
+    context 'a successful delete' do
+      before {MyTodo::Todo.start(%W[rm_note --id=#{@todo.id} --noteid=#{@todo.notes.first.id}])}
+      subject {@todo.reload.notes}
+      it {is_expected.to eq []}
+    end
+
+    context 'an unsuccessful delete' do
+      it 'returns exception if item is not found' do
+        expect {MyTodo::Todo.start(%W[rm_note --id=#{@todo.id + 1} --noteid=#{@todo.notes.first.id}])}.to output("undefined method `notes' for nil:NilClass\n").to_stdout
+      end
+
+      it 'returns exception if note is not found' do
+        expect {MyTodo::Todo.start(%W[rm_note --id=#{@todo.id} --noteid=#{@todo.notes.first.id + 1}])}.to output("undefined method `destroy!' for nil:NilClass\n").to_stdout
+      end
+    end
+  end
+end

data/spec/actions/search_spec.rb

@@ -10,11 +10,11 @@ describe MyTodo do
       end
 
       it 'finds todo item by id' do
-        expect{MyTodo::Todo.start( %W(search #{@todo3.id} ))}.to output("ToDos FOUND: 1\n\nID: 3\nToDo: always\nTags: \nComplete: \n").to_stdout
+        expect{MyTodo::Todo.start( %W(search #{@todo3.id} ))}.to output("ToDos FOUND: 1\n\nID: 3\nToDo: always\nTags: \nComplete: \n\n").to_stdout
       end
 
       it 'finds todo item by body' do
-        expect{MyTodo::Todo.start( %w[search nfl])}.to output("ToDos FOUND: 1\n\nID: 1\nToDo: nfl\nTags: \nComplete: \n").to_stdout
+        expect{MyTodo::Todo.start( %w[search nfl])}.to output("ToDos FOUND: 1\n\nID: 1\nToDo: nfl\nTags: \nComplete: \n\n").to_stdout
       end
     end
 

data/spec/actions/{add_tag_spec.rb → tag_spec.rb}

@@ -5,7 +5,7 @@ describe MyTodo do
     context 'a successful add' do
       before do
         @todo = FactoryGirl.create(:item)
-        MyTodo::Todo.start(%W[add_tag --id=#{@todo.id} --tag=tag1])
+        MyTodo::Todo.start(%W[tag --id=#{@todo.id} --tag=tag1])
       end
       subject {@todo.tags}
       it {is_expected.to include(Tag.first)}
@@ -15,11 +15,11 @@ describe MyTodo do
       before {@todo = FactoryGirl.create(:item)}
 
       it 'returns a validation error when tag name is missing' do
-        expect{MyTodo::Todo.start(%W[add_tag --id=#{@todo.id} --tag=])}.to output("Validation failed: Name can't be blank\n").to_stdout
+        expect{MyTodo::Todo.start(%W[tag --id=#{@todo.id} --tag=])}.to output("Validation failed: Name can't be blank\n").to_stdout
       end
 
       it 'returns nil exception' do
-        expect{MyTodo::Todo.start(%W[add_tag --id=#{@todo.id + 1} --tag=tag1])}.to output("undefined method `tags' for nil:NilClass\n").to_stdout
+        expect{MyTodo::Todo.start(%W[tag --id=#{@todo.id + 1} --tag=tag1])}.to output("undefined method `tags' for nil:NilClass\n").to_stdout
       end
     end
   end

data/spec/models/item_spec.rb

@@ -1,7 +1,8 @@
 require 'spec_helper'
 
 describe Item, type: :model do
-  it {should have_many(:stubs)}
-  it {should have_many(:tags)}
-  it {should validate_presence_of(:body)}
+  it {should have_many :stubs}
+  it {should have_many :tags}
+  it {should have_many :notes}
+  it {should validate_presence_of :body}
 end

data/spec/models/note_spec.rb

@@ -0,0 +1,6 @@
+require 'spec_helper'
+
+describe Note, type: :model do
+  it {should belong_to :item}
+  it {should validate_presence_of :body}
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: my_todo
 version: !ruby/object:Gem::Version
-  version: 2.0.0
+  version: 2.1.0
 platform: ruby
 authors:
 - Vell
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-10-01 00:00:00.000000000 Z
+date: 2016-10-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -214,7 +214,9 @@ files:
 - lib/db/migrate/20160912172429_create_items.rb
 - lib/db/migrate/20160913134552_create_tags.rb
 - lib/db/migrate/20160913134610_create_stubs.rb
+- lib/db/migrate/20161003121448_create_notes.rb
 - lib/db/schema.rb
+- lib/db/todos_development.sqlite3
 - lib/db/todos_test.sqlite3
 - lib/item.rb
 - lib/my_todo.rb
@@ -222,17 +224,21 @@ files:
 - lib/my_todo/templates/output.erb
 - lib/my_todo/templates/standalone_migrations.yml.erb
 - lib/my_todo/version.rb
+- lib/note.rb
 - lib/setup.rb
 - lib/stub.rb
 - lib/tag.rb
 - my_todo.gemspec
-- spec/actions/add_tag_spec.rb
 - spec/actions/create_spec.rb
 - spec/actions/delete_spec.rb
+- spec/actions/note_spec.rb
+- spec/actions/rm_note_spec.rb
 - spec/actions/rm_tag_spec.rb
 - spec/actions/search_spec.rb
+- spec/actions/tag_spec.rb
 - spec/actions/update_spec.rb
 - spec/models/item_spec.rb
+- spec/models/note_spec.rb
 - spec/models/stub_spec.rb
 - spec/models/tag_spec.rb
 - spec/spec_helper.rb
@@ -263,13 +269,16 @@ signing_key:
 specification_version: 4
 summary: A basic todo application with tags.
 test_files:
-- spec/actions/add_tag_spec.rb
 - spec/actions/create_spec.rb
 - spec/actions/delete_spec.rb
+- spec/actions/note_spec.rb
+- spec/actions/rm_note_spec.rb
 - spec/actions/rm_tag_spec.rb
 - spec/actions/search_spec.rb
+- spec/actions/tag_spec.rb
 - spec/actions/update_spec.rb
 - spec/models/item_spec.rb
+- spec/models/note_spec.rb
 - spec/models/stub_spec.rb
 - spec/models/tag_spec.rb
 - spec/spec_helper.rb