Implement RSpec

.rspec

+--require spec_helper

Gemfile

@@ -69,6 +69,16 @@ gem 'rsolr', '~> 2.0', '>= 2.0.2'
 group :development, :test do
   # Call 'byebug' anywhere in the code to stop execution and get a debugger console
   gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
+  # rspec-rails is a testing framework for Rails 3+.
+  gem 'rspec-rails', '~> 3.6'
+  # factory_girl_rails provides integration between factory_girl and rails 3 or newer
+  gem 'factory_girl_rails', '~> 4.8'
+  # Making tests easy on the fingers and eyes
+  gem 'shoulda', '~> 3.5'
+  # An IRB alternative and runtime developer console
+  gem 'pry', '~> 0.10.4'
+  # Extracting `assigns` and `assert_template` from ActionDispatch.
+  gem 'rails-controller-testing', '~> 1.0', '>= 1.0.2'
   # Adds support for Capybara system testing and selenium driver
   gem 'capybara', '~> 2.13'
   gem 'selenium-webdriver'

Gemfile.lock

@@ -68,6 +68,7 @@ GEM
       mime-types (>= 1.16)
     childprocess (0.7.0)
       ffi (~> 1.0, >= 1.0.11)
+    coderay (1.1.1)
     coffee-rails (4.2.2)
       coffee-script (>= 2.2.0)
       railties (>= 4.0.0)
@@ -87,8 +88,14 @@ GEM
       railties (>= 4.1.0, < 5.2)
       responders
       warden (~> 1.2.3)
+    diff-lcs (1.3)
     erubi (1.6.0)
     execjs (2.7.0)
+    factory_girl (4.8.0)
+      activesupport (>= 3.0.0)
+    factory_girl_rails (4.8.0)
+      factory_girl (~> 4.8.0)
+      railties (>= 3.0.0)
     faker (1.6.6)
       i18n (~> 0.5)
     faraday (0.12.1)
@@ -148,6 +155,10 @@ GEM
     parser (2.4.0.0)
       ast (~> 2.2)
     powerpack (0.1.1)
+    pry (0.10.4)
+      coderay (~> 1.1.0)
+      method_source (~> 0.8.1)
+      slop (~> 3.4)
     public_suffix (2.0.5)
     puma (3.9.1)
     rack (2.0.3)
@@ -173,6 +184,10 @@ GEM
       rails-assets-bootstrap (>= 3.0.0)
       rails-assets-jquery (>= 1.9.0)
     rails-assets-jquery (3.2.1)
+    rails-controller-testing (1.0.2)
+      actionpack (~> 5.x, >= 5.0.1)
+      actionview (~> 5.x, >= 5.0.1)
+      activesupport (~> 5.x)
     rails-dom-testing (2.0.3)
       activesupport (>= 4.2.0)
       nokogiri (>= 1.6)
@@ -197,6 +212,23 @@ GEM
     rsolr (2.0.2)
       builder (>= 2.1.2)
       faraday
+    rspec-core (3.6.0)
+      rspec-support (~> 3.6.0)
+    rspec-expectations (3.6.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.6.0)
+    rspec-mocks (3.6.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.6.0)
+    rspec-rails (3.6.0)
+      actionpack (>= 3.0)
+      activesupport (>= 3.0)
+      railties (>= 3.0)
+      rspec-core (~> 3.6.0)
+      rspec-expectations (~> 3.6.0)
+      rspec-mocks (~> 3.6.0)
+      rspec-support (~> 3.6.0)
+    rspec-support (3.6.0)
     rubocop (0.49.1)
       parallel (~> 1.10)
       parser (>= 2.3.3.1, < 3.0)
@@ -218,11 +250,18 @@ GEM
       childprocess (~> 0.5)
       rubyzip (~> 1.0)
       websocket (~> 1.0)
+    shoulda (3.5.0)
+      shoulda-context (~> 1.0, >= 1.0.1)
+      shoulda-matchers (>= 1.4.1, < 3.0)
+    shoulda-context (1.2.2)
+    shoulda-matchers (2.8.0)
+      activesupport (>= 3.0.0)
     sidekiq (5.0.2)
       concurrent-ruby (~> 1.0)
       connection_pool (~> 2.2, >= 2.2.0)
       rack-protection (>= 1.5.0)
       redis (~> 3.3, >= 3.3.3)
+    slop (3.6.0)
     spring (2.0.2)
       activesupport (>= 4.2)
     spring-watcher-listen (2.0.1)
@@ -274,6 +313,7 @@ DEPENDENCIES
   coffee-rails (~> 4.2)
   config
   devise (~> 4.3)
+  factory_girl_rails (~> 4.8)
   faker (~> 1.6, >= 1.6.3)
   font-awesome-rails (~> 4.7, >= 4.7.0.2)
   jbuilder (~> 2.5)
@@ -283,13 +323,17 @@ DEPENDENCIES
   listen (>= 3.0.5, < 3.2)
   mini_magick
   mysql2 (>= 0.3.18, < 0.5)
+  pry (~> 0.10.4)
   puma (~> 3.7)
   rails (~> 5.1.1)
   rails-assets-bootstrap-touchspin!
+  rails-controller-testing (~> 1.0, >= 1.0.2)
   rsolr (~> 2.0, >= 2.0.2)
+  rspec-rails (~> 3.6)
   rubocop (~> 0.49.1)
   sass-rails (~> 5.0)
   selenium-webdriver
+  shoulda (~> 3.5)
   sidekiq (~> 5.0, >= 5.0.2)
   spring
   spring-watcher-listen (~> 2.0.0)

spec/controllers/orders_controller_spec.rb

+require 'rails_helper'
+
+RSpec.describe OrdersController, type: :controller do
+  let!(:user) { FactoryGirl.create(:user) }
+
+  describe '#show' do
+    before { sign_in user }
+
+    def do_request
+      get :show, params: { id: order.id }
+    end
+
+    context 'as an authorized user' do
+      let!(:order) { FactoryGirl.create(:order, user: user) }
+
+      it 'assigns the requested order to @order' do
+        do_request
+        expect(assigns(:order)).to eq order
+      end
+
+      it 'renders the :show template' do
+        do_request
+        expect(response).to render_template(:show)
+      end
+    end
+
+    context 'as an unauthorized user' do
+      let(:other_user) { FactoryGirl.create(:user) }
+      let(:order) { FactoryGirl.create(:order, user: other_user) }
+
+      it 'redirects to home page' do
+        do_request
+        expect(response).to redirect_to(root_url)
+      end
+    end
+  end
+
+  describe '#new' do
+    context 'as an authorized user' do
+      before { sign_in user }
+
+      it 'assigns a new Order to @order' do
+        get :new
+        expect(assigns(:order)).to be_a_new(Order)
+      end
+
+      it 'render the :new template' do
+        get :new
+        expect(response).to render_template(:new)
+      end
+    end
+
+    context 'as a guest' do
+      it 'redirects to the sign-in page' do
+        get :new
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+  end
+
+  describe '#edit' do
+    def do_request
+      get :edit, params: { id: order.id }
+    end
+
+    context 'as an authorized user' do
+      let(:admin_user) { FactoryGirl.create(:user, admin: true) }
+      let!(:order) { FactoryGirl.create(:order) }
+      before { sign_in admin_user }
+
+      it 'assigns the requested order to @order' do
+        do_request
+        expect(assigns(:order)).to eq order
+      end
+
+      it 'render the :edit template' do
+        do_request
+        expect(response).to render_template(:edit)
+      end
+    end
+
+    context 'as an unauthorized user' do
+      let(:order) { FactoryGirl.create(:order) }
+      before { sign_in user }
+
+      it 'redirects to home page' do
+        do_request
+        expect(response).to redirect_to(root_url)
+      end
+    end
+  end
+
+  describe '#create' do
+    def do_request
+      get :create, params: { order: order_params }
+    end
+
+    context 'as an authorized user' do
+      let!(:order) { FactoryGirl.create(:order, user: user) }
+      before { sign_in user }
+
+      context 'with valid attributes' do
+        let!(:order_params) { FactoryGirl.attributes_for(:order) }
+
+        it 'save the new order to database' do
+          expect { do_request }.to change(Order, :count).by(1)
+        end
+
+        it 'redirect to order details' do
+          do_request
+          expect(response).to redirect_to order_url(assigns(:order))
+        end
+      end
+
+      context 'with invalid attributes' do
+        let(:order_params) { FactoryGirl.attributes_for(:order, order_status: 'Not valid') }
+
+        it 'does not save changes to database' do
+          expect { do_request }.to raise_error(ArgumentError)
+        end
+      end
+    end
+
+    context 'as a guest' do
+      let(:order_params) { FactoryGirl.attributes_for(:order) }
+
+      it 'redirects to home page' do
+        do_request
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+  end
+
+  describe '#update' do
+    def do_request
+      patch :update, params: { id: order.id, order: order_params }
+    end
+
+    context 'as an authorized user' do
+      let(:admin_user) { FactoryGirl.create(:user, admin: true) }
+      let!(:order) { FactoryGirl.create(:order) }
+      before { sign_in admin_user }
+
+      context 'with valid attributes' do
+        let!(:order_params) { FactoryGirl.attributes_for(:order, order_status: 'Done') }
+
+        it 'save changes to database' do
+          do_request
+          expect(order.reload.order_status).to eq 'Done'
+        end
+      end
+
+      context 'with invalid attributes' do
+        let!(:order_params) { FactoryGirl.attributes_for(:order, user_id: nil) }
+
+        it 'does not save changes to database' do
+          do_request
+          expect(order.reload.order_status).to eq 'Pending'
+        end
+      end
+    end
+
+    context 'as an unauthorized user' do
+      let(:order) { FactoryGirl.create(:order) }
+      let(:order_params) { FactoryGirl.attributes_for(:order) }
+      before { sign_in user }
+
+      it 'redirects to home page' do
+        do_request
+        expect(response).to redirect_to(root_url)
+      end
+    end
+  end
+end

spec/controllers/products_controller_spec.rb

+require 'rails_helper'
+
+RSpec.describe ProductsController, type: :controller do
+  let!(:user) { FactoryGirl.create(:user) }
+
+  describe '#show' do
+    let!(:product) { FactoryGirl.create(:product) }
+
+    def do_request
+      get :show, params: { id: product.id }
+    end
+
+    it 'assigns the requested product to @product' do
+      do_request
+      expect(assigns(:product)).to eq product
+    end
+
+    it 'renders the :show template' do
+      do_request
+      expect(response).to render_template(:show)
+    end
+  end
+
+  describe '#new' do
+    context 'as an authorized user' do
+      before { sign_in user }
+
+      it 'assigns a new Product to @product' do
+        get :new
+        expect(assigns(:product)).to be_a_new(Product)
+      end
+
+      it 'render the :new template' do
+        get :new
+        expect(response).to render_template(:new)
+      end
+    end
+  end
+
+  describe '#edit' do
+    let!(:product) { FactoryGirl.create(:product, user: user) }
+
+    def do_request
+      get :edit, params: { id: product.id }
+    end
+
+    context 'as an authorized user' do
+      before { sign_in user }
+
+      it 'assigns the requested product to @product' do
+        do_request
+        expect(assigns(:product)).to eq product
+      end
+
+      it 'render the :edit template' do
+        do_request
+        expect(response).to render_template(:edit)
+      end
+    end
+
+    context 'as a guest' do
+      it 'redirect to the sign-in page' do
+        do_request
+        expect(response).to redirect_to(new_user_session_path)
+      end
+    end
+  end
+
+  describe '#create' do
+    def do_request
+      post :create, params: { product: product_params }
+    end
+
+    context 'as an authorized user' do
+      let!(:product) { FactoryGirl.create(:product) }
+      before { sign_in user }
+
+      context 'with valid attributes' do
+        let(:category) { FactoryGirl.create(:category) }
+        let!(:product_params) do
+          FactoryGirl.attributes_for(:product,
+                                     user_id: user.id,
+                                     category_id: category)
+        end
+
+        it 'save the new product in the database' do
+          expect { do_request }.to change(Product, :count).by(1)
+        end
+        it 'redirects to home page' do
+          do_request
+          expect(response).to redirect_to root_url
+        end
+      end
+
+      context 'with invalid attributes' do
+        let(:product_params) { FactoryGirl.attributes_for(:product) }
+
+        it 'does not save the new product' do
+          expect { do_request }.to_not change(Product, :count)
+        end
+        it 're-renders the :new template' do
+          do_request
+          expect(response).to render_template :new
+        end
+      end
+    end
+  end
+
+  describe '#update' do
+    before do
+      sign_in user
+    end
+
+    def do_request
+      patch :update, params: { id: product.id, product: product_params }
+    end
+
+    context 'as an authorized user' do
+      let!(:product) { FactoryGirl.create(:product, user: user) }
+
+      context 'with valid attributes' do
+        let(:product_params) { FactoryGirl.attributes_for(:product, title: 'New Product Title') }
+
+        it 'update changes to database' do
+          do_request
+          expect(product.reload.title).to eq 'New Product Title'
+        end
+      end
+
+      context 'with invalid attributes' do
+        let(:product_params) { FactoryGirl.attributes_for(:product, category_id: nil) }
+
+        it 'does not update changes to database' do
+          do_request
+          expect(response).to render_template(:edit)
+        end
+      end
+    end
+
+    context 'as an unauthorized user' do
+      let(:other_user) { FactoryGirl.create(:user) }
+      let(:product) { FactoryGirl.create(:product, user: other_user) }
+      let(:product_params) { FactoryGirl.attributes_for(:product) }
+
+      it 'redirect to home page' do
+        do_request
+        expect(response).to redirect_to(root_url)
+      end
+    end
+  end
+end

spec/factories/categories.rb

+FactoryGirl.define do
+  factory :category do
+    title 'Test Category'
+  end
+end

spec/factories/orders.rb

+FactoryGirl.define do
+  factory :order do
+    user
+    order_status 'Pending'
+  end
+end

spec/factories/products.rb

+FactoryGirl.define do
+  factory :product do
+    sequence(:title) { |n| "Product #{n}" }
+    description Faker::Lorem.paragraphs
+    sequence(:sku) { |n| "u-#{n}" }
+    price Faker::Commerce.price
+    category
+    user
+  end
+end
\ No newline at end of file

spec/factories/users.rb

+FactoryGirl.define do
+  factory :user do
+    sequence(:email) { |n| "user_test#{n}@test.com" }
+    password 'password'
+    password_confirmation 'password'
+  end
+end

spec/models/category_spec.rb

+require 'rails_helper'
+
+RSpec.describe Category, type: :model do
+  describe 'db schema' do
+    context 'columns' do
+      it { is_expected.to have_db_column(:title).of_type(:string) }
+      it { is_expected.to have_db_column(:description).of_type(:text) }
+    end
+  end
+end

spec/models/order_spec.rb

+require 'rails_helper'
+
+RSpec.describe Order, type: :model do
+  describe 'db schema' do
+    context 'columns' do
+      it { is_expected.to define_enum_for(:order_status).with(['Pending', 'Done']) }
+    end
+  end
+
+  context 'validations' do
+    it { is_expected.to validate_presence_of(:order_status) }
+  end
+
+  context 'associations' do
+    it { is_expected.to belong_to(:user) }
+    it { is_expected.to have_many(:product_items) }
+  end
+end

spec/models/product_item_spec.rb

+require 'rails_helper'
+
+RSpec.describe ProductItem, type: :model do
+  describe 'db schema' do
+    context 'columns' do
+      it { is_expected.to have_db_column(:quantity).of_type(:integer) }
+    end
+  end
+
+  context 'validations' do
+    it { is_expected.to validate_numericality_of(:quantity).is_greater_than(0) }
+  end
+
+  context 'associations' do
+    it { is_expected.to belong_to(:order) }
+    it { is_expected.to belong_to(:product) }
+  end
+end

spec/models/product_spec.rb

+require 'rails_helper'
+
+RSpec.describe Product, type: :model do
+  describe 'db schema' do
+    context 'columns' do
+      it { is_expected.to have_db_column(:title).of_type(:string) }
+      it { is_expected.to have_db_column(:description).of_type(:text) }
+      it { is_expected.to have_db_column(:sku).of_type(:string) }
+      it { is_expected.to have_db_column(:quantity).of_type(:integer) }
+      it { is_expected.to have_db_column(:price).of_type(:decimal) }
+    end
+  end
+
+  context 'validations' do
+    it { is_expected.to validate_presence_of(:title) }
+    it { is_expected.to validate_presence_of (:price) }
+    it { is_expected.to validate_numericality_of (:price) }
+  end
+
+  context 'associations' do
+    it { is_expected.to belong_to(:user) }
+    it { is_expected.to belong_to(:category) }
+  end
+end

spec/models/user_spec.rb

+require 'rails_helper'
+
+RSpec.describe User, type: :model do
+  describe 'db schema' do
+    context 'columns' do
+      it { is_expected.to have_db_column(:first_name) }
+      it { is_expected.to have_db_column(:last_name) }
+      it { is_expected.to have_db_column(:email) }
+      it { is_expected.to have_db_column(:username) }
+    end
+  end
+
+  context 'validations' do
+    it { is_expected.to validate_presence_of(:email) }
+  end
+
+  context 'associations' do
+    it { is_expected.to have_many(:orders) }
+    it { is_expected.to have_many(:products) }
+  end
+end

spec/rails_helper.rb

+# This file is copied to spec/ when you run 'rails generate rspec:install'
+require 'spec_helper'
+ENV['RAILS_ENV'] ||= 'test'
+require File.expand_path('../../config/environment', __FILE__)
+# Prevent database truncation if the environment is production
+abort("The Rails environment is running in production mode!") if Rails.env.production?
+require 'rspec/rails'
+# Add additional requires below this line. Rails is not loaded until this point!
+require 'devise'
+
+# Requires supporting ruby files with custom matchers and macros, etc, in
+# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
+# run as spec files by default. This means that files in spec/support that end
+# in _spec.rb will both be required and run as specs, causing the specs to be
+# run twice. It is recommended that you do not name files matching this glob to
+# end with _spec.rb. You can configure this pattern with the --pattern
+# option on the command line or in ~/.rspec, .rspec or `.rspec-local`.
+#
+# The following line is provided for convenience purposes. It has the downside
+# of increasing the boot-up time by auto-requiring all files in the support
+# directory. Alternatively, in the individual `*_spec.rb` files, manually
+# require only the support files necessary.
+#
+Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }
+
+# Checks for pending migration and applies them before tests are run.
+# If you are not using ActiveRecord, you can remove this line.
+ActiveRecord::Migration.maintain_test_schema!
+
+RSpec.configure do |config|
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # Devise helpers
+  config.include Devise::Test::ControllerHelpers, type: :controller
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, remove the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = true
+
+  # RSpec Rails can automatically mix in different behaviours to your tests
+  # based on their file location, for example enabling you to call `get` and
+  # `post` in specs under `spec/controllers`.
+  #
+  # You can disable this behaviour by removing the line below, and instead
+  # explicitly tag your specs with their type, e.g.:
+  #
+  #     RSpec.describe UsersController, :type => :controller do
+  #       # ...
+  #     end
+  #
+  # The different available types are documented in the features, such as in
+  # https://relishapp.com/rspec/rspec-rails/docs
+  config.infer_spec_type_from_file_location!
+
+  # Filter lines from Rails gems in backtraces.
+  config.filter_rails_from_backtrace!
+  # arbitrary gems may also be filtered via:
+  # config.filter_gems_from_backtrace("gem name")
+end

spec/spec_helper.rb

+# This file was generated by the `rails generate rspec:install` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause
+# this file to always be loaded, without a need to explicitly require it in any
+# files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need
+# it.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    #     be_bigger_than(2).and_smaller_than(4).description
+    #     # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #     # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+  # This option will default to `:apply_to_host_groups` in RSpec 4 (and will
+  # have no way to turn it off -- the option exists only for backwards
+  # compatibility in RSpec 3). It causes shared context metadata to be
+  # inherited by the metadata hash of host groups and examples, rather than
+  # triggering implicit auto-inclusion in groups with matching metadata.
+  config.shared_context_metadata_behavior = :apply_to_host_groups
+
+# The settings below are suggested to provide a good initial experience
+# with RSpec, but feel free to customize to your heart's content.
+=begin
+  # This allows you to limit a spec run to individual examples or groups
+  # you care about by tagging them with `:focus` metadata. When nothing
+  # is tagged with `:focus`, all examples get run. RSpec also provides
+  # aliases for `it`, `describe`, and `context` that include `:focus`
+  # metadata: `fit`, `fdescribe` and `fcontext`, respectively.
+  config.filter_run_when_matching :focus
+
+  # Allows RSpec to persist some state between runs in order to support
+  # the `--only-failures` and `--next-failure` CLI options. We recommend
+  # you configure your source control system to ignore this file.
+  config.example_status_persistence_file_path = "spec/examples.txt"
+
+  # Limits the available syntax to the non-monkey patched syntax that is
+  # recommended. For more details, see:
+  #   - http://rspec.info/blog/2012/06/rspecs-new-expectation-syntax/
+  #   - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://rspec.info/blog/2014/05/notable-changes-in-rspec-3/#zero-monkey-patching-mode
+  config.disable_monkey_patching!
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = "doc"
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+=end
+end

spec/support/factory_girl.rb

+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+end
\ No newline at end of file