Chapter 10 complete

app/controllers/microposts_controller.rb

@@ -1,5 +1,6 @@
 class MicropostsController < ApplicationController
   before_filter :signed_in_user, only: [:create, :destroy]
+  before_filter :correct_user,   only: :destroy
 
   def create
     @micropost = current_user.microposts.build(params[:micropost])
@@ -13,5 +14,16 @@ def create
   end
 
   def destroy
+    @micropost.destroy
+    redirect_back_or root_path
   end
+
+  private
+
+  def correct_user
+    @micropost = current_user.microposts.find(params[:id])
+  rescue
+    redirect_to root_path
+  end
+
 end

app/views/microposts/_micropost.html.erb

@@ -3,4 +3,9 @@
   <span class="timestamp">
     Posted <%= time_ago_in_words(micropost.created_at) %> ago.
   </span>
+  <% if current_user?(micropost.user) %>
+      <%= link_to "delete", micropost, method:  :delete,
+          confirm: "You sure?",
+          title:   micropost.content %>
+  <% end %>
 </li>

app/views/shared/_feed_item.html.erb

@@ -1,10 +1,15 @@
 <li id="<%= feed_item.id %>">
   <%= link_to gravatar_for(feed_item.user), feed_item.user %>
   <span class="user">
-    <%= link_to feed_item.user.name, feed_item.user %>
-  </span>
+      <%= link_to feed_item.user.name, feed_item.user %>
+    </span>
   <span class="content"><%= feed_item.content %></span>
-  <span class="timestamp">
-    Posted <%= time_ago_in_words(feed_item.created_at) %> ago.
-  </span>
+    <span class="timestamp">
+      Posted <%= time_ago_in_words(feed_item.created_at) %> ago.
+    </span>
+  <% if current_user?(feed_item.user) %>
+      <%= link_to "delete", feed_item, method:  :delete,
+          confirm: "You sure?",
+          title:   feed_item.content %>
+  <% end %>
 </li>

spec/requests/micropost_pages_spec.rb

@@ -30,4 +30,17 @@
       end
     end
   end
+
+  describe "micropost destruction" do
+    before { FactoryGirl.create(:micropost, user: user) }
+
+    describe "as correct user" do
+      before { visit root_path }
+
+      it "should delete a micropost" do
+        expect { click_link "delete" }.should change(Micropost, :count).by(-1)
+      end
+    end
+  end
+
 end