Merge pull request #35 from thanhhung-131/rspec_sessions

Rspec for sessions controller

app/controllers/carts_controller.rb

@@ -5,7 +5,7 @@ class CartsController < ApplicationController
   before_action :load_current_account, only: :create
   before_action :load_product, only: %i(create destroy)
   before_action :set_cart, only: :create
-  before_action :load_cart_product, only: :show
+  before_action :logged_in_user, :load_cart_product, only: :show
 
   def show; end
 

app/controllers/orders_controller.rb

@@ -4,6 +4,7 @@ class OrdersController < ApplicationController
   before_action :parse_cart_data, only: :create
   before_action :find_products, only: :create
   before_action :load_order, only: %i(show cancel)
+  before_action :logged_in_user, only: :index
 
   def index
     @pagy, @orders = pagy(current_account.orders.order(created_at: :desc), items: Settings.PAGE_10)

app/views/layouts/_flash.html.erb

@@ -18,6 +18,5 @@
           <%= flash[:info] %>
         </div>
     <% end %>
-
   </div>
 <% end %>

app/views/sessions/new.html.erb

@@ -1,3 +1,4 @@
+<%= render "layouts/flash" %>
 <div class="row">
   <div class="col-md-4 offset-md-1">
     <h2 class = "sign-up-title"><%= uppercase t("accounts.login_customer") %></h2>

app/views/shared/_header.html.erb

@@ -29,7 +29,7 @@
             <%= link_to t("header.home"), root_url, class: "nav-link" %>
           </li>
           <li class="nav-item">
-            <%= link_to t("header.cart"), login_path, class: "nav-link" %>
+            <%= link_to t("header.cart"), cart_path, class: "nav-link" %>
           </li>
           <li class="nav-item">
             <%= link_to t("header.login"), login_path, class: "nav-link" %>

spec/controllers/accounts_controller_spec.rb

@@ -52,7 +52,7 @@
     end
 
     context "with invalid attributes" do
-      let(:invalid_attributes) do
+      let(:invalid_params) do
         {
           account: {
             name: "",
@@ -65,7 +65,7 @@
         }
       end
 
-      before { post :create, params: invalid_attributes }
+      before { post :create, params: invalid_params }
 
       it "does not save the new account" do
         expect(assigns(:account).id).not_to(be_present)

spec/controllers/sessions_controller_spec.rb

@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+
+# spec/controllers/sessions_controller_spec.rb
+require "rails_helper"
+require "support/shared_examples/login_examples"
+
+RSpec.describe(SessionsController, type: :controller) do
+  include SessionsHelper
+
+  describe "GET #new" do
+    it "renders the new template" do
+      get :new
+      expect(response).to(render_template(:new))
+    end
+  end
+
+  describe "POST #create" do
+    let(:account) { create(:account) }
+
+    context "load account failed" do
+      before do
+        post :create, params: { session: { email: "[email protected]", password: "password" } }
+      end
+
+      it "renders new template with error message" do
+        expect(flash.now[:error]).to(eq(I18n.t("sessions.login_email_err")))
+      end
+
+      it "returns unprocessable_entity status and render template new" do
+        expect(response).to(have_http_status(:unprocessable_entity))
+        expect(response).to(render_template(:new))
+      end
+    end
+
+    context "load account success but authenticate failed" do
+      before do
+        post :create, params: { session: { email: account.email, password: "incorrect_password" } }
+      end
+
+      it "renders new template with error message" do
+        expect(flash.now[:error]).to(eq(I18n.t("sessions.login_password_err")))
+      end
+
+      it "returns unprocessable_entity status and render template new" do
+        expect(response).to(have_http_status(:unprocessable_entity))
+        expect(response).to(render_template(:new))
+      end
+    end
+
+    context "login success" do
+      before do
+        post :create, params: { session: { email: account.email, password: account.password } }
+      end
+
+      it "logs in the account and redirects to root path" do
+        expect(session[:account_id]).to(eq(account.id))
+        expect(response).to(redirect_to(root_path))
+      end
+    end
+  end
+
+  describe "DELETE #destroy" do
+    include_examples "login examples"
+
+    it "logs out the current account and redirects to root path" do
+      delete :destroy
+      expect(session[:account_id]).to(be_nil)
+      expect(response).to(redirect_to(root_path))
+    end
+  end
+end

spec/support/shared_examples/login_examples.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+RSpec.shared_examples("login examples") do
+  let(:account) { create(:account) }
+  before { log_in(account) }
+
+  it "logs in successfully" do
+    expect(current_account).to(be_present)
+  end
+end