solidus_br_common 1.1.0

data/spec/features/address_checkout_spec.rb

@@ -0,0 +1,89 @@
+require 'spec_helper'
+
+describe 'Checkout', type: :feature, inaccessible: true do
+  include_context 'checkout setup'
+
+  let!(:city) { create(:city, state: Spree::State.first!) }
+
+  context 'visitor makes checkout as guest without registration' do
+    before(:each) do
+      stock_location.stock_items.update_all(count_on_hand: 1)
+    end
+
+    context 'full checkout', :vcr do
+      before do
+        shipping_method.calculator.update!(preferred_amount: 10)
+        mug.shipping_category = shipping_method.shipping_categories.first
+        mug.save!
+      end
+
+      it 'does not break the per-item shipping method calculator', :js => true do
+        add_mug_to_cart
+        click_button 'Checkout'
+
+        fill_in 'order_email', :with => 'test@example.com'
+        click_button 'Continue'
+
+        fill_in_address
+        click_button 'Save and Continue'
+
+        expect(page).not_to have_content("undefined method `promotion'")
+        click_button 'Save and Continue'
+        expect(page).to have_content('Shipping total: $10.00')
+      end
+
+      context 'in Brazil' do
+        before do
+          brazil = create(:country, name: 'Brasil')
+          minas_gerais_state = create(:state, abbr: 'MG', name: 'Minas Gerais', country: brazil)
+          minas_gerais_city = create(:city, name: 'Minas Gerais', state: minas_gerais_state)
+          @sao_paulo_state = create(:state, abbr: 'SP', name: 'São Paulo', country: brazil)
+          @sao_paulo_city = create(:city, name: 'São Paulo', state: @sao_paulo_state)
+
+          Spree::Config[:default_country_id] = brazil.id
+        end
+
+        it 'should autocomplete zipcode', :js => true do
+          add_mug_to_cart
+          click_button 'Checkout'
+
+          fill_in 'order_email', :with => 'test@example.com'
+          click_button 'Continue'
+
+          address = 'order_bill_address_attributes'
+          fill_in "#{address}_zipcode", with: '04089000'
+
+          wait_for_ajax
+
+          expect(find_by_id("#{address}_address1").value).to eq('Alameda dos Maracatins')
+          expect(find_by_id("#{address}_district").value).to eq('Indianópolis')
+          expect(find_by_id("#{address}_state_id").value).to eq(@sao_paulo_state.id.to_s)
+          expect(find_by_id("#{address}_city_id").value).to eq(@sao_paulo_city.id.to_s)
+        end
+
+        after do
+          Spree::Config[:default_country_id] = nil
+        end
+      end
+    end
+  end
+
+  def fill_in_address
+    address = 'order_bill_address_attributes'
+    fill_in "#{address}_firstname", with: 'Ryan'
+    fill_in "#{address}_lastname", with: 'Bigg'
+    fill_in "#{address}_address1", with: 'Swan Street'
+    fill_in "#{address}_number", with: '143'
+    fill_in "#{address}_district", with: 'Carville'
+    select 'Alabama', from: "#{address}_state_id"
+    select city.name, from: "#{address}_city_id"
+    fill_in "#{address}_zipcode", with: '12345'
+    fill_in "#{address}_phone", with: '(555) 555-5555'
+  end
+
+  def add_mug_to_cart
+    visit spree.root_path
+    click_link mug.name
+    click_button 'add-to-cart-button'
+  end
+end

data/spec/features/user_account_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe 'Accounts', type: :feature, inaccessible: true, js: true do
+  def login_as(user)
+    visit spree.login_path
+    fill_in 'Email', with: user.email
+    fill_in 'Password', with: user.password
+    click_button 'Login'
+  end
+
+  let(:user) { create(:user) }
+
+  before do
+    login_as(user)
+  end
+
+  context 'editing' do
+    scenario 'should rename user with success' do
+      Spree::Auth::Config.set(signout_after_password_change: false)
+      click_link 'My Account'
+
+      expect(page).to have_text user.email
+      click_link 'Edit'
+      fill_in 'First Name', with: 'Diego'
+      fill_in 'Last Name', with: 'Domingues'
+      fill_in Spree.t(:date_of_birth), with: 19.years.ago.strftime(Spree.t('date_picker.format', default: '%Y/%m/%d'))
+      click_button 'Update'
+
+      expect(page).to have_text 'Diego Domingues'
+      expect(page).to have_text 'Account updated'
+    end
+  end
+
+  context 'showing' do
+    scenario 'should show details of user' do
+      click_link 'My Account'
+
+      expect(page).to have_text user.full_name
+      expect(page).to have_text user.cpf_formatted
+      expect(page).to have_text user.phone
+      expect(page).to have_text user.alternative_phone if user.alternative_phone.present?
+      expect(page).to have_text user.date_of_birth.to_date
+      expect(page).to have_text user.email
+    end
+  end
+end

data/spec/features/user_signup_spec.rb

@@ -0,0 +1,24 @@
+require 'spec_helper'
+
+describe 'Sign Up', type: :feature, inaccessible: true do
+  context 'with valid data' do
+    scenario 'create a user with success', js: true do
+      visit spree.signup_path
+
+      fill_in Spree.t(:first_name), with: 'Carlos'
+      fill_in Spree.t(:last_name), with: 'Andrade'
+      fill_in Spree.t(:cpf), with: '53229124367'
+      fill_in Spree.t(:date_of_birth), with: 18.years.ago.strftime(Spree.t('date_picker.format', default: '%Y/%m/%d'))
+      fill_in Spree.t(:email), with: 'email@person.com'
+      fill_in Spree.t(:phone), with: '11 12211221'
+      fill_in Spree.t(:alternative_phone), with: '11 12211221'
+      fill_in Spree.t(:password), with: 'password'
+      fill_in 'Password Confirmation', with: 'password'
+
+      click_button 'Create'
+
+      expect(page).to have_text 'You have signed up successfully.'
+      expect(Spree::User.count).to eq(1)
+    end
+  end
+end

data/spec/fixtures/vcr_cassettes/Checkout/visitor_makes_checkout_as_guest_without_registration/full_checkout/does_not_break_the_per-item_shipping_method_calculator.yml

@@ -0,0 +1,49 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: http://api.postmon.com.br/v1/cep/12345
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      Accept:
+      - "*/*; q=0.5, application/xml"
+      Accept-Encoding:
+      - gzip, deflate
+      User-Agent:
+      - Ruby
+  response:
+    status:
+      code: 404
+      message: Not Found
+    headers:
+      Server:
+      - nginx/1.1.19
+      Date:
+      - Tue, 12 May 2015 05:46:09 GMT
+      Content-Type:
+      - text/html; charset=UTF-8
+      Content-Length:
+      - '368'
+      Connection:
+      - keep-alive
+      Vary:
+      - Accept-Encoding
+      Content-Encoding:
+      - gzip
+    body:
+      encoding: ASCII-8BIT
+      string: !binary |-
+        H4sIAAAAAAAAA3VSTU/CQBC9+ytGSDyJ2/KRYFt7EEokQSRaDh6X7pQ2lu66
+        uxga439325pAi+xpZ+fNmzfz9grM8a6nL5PwfRXAU/i8gNX6cTGfQKdHyDwI
+        Z4RMw2md6d9ZhATLjn9VlSV6l9XXOkTKjmH1pFOdoR9IyaUDQ2sIS65hxvc5
+        80ida+KVLjIEXQh86Gg8aBIp1WliAMq28L2h0cdWllS9iGclfxcRXYh5rnsx
+        3aVZ4YCiuXJ/WvUbzop/6+M4dk1WMjSRLQ6geJYy6DLG3BZHfQRlLM23BjwS
+        Bxd2VG7T/C9qdxUSL4u+0PTY4JzRI9WyTvZPmgZ45Zyt/Sb2BTNMookU/huX
+        srgFnSBI/Nyj0shg/bowrmr/pmsN7t1Ea+EQkvGIZglX2hlbY4t82SRCQez+
+        YDiqccZs3XYRIrpXhpHmgJUkj4i2Bol+qTEuNTpQc51zl7CTLRzHNmNVP/QX
+        8ak9XOUCAAA=
+    http_version: 
+  recorded_at: Tue, 12 May 2015 05:37:46 GMT
+recorded_with: VCR 2.9.3

data/spec/fixtures/vcr_cassettes/Checkout/visitor_makes_checkout_as_guest_without_registration/full_checkout/in_Brazil/should_autocomplete_zipcode.yml

@@ -0,0 +1,46 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: http://api.postmon.com.br/v1/cep/04089000
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      Accept:
+      - "*/*; q=0.5, application/xml"
+      Accept-Encoding:
+      - gzip, deflate
+      User-Agent:
+      - Ruby
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - nginx/1.1.19
+      Date:
+      - Tue, 12 May 2015 05:51:31 GMT
+      Content-Type:
+      - application/json
+      Content-Length:
+      - '334'
+      Connection:
+      - keep-alive
+      Access-Control-Allow-Origin:
+      - "*"
+      Cache-Control:
+      - public, max-age=2592000
+      X-Cache:
+      - HIT
+    body:
+      encoding: UTF-8
+      string: '{"complemento": "at\u00e9 470 - lado par", "bairro": "Indian\u00f3polis",
+        "cidade": "S\u00e3o Paulo", "logradouro": "Alameda dos Maracatins", "estado_info":
+        {"area_km2": "248.222,362", "codigo_ibge": "35", "nome": "S\u00e3o Paulo"},
+        "cep": "04089000", "cidade_info": {"area_km2": "1521,101", "codigo_ibge":
+        "3550308"}, "estado": "SP"}'
+    http_version: 
+  recorded_at: Tue, 12 May 2015 05:43:08 GMT
+recorded_with: VCR 2.9.3

data/spec/fixtures/vcr_cassettes/Spree_AddressesController/_show/does_not_find_an_incorrect_cep.yml

@@ -0,0 +1,38 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: http://api.postmon.com.br/v1/cep/00000000
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      Accept:
+      - "*/*; q=0.5, application/xml"
+      Accept-Encoding:
+      - gzip, deflate
+      User-Agent:
+      - Ruby
+  response:
+    status:
+      code: 404
+      message: CEP 00000000 nao encontrado
+    headers:
+      Server:
+      - nginx/1.1.19
+      Date:
+      - Fri, 08 May 2015 02:39:13 GMT
+      Content-Type:
+      - application/json
+      Content-Length:
+      - '0'
+      Connection:
+      - keep-alive
+      Access-Control-Allow-Origin:
+      - "*"
+    body:
+      encoding: UTF-8
+      string: ''
+    http_version: 
+  recorded_at: Fri, 08 May 2015 02:30:57 GMT
+recorded_with: VCR 2.9.3

data/spec/fixtures/vcr_cassettes/Spree_AddressesController/_show/shows_the_correct_address.yml

@@ -0,0 +1,46 @@
+---
+http_interactions:
+- request:
+    method: get
+    uri: http://api.postmon.com.br/v1/cep/04089000
+    body:
+      encoding: US-ASCII
+      string: ''
+    headers:
+      Accept:
+      - "*/*; q=0.5, application/xml"
+      Accept-Encoding:
+      - gzip, deflate
+      User-Agent:
+      - Ruby
+  response:
+    status:
+      code: 200
+      message: OK
+    headers:
+      Server:
+      - nginx/1.1.19
+      Date:
+      - Fri, 08 May 2015 02:39:13 GMT
+      Content-Type:
+      - application/json
+      Content-Length:
+      - '334'
+      Connection:
+      - keep-alive
+      Access-Control-Allow-Origin:
+      - "*"
+      Cache-Control:
+      - public, max-age=2592000
+      X-Cache:
+      - MISS
+    body:
+      encoding: UTF-8
+      string: '{"complemento": "at\u00e9 470 - lado par", "bairro": "Indian\u00f3polis",
+        "cidade": "S\u00e3o Paulo", "logradouro": "Alameda dos Maracatins", "estado_info":
+        {"area_km2": "248.222,801", "codigo_ibge": "35", "nome": "S\u00e3o Paulo"},
+        "cep": "04089000", "cidade_info": {"area_km2": "1521,101", "codigo_ibge":
+        "3550308"}, "estado": "SP"}'
+    http_version: 
+  recorded_at: Fri, 08 May 2015 02:30:57 GMT
+recorded_with: VCR 2.9.3

data/spec/models/spree/address_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe Spree::Address do
+  let(:address) {build(:address)}
+
+  it { is_expected.to validate_presence_of(:number) }
+  it { is_expected.to validate_presence_of(:district) }
+
+  it 'should validate length of district' do
+    address.district = string_with_length(151)
+
+    expect(address).to_not be_valid
+  end
+
+  it 'should validate number negative' do
+    address.number = -1
+
+    expect(address).to_not be_valid
+  end
+
+  context '#save' do
+    subject { address }
+
+    before { address.save! }
+
+    it { is_expected.to be_persisted }
+  end
+
+  def string_with_length(length)
+    (1..length).map { (65 + rand(26)).chr }.join
+  end
+end

data/spec/models/spree/city_spec.rb

@@ -0,0 +1,7 @@
+require 'spec_helper'
+
+describe Spree::City do
+  it { is_expected.to validate_presence_of(:state) }
+  it { is_expected.to validate_presence_of(:name) }
+  it { is_expected.to validate_presence_of(:ibge_code) }
+end

data/spec/models/spree/user_spec.rb

@@ -0,0 +1,88 @@
+require 'spec_helper'
+
+describe Spree::User do
+  def string_with_length(length)
+    (1..length).map { (65 + rand(26)).chr }.join
+  end
+
+  let(:user) { build(:user) }
+
+  it {
+    is_expected.to validate_presence_of(:cpf)
+    is_expected.to validate_presence_of(:first_name)
+    is_expected.to validate_presence_of(:last_name)
+    is_expected.to validate_presence_of(:date_of_birth)
+    is_expected.to validate_presence_of(:phone)
+  }
+
+  context '#full_name' do
+    it 'should return first_name + last_name' do
+      full_name = "#{user.first_name} #{user.last_name}"
+
+      expect(user.full_name).to eq(full_name)
+    end
+  end
+
+  context '#cpf_formatted' do
+    it 'should return cpf formatted' do
+      user.cpf = '12345678910'
+
+      expect(user.cpf_formatted).to eql('123.456.789-10')
+    end
+  end
+
+  it 'should respond to extra properties' do
+    expect(user).to respond_to :first_name, :last_name, :cpf, :date_of_birth, :phone, :alternative_phone
+  end
+
+  it 'should save user with success' do
+    expect(user.save!).to be_truthy
+  end
+
+  it 'should save user with sanitize cpf' do
+    cpf_formatted = CPF.generate(true)
+    user.cpf = cpf_formatted
+    cpf = CPF.new(cpf_formatted)
+
+    user.save!
+
+    expect(user.cpf).to eq(cpf.stripped)
+  end
+
+  it 'should save user with sanitize phone' do
+    user.phone = '(011) 3882-8877'
+    user.save!
+
+    expect(user.phone).to eq('01138828877')
+  end
+
+  it 'should validate cpf' do
+    expect(user).to be_valid
+
+    user.cpf = '123456789'
+
+    expect(user).to_not be_valid
+
+    user.cpf = '12345678910'
+
+    expect(user).to_not be_valid
+  end
+
+  it 'should validate length of first_name' do
+    user.first_name = string_with_length(101)
+
+    expect(user).to_not be_valid
+  end
+
+  it 'should validate length of last_name' do
+    user.last_name = string_with_length(101)
+
+    expect(user).to_not be_valid
+  end
+
+  it 'should validate date_of_birth is greater or equal to 18 years' do
+    user.date_of_birth = 17.years.ago
+
+    expect(user).to_not be_valid
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,61 @@
+# Run Coverage report
+require 'codeclimate-test-reporter'
+CodeClimate::TestReporter.start
+
+# Configure Rails Environment
+ENV['RAILS_ENV'] = 'test'
+
+require File.expand_path('../dummy/config/environment.rb',  __FILE__)
+
+require 'rspec/rails'
+require 'database_cleaner'
+require 'ffaker'
+
+require 'capybara/rspec'
+require 'capybara-screenshot/rspec'
+
+if ENV['WEBDRIVER'] == 'accessible'
+  require 'capybara/accessible'
+  Capybara.javascript_driver = :accessible
+else
+  require 'capybara/poltergeist'
+  Capybara.javascript_driver = :poltergeist
+end
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.join(File.dirname(__FILE__), 'support/**/*.rb')].each { |f| require f }
+
+# Requires factories and other useful helpers defined in spree_core.
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/capybara_ext'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/factories'
+require 'spree/testing_support/url_helpers'
+
+require 'spree/api/testing_support/helpers'
+
+# Requires factories defined in lib/solidus_br_common/factories.rb
+require 'solidus_br_common/factories'
+
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+
+  # Infer an example group's spec type from the file location.
+  config.infer_spec_type_from_file_location!
+
+  # == URL Helpers
+  #
+  # Allows access to Spree's routes in specs:
+  #
+  # visit spree.admin_path
+  # current_path.should eql(spree.products_path)
+  config.include Spree::TestingSupport::UrlHelpers
+  config.include Spree::Api::TestingSupport::Helpers
+
+  config.mock_with :rspec
+  config.color = true
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+  config.fail_fast = ENV['FAIL_FAST'] || false
+  config.order = 'random'
+end

data/spec/support/controller_hacks.rb

@@ -0,0 +1,33 @@
+require 'active_support/all'
+module ControllerHacks
+  extend ActiveSupport::Concern
+
+  included do
+    routes { Spree::Core::Engine.routes }
+  end
+
+  def api_get(action, params={}, session=nil, flash=nil)
+    api_process(action, params, session, flash, "GET")
+  end
+
+  def api_post(action, params={}, session=nil, flash=nil)
+    api_process(action, params, session, flash, "POST")
+  end
+
+  def api_put(action, params={}, session=nil, flash=nil)
+    api_process(action, params, session, flash, "PUT")
+  end
+
+  def api_delete(action, params={}, session=nil, flash=nil)
+    api_process(action, params, session, flash, "DELETE")
+  end
+
+  def api_process(action, params={}, session=nil, flash=nil, method="get")
+    scoping = respond_to?(:resource_scoping) ? resource_scoping : {}
+    process(action, method, params.merge(scoping).reverse_merge!(:format => :json), session, flash)
+  end
+end
+
+RSpec.configure do |config|
+  config.include ControllerHacks, type: :controller
+end

data/spec/support/database_cleaner.rb

@@ -0,0 +1,24 @@
+RSpec.configure do |config|
+  # Capybara javascript drivers require transactional fixtures set to false, and we use DatabaseCleaner
+  # to cleanup after each test instead.  Without transactional fixtures set to false the records created
+  # to setup a test will be unavailable to the browser, which runs under a separate server instance.
+  config.use_transactional_fixtures = false
+
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with(:truncation)
+  end
+
+  # Before each spec check if it is a Javascript test and switch between using database transactions or not where necessary.
+  config.before(:each) do
+    DatabaseCleaner.strategy = RSpec.current_example.metadata[:js] ? :truncation : :transaction
+  end
+
+  config.before(:each) do
+    DatabaseCleaner.start
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+end

data/spec/support/shared_context/checkout_setup.rb

@@ -0,0 +1,9 @@
+shared_context 'checkout setup' do
+  let!(:country) { create(:country, states_required: true) }
+  let!(:state) { create(:state, country: country) }
+  let!(:shipping_method) { create(:shipping_method) }
+  let!(:stock_location) { create(:stock_location) }
+  let!(:mug) { create(:product, name: "RoR Mug") }
+  let!(:payment_method) { create(:check_payment_method) }
+  let!(:zone) { create(:zone) }
+end

data/spec/support/shoulda_matchers.rb

@@ -0,0 +1,6 @@
+Shoulda::Matchers.configure do |config|
+  config.integrate do |with|
+    with.test_framework :rspec
+    with.library :rails
+  end
+end

data/spec/support/vcr.rb

@@ -0,0 +1,8 @@
+require 'vcr'
+
+VCR.configure do |c|
+  c.cassette_library_dir = 'spec/fixtures/vcr_cassettes'
+  c.hook_into :webmock
+  c.configure_rspec_metadata!
+  c.ignore_hosts '127.0.0.1', 'localhost', 'codeclimate.com'
+end