Js second part

Gemfile

@@ -42,7 +42,7 @@ group :development, :test do
   gem 'rails-controller-testing'
   gem 'capybara'
   gem 'launchy'
-  gem 'selenium-webdriver', '~> 2.53.4'
+  gem 'capybara-webkit'
   gem 'geckodriver-helper'
   gem 'database_cleaner'
 end

Gemfile.lock

@@ -60,8 +60,9 @@ GEM
       rack (>= 1.0.0)
       rack-test (>= 0.5.4)
       xpath (~> 2.0)
-    childprocess (0.7.1)
-      ffi (~> 1.0, >= 1.0.11)
+    capybara-webkit (1.1.0)
+      capybara (~> 2.0, >= 2.0.2)
+      json
     coffee-rails (4.2.2)
       coffee-script (>= 2.2.0)
       railties (>= 4.0.0)
@@ -113,6 +114,7 @@ GEM
       rails-dom-testing (>= 1, < 3)
       railties (>= 4.2.0)
       thor (>= 0.14, < 2.0)
+    json (2.1.0)
     launchy (2.4.3)
       addressable (~> 2.3)
     listen (3.0.8)
@@ -206,7 +208,6 @@ GEM
     ruby-progressbar (1.8.1)
     ruby_parser (3.9.0)
       sexp_processor (~> 4.1)
-    rubyzip (1.2.1)
     sass (3.5.0)
       sass-listen (~> 3.0.7)
     sass-listen (3.0.7)
@@ -218,10 +219,6 @@ GEM
       sprockets (>= 2.8, < 4.0)
       sprockets-rails (>= 2.0, < 4.0)
       tilt (>= 1.1, < 3)
-    selenium-webdriver (2.53.4)
-      childprocess (~> 0.5)
-      rubyzip (~> 1.0)
-      websocket (~> 1.0)
     sexp_processor (4.9.0)
     shoulda-matchers (3.1.2)
       activesupport (>= 4.0.0)
@@ -256,7 +253,6 @@ GEM
       activemodel (>= 5.0)
       bindex (>= 0.4.0)
       railties (>= 5.0)
-    websocket (1.2.4)
     websocket-driver (0.6.5)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.2)
@@ -271,6 +267,7 @@ DEPENDENCIES
   bootstrap-sass
   byebug
   capybara
+  capybara-webkit
   coffee-rails (~> 4.2)
   database_cleaner
   devise
@@ -288,7 +285,6 @@ DEPENDENCIES
   rspec-rails
   rubocop
   sass-rails (~> 5.0)
-  selenium-webdriver (~> 2.53.4)
   shoulda-matchers
   spring
   spring-watcher-listen (~> 2.0.0)

app/assets/javascripts/answers.js.coffee

@@ -0,0 +1,9 @@
+editanswer = ->
+  $('.edit-answer-link').click (e) ->
+    e.preventDefault()
+    $(this).hide()
+    answer_id = $(this).data('answerId')
+    $('form#edit-answer-' + answer_id).show()
+
+$(document).ready(editanswer)
+$(document).on('turbolinks:load', editanswer)

app/assets/stylesheets/application.css.scss

@@ -1,3 +1,4 @@
 @import "bootstrap-sprockets";
 @import "bootstrap";
 
+.edit_answer {display: none;}

app/controllers/answers_controller.rb

@@ -1,7 +1,7 @@
 class AnswersController < ApplicationController
   before_action :authenticate_user!
-  before_action :set_question, only: [:new, :create, :destroy]
-  before_action :set_answer, only: [:destroy]
+  before_action :set_question, only: [:new, :create, :destroy, :update, :best]
+  before_action :set_answer, only: [:destroy, :update, :best]
 
   def create
     @answer = @question.answers.new(answer_params)
@@ -10,14 +10,30 @@ def create
 
   end
 
+  def update
+    if current_user.author_of?(@answer)
+      render "error" unless @answer.update(answer_params)
+    else
+      flash[:notice] = "You are not auther of the answer"
+    end
+  end
+
   def destroy
     if current_user.author_of?(@answer)
       flash[:notice] = "Your answer successfully deleted."
       @answer.destroy
     else
       flash[:notice] = "You are not author of the answer"
     end
-    redirect_to question_path(@question)
+  end
+
+  def best
+    if current_user.author_of?(@question)
+      @answer.set_best
+    else
+      render[:notice] = "You are not author of the question"
+    end
+
   end
 
   private

app/controllers/questions_controller.rb

@@ -25,13 +25,22 @@ def show
   end
 
   def edit
+    if current_user.author_of?(@question)
+      render :edit
+    else
+      redirect_to questions_path
+    end
   end
 
   def update
-    if @question.update(questions_params)
-      redirect_to question_path(@question)
+    if current_user.author_of?(@question)
+      if @question.update(questions_params)
+        redirect_to questions_path
+      else
+       render :edit
+      end
     else
-      render :edit
+      redirect_to questions_path
     end
   end
 

app/models/answer.rb

@@ -3,4 +3,11 @@ class Answer < ApplicationRecord
   belongs_to :user
 
   validates :body, presence: true
+
+  def set_best
+    Answer.transaction do
+      self.question.answers.update_all(best: false)
+      self.update!(best: true)
+    end
+  end
 end

app/models/question.rb

@@ -1,5 +1,5 @@
 class Question < ApplicationRecord
-  has_many :answers, dependent: :destroy
+  has_many :answers,-> { order(best: :desc) }, dependent: :destroy
   belongs_to :user
 
   validates :title, presence: true

app/views/answers/_answer.html.haml

@@ -1,7 +1,16 @@
 - if answer.persisted?
-  .row
+  .row{id: "answer-body-#{answer.id}", class: ("best-answer" unless answer.best?)}
     = answer.body
-  .row
-    - if current_user&.author_of?(answer)
-      = link_to 'Delete answer', question_answer_path(@question, answer), method: :delete
+    .row
+      - if current_user && current_user.author_of?(@question) && !answer.best?
+        = link_to "best", best_question_answer_path(@question, answer), remote: true, method: :put
+      - if current_user&.author_of?(answer)
+        %p
+          = link_to "Edit", '', class: "edit-answer-link", data: {answer_id: answer.id}
+          = form_for [@question, answer], remote: true, html: {id: "edit-answer-#{answer.id}"} do |f|
+            = f.label :body
+            = f.text_area :body
+            = f.submit "Save"
+        %p
+          = link_to 'Delete answer', question_answer_path(@question, answer), method: :delete, remote: true
 

app/views/answers/best.js.erb

@@ -0,0 +1,5 @@
+alert('Hello');
+$(".answers").html("<%= j render @question.answers%>");
+$(".alert").html("You chose best answer.");
+
+

app/views/answers/create.js.erb

@@ -1,2 +1,4 @@
+$("textarea#new_answer_body").val("");
 $(".answers").append("<%= j render @answer%>");
 
+

app/views/answers/destroy.js.erb

@@ -0,0 +1,4 @@
+$(".row#answer-body-<%= @answer.id%>").html('');
+$(".alert").html("Your answer successfully deleted.")
+
+

app/views/answers/update.js.erb

@@ -0,0 +1,5 @@
+$('.row#answer-body-' + <%= @answer.id %>).html('<%= j  @answer.body%>');
+$('form#edit-answer-' + <%= @answer.id %>).hide();
+$('.edit-answer-link[data-answer-id="<%= @answer.id %>"]').show();
+
+

app/views/questions/_form.html.haml

@@ -13,4 +13,4 @@
       .col-md-8
         = f.text_area :body
     .row
-      = f.submit "Create"
+      = f.submit "Save"

app/views/questions/edit.html.haml

@@ -1 +1,3 @@
-
+= render "form"
+%br
+= link_to "Назад", root_path

app/views/questions/index.html.haml

@@ -4,6 +4,8 @@
 - @questions.each do |question|
   .row
     %h2= link_to question.title, question_path(question)
+    - if current_user && current_user.author_of?(question)
+      %p= link_to "Edit",edit_question_path(question)
 .row
   %br
   %br

app/views/questions/show.html.haml

@@ -5,7 +5,7 @@
   = @question.body
 .row
   - if current_user&.author_of?(@question)
-    = link_to 'Delete question', question_path(@question), method: :delete
+    = link_to 'Delete question', question_path(@question), method: :delete, remote: true
 .row
   %h3Комментарии
 .row
@@ -19,7 +19,7 @@
       = render "shared/error_form", object: f.object
     .row
     = f.label :body
-    = f.text_field :body
+    = f.text_area :body, id: "new_answer_body"
     = f.submit "Create answer"
 .row
   = link_to "Назад", questions_path

config/routes.rb

@@ -4,6 +4,8 @@
   root "questions#index"
 
   resources :questions do
-    resources :answers
+    resources :answers do
+      put :best, on: :member
+    end
   end
 end

db/migrate/20170929121314_add_best_to_answers.rb

@@ -0,0 +1,5 @@
+class AddBestToAnswers < ActiveRecord::Migration[5.0]
+  def change
+    add_column :answers, :best, :boolean, default: false
+  end
+end

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: 20170713165150) do
+ActiveRecord::Schema.define(version: 20170929121314) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -21,6 +21,7 @@
     t.datetime "updated_at",  null: false
     t.integer  "question_id"
     t.integer  "user_id"
+    t.boolean  "best"
     t.index ["question_id"], name: "index_answers_on_question_id", using: :btree
     t.index ["user_id"], name: "index_answers_on_user_id", using: :btree
   end

spec/controllers/answers_controller_spec.rb

@@ -3,7 +3,7 @@
 RSpec.describe AnswersController, :type => :controller do
   sign_in_user
   let!(:question) { create(:question, user: @user) }
-  let!(:answer) {create( :answer, question: question)}
+  let!(:answer) {create( :answer, question: question, user: @user)}
 
   describe 'POST #create' do
     context 'valid answer' do
@@ -36,23 +36,58 @@
     end
   end
 
+  describe 'PATCH #update' do
+
+    it 'assigns @answer to requested answer' do
+      patch :update, params: {id: answer, question_id: question, answer: attributes_for(:answer)}, format: :js
+      expect(assigns(:answer)).to eq answer
+    end
+
+    it 'changes answer attributes' do
+      patch :update, params: {id: answer, question_id: question, answer: {body: "NewBody"}}, format: :js
+      answer.reload
+      expect(answer.body).to eq "NewBody"
+
+    end
+
+    it 'render template update' do
+      patch :update, params: {id: answer, question_id: question, answer: {body: "NewBody"}}, format: :js
+      expect(response).to render_template :update
+    end
+
+    it 'render error template if errors exists' do
+      patch :update, params: {id: answer, question_id: question, answer: {body: ""}}, format: :js
+      expect(response).to render_template :error
+    end
+
+    let!(:another_user) {create(:user)}
+    let(:another_answer) {create(:answer, question: question, user: another_user)}
+    it 'not change answer if user not author' do
+       patch :update, params: {id: another_answer, question_id: question, answer: {body: "NewBody"}}, format: :js
+       another_answer.reload
+       expect(another_answer.body).to eq "Answer body"
+
+    end
+
+  end
+
   describe 'DELETE #destroy' do
     sign_in_user
     it "user tries to delete own answer" do
       answer = create(:answer, question: question, user: @user)
 
-      expect{delete :destroy, params:{ question_id: question, id: answer}}.to change(Answer, :count).by(-1)
+      expect{delete :destroy, params:{ question_id: question, id: answer}, format: :js}.to change(Answer, :count).by(-1)
 
     end
 
     it "user can't delete someone else question" do
       another_answer = create(:answer, question: question, user: create(:user))
-      expect{delete :destroy, params: {question_id: question, id: another_answer}}.to_not change(Answer, :count)
+      expect{delete :destroy, params: {question_id: question, id: another_answer}, format: :js}.to_not change(Answer, :count)
     end
 
-    it "redirects to question show page" do
-      delete :destroy, params: {question_id: question, id: answer}
-      expect(response).to redirect_to question_path(question)
+    it "render destroy template" do
+      delete :destroy, params: {question_id: question, id: answer}, format: :js
+      expect(response).to render_template :destroy
     end
   end
 end