Section 5.3.2: We can now edit tickets

Author: Ryan Bigg

ticketee/app/controllers/tickets_controller.rb

@@ -18,6 +18,16 @@ def create
     end
   end
 
+  def update
+    if @ticket.update(ticket_params)
+      flash[:success] = "Ticket has been updated."
+      redirect_to [@project, @ticket]
+    else
+      flash[:error] = "Ticket has not been updated."
+      render action: "edit"
+    end
+  end
+
   private
     def ticket_params
       params.require(:ticket).permit(:title, :description)

ticketee/app/views/tickets/edit.html.erb

@@ -0,0 +1,5 @@
+<div class='row'>
+  <h2>Editing a ticket in <%= @project.name %></h2>
+
+  <%= render "form" %>
+</div>

ticketee/app/views/tickets/show.html.erb

@@ -1,4 +1,7 @@
 <div id='ticket'>
   <h2><%= @ticket.title %></h2>
+  <%= link_to "Edit Ticket",
+      [:edit, @project, @ticket],
+      class: "edit" %>
   <%= simple_format(@ticket.description) %>
 </div>

ticketee/spec/features/editing_tickets_spec.rb

@@ -0,0 +1,33 @@
+require "rails_helper"
+
+feature "Editing tickets" do
+  let!(:project) { FactoryGirl.create(:project) }
+  let!(:ticket) { FactoryGirl.create(:ticket, project: project) }
+
+  before do
+    visit "/"
+    click_link project.name
+    click_link ticket.title
+    click_link "Edit Ticket"
+  end
+
+  scenario "Updating a ticket" do
+    fill_in "Title", with: "Make it really shiny!"
+    click_button "Update Ticket"
+
+    expect(page).to have_content "Ticket has been updated."
+
+    within("#ticket h2") do
+      expect(page).to have_content("Make it really shiny!")
+    end
+
+    expect(page).to_not have_content ticket.title
+  end
+
+  scenario "Updating a ticket with invalid information" do
+    fill_in "Title", with: ""
+    click_button "Update Ticket"
+
+    expect(page).to have_content("Ticket has not been updated.")
+  end
+end