Refs #16: add rails-jquery to make deletion work

hpi-swt2-exercise · Nov 3, 2017 · c7cf4de · c7cf4de
1 parent 50a5992
commit c7cf4de
Show file tree

Hide file tree

Showing 9 changed files with 591 additions and 48 deletions.
diff --git a/.idea/.rakeTasks b/.idea/.rakeTasks
diff --git a/.idea/exercise.iml b/.idea/exercise.iml
diff --git a/.idea/workspace.xml b/.idea/workspace.xml
diff --git a/Gemfile b/Gemfile
@@ -1,5 +1,7 @@
 source 'https://rubygems.org'
 
+gem 'jquery-rails'
+gem 'turbolinks'
 
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
 gem 'rails', '4.2.4'

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -58,6 +58,10 @@ GEM
     globalid (0.3.6)
       activesupport (>= 4.1.0)
     i18n (0.7.0)
+    jquery-rails (4.3.1)
+      rails-dom-testing (>= 1, < 3)
+      railties (>= 4.2.0)
+      thor (>= 0.14, < 2.0)
     json (1.8.3)
     loofah (2.0.3)
       nokogiri (>= 1.5.9)
@@ -123,6 +127,9 @@ GEM
     sqlite3 (1.3.10)
     thor (0.19.1)
     thread_safe (0.3.5)
+    turbolinks (5.0.1)
+      turbolinks-source (~> 5)
+    turbolinks-source (5.0.3)
     tzinfo (1.2.2)
       thread_safe (~> 0.1)
     web-console (2.2.1)
@@ -140,11 +147,13 @@ DEPENDENCIES
   byebug
   capybara (~> 2.5)
   factory_girl_rails (~> 4.5)
+  jquery-rails
   rails (= 4.2.4)
   rspec-rails (~> 3.3)
   spring
   sqlite3
+  turbolinks
   web-console (~> 2.0)
 
 BUNDLED WITH
-   1.11.2
+   1.16.0
diff --git a/app/assets/javascripts/application.js b/app/assets/javascripts/application.js
@@ -13,4 +13,4 @@
 //= require jquery
 //= require jquery_ujs
 //= require turbolinks
-//= require_tree .
+//= require_tree .
diff --git a/app/controllers/authors_controller.rb b/app/controllers/authors_controller.rb
@@ -34,6 +34,13 @@ def update
       render 'edit'
     end
   end
+
+  def destroy
+    @author = Author.find(params[:id])
+    @author.destroy
+
+    redirect_to authors_path
+  end
 end
 
 private

diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
@@ -3,6 +3,7 @@
 <head>
   <title>Paper Management System</title>
   <%= stylesheet_link_tag    'application', media: 'all' %>
+  <%= javascript_include_tag    'application' %>
   <%= csrf_meta_tags %>
 </head>
 <body>

diff --git a/spec/features/author/index_author_spec.rb b/spec/features/author/index_author_spec.rb
@@ -38,6 +38,7 @@
     visit authors_path
 
     expect(page).to have_link('Destroy')
+    expect(Author.find_by_first_name('Alan')).not_to be_nil
     click_link('Destroy')
     expect(Author.find_by_first_name('Alan')).to be_nil
   end