Section 3.4.5: Add validation to ensure names are specified when creating projects

Author: Ryan Bigg

ticketee/app/controllers/projects_controller.rb

@@ -13,7 +13,8 @@ def create
       flash[:notice] = "Project has been created."
       redirect_to @project
     else
-      # nothing, yet
+      flash[:alert] = "Project has not been created."
+      render "new"
     end
   end
 

ticketee/app/models/project.rb

@@ -1,2 +1,3 @@
 class Project < ActiveRecord::Base
+  validates :name, presence: true
 end

ticketee/app/views/projects/new.html.erb

@@ -1,5 +1,17 @@
 <h2>New Project</h2>
 <%= form_for(@project) do |f| %>
+<% if @project.errors.any? %>
+ <div id="error_explanation">
+   <h2><%= pluralize(@project.errors.count, "error") %>
+   prohibited this project from being saved:</h2>
+
+   <ul>
+     <% @project.errors.full_messages.each do |msg| %>
+      <li><%= msg %></li>
+     <% end %>
+   </ul>
+ </div>
+<% end %>
   <p>
     <%= f.label :name %><br>
     <%= f.text_field :name %>

ticketee/spec/features/creating_projects_spec.rb

@@ -1,22 +1,31 @@
 require 'rails_helper'
 
 feature 'Creating Projects' do
-  scenario 'can create a project' do
+  before do
     visit '/'
 
     click_link 'New Project'
+  end
 
+  scenario 'can create a project' do
     fill_in 'Name', with: 'Sublime Text 3'
-    fill_in 'Description', with: 'A text-editor for everyone'
+    fill_in 'Description', with: 'A text-editor for OS X'
     click_button 'Create Project'
 
     expect(page).to have_content('Project has been created.')
 
-    project = Project.where(name: "Sublime Text 3").first
+    project = Project.where(name: 'Sublime Text 3').first
 
     expect(page.current_url).to eql(project_url(project))
 
-    title = "Sublime Text 3 - Projects - Ticketee"
+    title = 'Sublime Text 3 - Projects - Ticketee'
     expect(page).to have_title(title)
   end
+
+  scenario "can not create a project without a name" do
+    click_button 'Create Project'
+
+    expect(page).to have_content("Project has not been created.")
+    expect(page).to have_content("Name can't be blank")
+  end
 end