spree_zaez_brazilian_fields 3.0.0 → 3.0.1

data/lib/generators/spree_zaez_brazilian_fields/install/install_generator.rb

@@ -0,0 +1,31 @@
+module SpreeZaezBrazilianFields
+  module Generators
+    class InstallGenerator < Rails::Generators::Base
+
+      class_option :auto_run_migrations, :type => :boolean, :default => false
+
+      def add_javascripts
+        append_file 'vendor/assets/javascripts/spree/frontend/all.js', "//= require spree/frontend/spree_zaez_brazilian_fields\n"
+        append_file 'vendor/assets/javascripts/spree/backend/all.js', "//= require spree/backend/spree_zaez_brazilian_fields\n"
+      end
+
+      def add_stylesheets
+        inject_into_file 'vendor/assets/stylesheets/spree/frontend/all.css', " *= require spree/frontend/spree_zaez_brazilian_fields\n", :before => /\*\//, :verbose => true
+        inject_into_file 'vendor/assets/stylesheets/spree/backend/all.css', " *= require spree/backend/spree_zaez_brazilian_fields\n", :before => /\*\//, :verbose => true
+      end
+
+      def add_migrations
+        run 'bundle exec rake railties:install:migrations FROM=spree_zaez_brazilian_fields'
+      end
+
+      def run_migrations
+        run_migrations = options[:auto_run_migrations] || ['', 'y', 'Y'].include?(ask 'Would you like to run the migrations now? [Y/n]')
+        if run_migrations
+          run 'bundle exec rake db:migrate'
+        else
+          puts 'Skipping rake db:migrate, don\'t forget to run it!'
+        end
+      end
+    end
+  end
+end

data/lib/spree_zaez_brazilian_fields.rb

@@ -0,0 +1,7 @@
+require 'spree_core'
+require 'spree_zaez_brazilian_fields/engine'
+require 'cpf_cnpj'
+
+# Permitted attributes for users
+Spree::PermittedAttributes.user_attributes.push :name, :account_type, :newsletter, :receive_sms, :cpf, :rg,
+                                                :birth_date, :gender, :cnpj, :state_registry, :contact

data/lib/spree_zaez_brazilian_fields/engine.rb

@@ -0,0 +1,20 @@
+module SpreeBrazilianFields
+  class Engine < Rails::Engine
+    require 'spree/core'
+    isolate_namespace Spree
+    engine_name 'spree_zaez_brazilian_fields'
+
+    # use rspec for tests
+    config.generators do |g|
+      g.test_framework :rspec
+    end
+
+    def self.activate
+      Dir.glob(File.join(File.dirname(__FILE__), '../../app/**/*_decorator*.rb')) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+    end
+
+    config.to_prepare &method(:activate).to_proc
+  end
+end

data/lib/spree_zaez_brazilian_fields/factories.rb

@@ -0,0 +1,25 @@
+FactoryGirl.define do
+
+  factory :personal_account, class: Spree.user_class do
+    email { generate(:random_email) }
+    login { email }
+    password 'secret'
+    password_confirmation { password }
+    account_type 'personal'
+    cpf CPF.generate(true)
+    rg '123'
+    birth_date Date.today
+    gender 'f'
+  end
+
+  factory :business_account, class: Spree.user_class do
+    email { generate(:random_email) }
+    login { email }
+    password 'secret'
+    password_confirmation { password }
+    account_type 'business'
+    cnpj CNPJ.generate(true)
+    state_registry '123'
+    contact 'some name'
+  end
+end

data/spec/features/account_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe 'Account user', type: :feature do
+  context 'show brazilian fields' do
+
+    # login into user page
+    before do
+      @user = Spree.user_class.create(email: 'user@test.com',
+                                      password: 'password',
+                                      password_confirmation: 'password',
+                                      name: 'users name',
+                                      cpf: CPF.generate(true),
+                                      rg: '123',
+                                      birth_date: Date.today,
+                                      gender: 'f')
+      @role = Spree::Role.create(name: 'user')
+      @role.users << @user
+      sign_in_as! @user
+    end
+
+    it 'should show the brazilian fields in users#show', js: true do
+      visit spree.account_path
+
+      expect(page).to have_text @user.name
+      expect(page).to have_text @user.cpf
+      expect(page).to have_text @user.rg
+      expect(page).to have_text @user.birth_date.strftime(I18n.t('date.formats.default'))
+      expect(page).to have_text 'Female'
+    end
+  end
+end

data/spec/features/admin/general_settings_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+describe 'General Settings', type: :feature do
+
+  # login into admin page
+  before do
+    @admin = Spree.user_class.create(email: 'admin@admin.com', password: 'password', password_confirmation: 'password')
+    @role = Spree::Role.create(name: 'admin')
+    @role.users << @admin
+    sign_in_admin! @admin
+  end
+
+  context 'user registration settings' do
+    it 'should show the attribute sms permission', js: true do
+      visit spree.edit_admin_general_settings_path
+
+      expect(page).to have_text Spree.t(:user_registration_settings)
+      expect(page).to have_selector '#sms_permission'
+    end
+
+    it 'should possible check/uncheck the sms permission', js: true do
+      visit spree.edit_admin_general_settings_path
+
+      find(:css, '#sms_permission').set false
+
+      click_button 'Update'
+
+      expect(Spree::Config.sms_permission).to be(false)
+      expect(find_field('sms_permission')).not_to be_checked
+    end
+  end
+end

data/spec/features/admin/users_spec.rb

@@ -0,0 +1,150 @@
+require 'spec_helper'
+
+describe 'Users', type: :feature do
+
+  # login into admin page
+  before do
+    @admin = Spree.user_class.create(email: 'admin@admin.com', password: 'password', password_confirmation: 'password')
+    @role = Spree::Role.create(name: 'admin')
+    @role.users << @admin
+    sign_in_admin! @admin
+  end
+
+  context 'adding brazilian fields to users form' do
+
+    it 'should show common brazilian fields in users form', js: true do
+      Spree::Config.sms_permission = true
+
+      visit spree.new_admin_user_path
+
+      expect(page).to have_selector '#user_account_type_personal'
+      expect(page).to have_selector '#user_account_type_business'
+      expect(page).to have_selector '#user_newsletter'
+      expect(page).to have_selector '#user_receive_sms'
+    end
+
+    context 'disable sms permission' do
+      it 'should hide if the sms permission is false' do
+        Spree::Config.sms_permission = false
+
+        visit spree.new_admin_user_path
+
+        expect(page).not_to have_selector '#user_receive_sms'
+
+        # set default
+        Spree::Config.sms_permission = true
+      end
+    end
+
+    context 'choosing account type' do
+      it 'should show only personal account fields', js: true do
+        visit spree.new_admin_user_path
+
+        find(:css, '#user_account_type_personal').set true
+
+        expect(page).to have_text Spree.t(:name).upcase
+        expect(page).to have_selector '#user_cpf'
+        expect(page).to have_selector '#user_rg'
+        expect(page).to have_selector '#user_birth_date'
+        expect(page).to have_selector '#user_gender_f'
+        expect(page).to have_selector '#user_gender_m'
+
+        expect(page).not_to have_selector '#user_cnpj'
+        expect(page).not_to have_selector '#user_state_registry'
+        expect(page).not_to have_selector '#user_contact'
+      end
+
+      it 'should show only business account fields', js: true do
+        visit spree.new_admin_user_path
+
+        find(:css, '#user_account_type_business').set true
+
+        expect(page).to have_text Spree.t(:corporate_name).upcase
+        expect(page).to have_selector '#user_cnpj'
+        expect(page).to have_selector '#user_state_registry'
+        expect(page).to have_selector '#user_contact'
+
+        expect(page).not_to have_selector '#user_cpf'
+        expect(page).not_to have_selector '#user_rg'
+        expect(page).not_to have_selector '#user_birth_date'
+        expect(page).not_to have_selector '#user_gender_f'
+        expect(page).not_to have_selector '#user_gender_m'
+      end
+    end
+
+    context 'saving brazilian fields' do
+      context 'personal account' do
+        it 'can edit the personal account', js: true do
+          # gera um cnpj valido
+          cpf = CPF.generate true
+
+          Spree::Config.sms_permission = true
+
+          visit spree.edit_admin_user_path @admin.id
+
+          find(:css, '#user_account_type_personal').set true
+
+          fill_in 'user_name', with: 'users name'
+          find(:css, '#user_newsletter').set true
+          find(:css, '#user_receive_sms').set true
+          # preenche o campo cpf por js
+          # por causa da mascara
+          page.execute_script "$('#user_cpf').val('#{cpf}');"
+          fill_in 'user_rg', with: '123'
+          fill_in 'user_birth_date', with: '1990/01/01'
+          find(:css, '#user_gender_m').set true
+          click_button 'Update'
+
+          # verificando o objeto
+          @admin.reload
+          expect(@admin.name).to eq('users name')
+          expect(@admin.cpf).to eq cpf
+          expect(@admin.rg).to eq '123'
+          expect(@admin.birth_date).to eq Date.parse('1990/01/01')
+          expect(@admin.gender).to eq 'm'
+          expect(@admin.receive_sms).to eq 1
+          expect(@admin.newsletter).to eq 1
+          # verificando o formulario
+          expect(find_field('user_name').value).to eq 'users name'
+          expect(find_field('user_cpf').value).to eq cpf
+          expect(find_field('user_rg').value).to eq '123'
+          expect(find_field('user_birth_date').value).to eq '1990/01/01'
+          expect(find_field('user_gender_m')).to be_checked
+          expect(find_field('user_newsletter')).to be_checked
+          expect(find_field('user_receive_sms')).to be_checked
+        end
+      end
+
+      context 'business account' do
+        it 'can edit the business account', js: true do
+          # gera um cnpj valido
+          cnpj = CNPJ.generate true
+
+          visit spree.edit_admin_user_path @admin.id
+
+          find(:css, '#user_account_type_business').set true
+
+          fill_in 'user_name', with: 'users name'
+          # preenche o campo cnpj por js
+          # por causa da mascara
+          page.execute_script "$('#user_cnpj').val('#{cnpj}');"
+          fill_in 'user_state_registry', with: '123'
+          fill_in 'user_contact', with: 'company contact'
+          click_button 'Update'
+
+          # verificando o objeto
+          @admin.reload
+          expect(@admin.name).to eq 'users name'
+          expect(@admin.cnpj).to eq cnpj
+          expect(@admin.state_registry).to eq '123'
+          expect(@admin.contact).to eq 'company contact'
+          # verificando o formulario
+          expect(find_field('user_name').value).to eq 'users name'
+          expect(find_field('user_cnpj').value).to eq cnpj
+          expect(find_field('user_state_registry').value).to eq '123'
+          expect(find_field('user_contact').value).to eq 'company contact'
+        end
+      end
+    end
+  end
+end

data/spec/features/sign_up_spec.rb

@@ -0,0 +1,94 @@
+require 'spec_helper'
+
+describe 'Sign Up', type: :feature do
+  context 'brazilian fields' do
+
+    it 'should show common brazilian fields', js: true do
+      # set sms permission
+      Spree::Config.sms_permission = true
+
+      visit spree.signup_path
+
+      expect(page).to have_selector '#spree_user_account_type_personal'
+      expect(page).to have_selector '#spree_user_account_type_business'
+      expect(page).to have_selector '#spree_user_newsletter'
+      expect(page).to have_selector '#spree_user_receive_sms'
+    end
+
+    context 'disable sms permission' do
+      it 'should hide if the sms permission is false', js: true do
+        Spree::Config.sms_permission = false
+
+        visit spree.signup_path
+
+        expect(page).not_to have_selector '#spree_user_receive_sms'
+
+        # set default
+        Spree::Config.sms_permission = true
+      end
+    end
+
+    context 'choosing account type' do
+      it 'should show only personal account fields', js: true do
+        visit spree.signup_path
+
+        find(:css, '#spree_user_account_type_personal').set true
+
+        expect(page).to have_selector '#spree_user_cpf'
+        expect(page).to have_selector '#spree_user_rg'
+        expect(page).to have_selector '#spree_user_birth_date'
+        expect(page).to have_selector '#spree_user_gender_f'
+        expect(page).to have_selector '#spree_user_gender_m'
+
+        expect(page).not_to have_selector '#spree_user_cnpj'
+        expect(page).not_to have_selector '#spree_user_state_registry'
+        expect(page).not_to have_selector '#spree_user_contact'
+      end
+
+      it 'should show only business account fields', js: true do
+        visit spree.signup_path
+
+        find(:css, '#spree_user_account_type_business').set true
+
+        expect(page).to have_selector '#spree_user_cnpj'
+        expect(page).to have_selector '#spree_user_state_registry'
+        expect(page).to have_selector '#spree_user_contact'
+
+        expect(page).not_to have_selector '#spree_user_cpf'
+        expect(page).not_to have_selector '#spree_user_rg'
+        expect(page).not_to have_selector '#spree_user_birth_date'
+        expect(page).not_to have_selector '#spree_user_gender_f'
+        expect(page).not_to have_selector '#spree_user_gender_m'
+      end
+    end
+
+    context 'creating a new user' do
+      it 'should create a new user', js: true do
+        # gera um cnpj valido
+        cnpj = CNPJ.generate true
+
+        visit spree.signup_path
+
+        find(:css, '#spree_user_account_type_business').set true
+
+        fill_in 'Corporate Name', with: 'some name'
+        fill_in 'Email', with: 'email@person.com'
+        fill_in 'Password', with: 'password'
+        fill_in 'Password Confirmation', with: 'password'
+        # preenche o campo cnpj por js
+        # por causa da mascara
+        page.execute_script '$("#spree_user_cnpj").val("' + cnpj + '");'
+        fill_in 'State registry', with: '123'
+        fill_in 'Contact', with: 'contact'
+        click_button 'Create'
+
+        new_user = Spree::User.last
+
+        expect(new_user.business_account?).to be(true)
+        expect(new_user.cnpj).to eq(cnpj)
+        expect(new_user.state_registry).to eq('123')
+        expect(new_user.contact).to eq('contact')
+      end
+    end
+  end
+end

data/spec/models/spree/app_configuration_decorator_spec.rb

@@ -0,0 +1,9 @@
+require 'spec_helper'
+
+describe Spree::AppConfiguration, type: :model do
+
+  it 'attribute sms permission should exist' do
+    expect(Spree::Config.has_preference?(:sms_permission)).to be true
+  end
+
+end

data/spec/models/spree/user_decorator_spec.rb

@@ -0,0 +1,97 @@
+require 'spec_helper'
+
+describe Spree::User do
+
+  context 'validate brazilian fields' do
+
+    let(:user) { FactoryGirl.build(:user) }
+
+    it 'should be invalid if the account type has a invalid value' do
+      # values for account type:
+      # personal or business
+      user.account_type = ''
+      expect(user.valid?).to be(false)
+    end
+
+    it 'should be invalid if cpf is invalid' do
+      user.cpf = '1234'
+      expect(user.valid?).to be false
+    end
+
+    it 'should be invalid if the user has the same CPF of other user' do
+      # gera um cnpj valido
+      cpf = CPF.generate true
+
+      user.cpf = cpf
+      user.save
+
+      new_user = FactoryGirl.build(:user)
+      new_user.cpf = cpf
+
+      expect(new_user.valid?).to be(false)
+    end
+
+    it 'should be invalid if cnpj is invalid' do
+      user.cnpj = '1234'
+      expect(user.valid?).to be false
+    end
+
+    it 'should be invalid if the user has the same CNPJ of other user' do
+      # gera um cnpj valido
+      cnpj = CNPJ.generate true
+
+      user.cnpj = cnpj
+      user.save
+
+      new_user = FactoryGirl.build(:user)
+      new_user.cnpj = cnpj
+
+      expect(new_user.valid?).to be(false)
+    end
+
+    context 'personal or business account' do
+      let(:user) { Spree::User.new }
+
+      it 'the default account type should be personal' do
+        expect(user.account_type).to eq('personal')
+      end
+
+      it 'verify if the user has a personal account' do
+        expect(user.personal_account?).to be(true)
+      end
+
+      it 'verify if the user has a business account' do
+        user.account_type = 'business'
+        expect(user.business_account?).to be(true)
+      end
+    end
+
+    context 'document' do
+      let(:user) { Spree::User.new }
+
+      it 'should return the CPF in method document if personal account' do
+        user.cpf = CPF.generate true
+        expect(user.document).to eq user.cpf
+      end
+
+      it 'should return the CNPJ in method document if business account' do
+        user.account_type = 'business'
+        user.cnpj = CNPJ.generate true
+        expect(user.document).to eq user.cnpj
+      end
+    end
+
+    context 'overriding attributes methods' do
+      let(:user) { Spree::User.new }
+
+      it 'should return document in the list of attribute names' do
+        expect(user.attribute_names.include?('document')).to be true
+      end
+
+      it 'should return the document in method attributes' do
+        user.cpf = CPF.generate true
+        expect(user.attributes['document']).to eq user.document
+      end
+    end
+  end
+end