genkan 0.2.1 → 0.2.2

data/spec/lib/genkan/config_spec.rb

@@ -1,58 +1,58 @@
-require 'rails_helper'
+require "rails_helper"
 
 RSpec.describe Genkan::Config do
   let(:config) { described_class.new }
 
-  describe 'initialize' do
+  describe "initialize" do
     it { expect(config.user_class_name).to eq "User" }
     it { expect(config.auto_acceptance).to eq false }
   end
 
-  describe '#user_class' do
+  describe "#user_class" do
     before { config.user_class_name = user_class_name }
     subject { config.user_class }
 
-    context 'user_class_name exist' do
+    context "user_class_name exist" do
       let(:user_class_name) { "User" }
-      it('returns constant') { is_expected.to be User }
+      it("returns constant") { is_expected.to be User }
     end
 
-    context 'user_class_name unexist' do
+    context "user_class_name unexist" do
       let(:user_class_name) { "Anonymous" }
       it { is_expected.to be_nil }
     end
   end
 
-  describe '#current_user_method_name' do
+  describe "#current_user_method_name" do
     before { config.user_class_name = user_class_name }
     subject { config.current_user_method_name }
 
-    context 'without namespace' do
+    context "without namespace" do
       let(:user_class_name) { "AdminUser" }
-      it('returns correctly method name') { is_expected.to eq "current_admin_user" }
+      it("returns correctly method name") { is_expected.to eq "current_admin_user" }
     end
 
-    context 'with namespace' do
+    context "with namespace" do
       let(:user_class_name) { "Admin::User" }
-      it('returns correctly method name') { is_expected.to eq "current_admin_user" }
+      it("returns correctly method name") { is_expected.to eq "current_admin_user" }
     end
   end
 
-  describe '#auto_acceptance?' do
+  describe "#auto_acceptance?" do
     before { config.auto_acceptance = auto_acceptance }
     subject { config.auto_acceptance? }
 
-    context 'auto_acceptance is true' do
+    context "auto_acceptance is true" do
       let(:auto_acceptance) { true }
       it { is_expected.to eq true }
     end
 
-    context 'auto_acceptance is false' do
+    context "auto_acceptance is false" do
       let(:auto_acceptance) { false }
       it { is_expected.to eq false }
     end
 
-    context 'auto_acceptance is nil' do
+    context "auto_acceptance is nil" do
       let(:auto_acceptance) { nil }
       it { is_expected.to eq false }
     end

data/spec/lib/genkan_spec.rb

@@ -1,7 +1,7 @@
-require 'rails_helper'
+require "rails_helper"
 
 RSpec.describe Genkan do
-  describe '.configure' do
+  describe ".configure" do
     example do
       expect {
         described_class.configure do |config|
@@ -11,12 +11,12 @@ RSpec.describe Genkan do
     end
   end
 
-  describe '.config' do
+  describe ".config" do
     subject { described_class.config }
     it { is_expected.to be_an_instance_of Genkan::Config }
   end
 
-  describe 'version' do
+  describe "version" do
     subject { described_class::VERSION }
     it { is_expected.to be_a String }
   end

data/spec/models/user_spec.rb

@@ -1,94 +1,94 @@
-require 'rails_helper'
+require "rails_helper"
 
 RSpec.describe User do
-  it 'has a valid factory' do
+  it "has a valid factory" do
     expect(build(:user)).to be_valid
   end
 
-  describe 'scope' do
+  describe "scope" do
     let!(:accepted)     { create :user, :accepted }
     let!(:not_accepted) { create :user, :not_accepted }
     let!(:banned)       { create :user, :banned }
     let!(:not_banned)   { create :user, :not_banned }
     let!(:active)       { create :user, :active }
 
-    describe '.accepted' do
+    describe ".accepted" do
       subject { described_class.accepted }
 
-      it('includes accepted') { is_expected.to include accepted }
-      it('includes active') { is_expected.to include active }
-      it('does not include not_accepted') { is_expected.not_to include not_accepted }
+      it("includes accepted") { is_expected.to include accepted }
+      it("includes active") { is_expected.to include active }
+      it("does not include not_accepted") { is_expected.not_to include not_accepted }
     end
 
-    describe '.not_banned' do
+    describe ".not_banned" do
       subject { described_class.not_banned }
 
-      it('includes not_banned') { is_expected.to include not_banned }
-      it('includes active') { is_expected.to include active }
-      it('does not include banned') { is_expected.not_to include banned }
+      it("includes not_banned") { is_expected.to include not_banned }
+      it("includes active") { is_expected.to include active }
+      it("does not include banned") { is_expected.not_to include banned }
     end
 
-    describe '.active' do
+    describe ".active" do
       subject { described_class.active }
 
-      it('includes active') { is_expected.to include active }
-      it('does not include not_accepted') { is_expected.not_to include not_accepted }
-      it('does not include not_banned') { is_expected.not_to include not_banned }
+      it("includes active") { is_expected.to include active }
+      it("does not include not_accepted") { is_expected.not_to include not_accepted }
+      it("does not include not_banned") { is_expected.not_to include not_banned }
     end
   end
 
-  describe 'validation' do
-    describe 'email' do
+  describe "validation" do
+    describe "email" do
       before { create :user }
 
       it { is_expected.to validate_presence_of(:email) }
       it { is_expected.to validate_uniqueness_of(:email) }
     end
 
-    describe '#ensure_record_has_accepted' do
+    describe "#ensure_record_has_accepted" do
       before { model.valid?(:login) }
 
       subject { model.errors.full_messages }
 
-      context 'with accepted' do
+      context "with accepted" do
         let(:model) { build :user, :accepted }
-        it { is_expected.not_to include I18n.t('errors.messages.not_accepted') }
+        it { is_expected.not_to include I18n.t("errors.messages.not_accepted") }
       end
 
-      context 'with not_accepted' do
+      context "with not_accepted" do
         let(:model) { build :user, :not_accepted }
-        it { is_expected.to include I18n.t('errors.messages.not_accepted') }
+        it { is_expected.to include I18n.t("errors.messages.not_accepted") }
       end
     end
 
-    describe '#ensure_record_has_not_banned' do
+    describe "#ensure_record_has_not_banned" do
       before { model.valid?(:login) }
 
       subject { model.errors.full_messages }
 
-      context 'with not_banned' do
+      context "with not_banned" do
         let(:model) { build :user, :not_banned }
-        it { is_expected.not_to include I18n.t('errors.messages.banned') }
+        it { is_expected.not_to include I18n.t("errors.messages.banned") }
       end
 
-      context 'with banned' do
+      context "with banned" do
         let(:model) { build :user, :banned }
-        it { is_expected.to include I18n.t('errors.messages.banned') }
+        it { is_expected.to include I18n.t("errors.messages.banned") }
       end
     end
   end
 
-  describe 'callback' do
+  describe "callback" do
     let(:user) { build(:user) }
 
-    it 'set remember_token before create' do
+    it "set remember_token before create" do
       expect(user.remember_token).to be_blank
       user.save!
       expect(user.remember_token).to be_present
     end
   end
 
-  describe '#login!' do
+  describe "#login!" do
     let(:logged_in_count) { rand(1..20) }
     let(:user) { build :user, logged_in_count: logged_in_count }
 
@@ -99,44 +99,45 @@ RSpec.describe User do
       user.login!
     end
 
-    context 'when auto_acceptance is true' do
+    context "when auto_acceptance is true" do
       let(:auto_acceptance) { true }
 
       before { execute }
 
-      it 'stores last_logged_in_at' do
+      it "stores last_logged_in_at" do
         expect(user.last_logged_in_at).to be_present
       end
 
-      it 'increments loggin_in_count' do
-        expect(user.logged_in_count).to eq (logged_in_count + 1)
+      it "increments loggin_in_count" do
+        result = logged_in_count + 1
+        expect(user.logged_in_count).to eq result
       end
     end
 
-    context 'when auto_acceptance is false' do
+    context "when auto_acceptance is false" do
       let(:auto_acceptance) { false }
 
-      it 'raise error ActiveRecord::RecordInvalid' do
+      it "raise error ActiveRecord::RecordInvalid" do
         expect { execute }.to raise_error(ActiveRecord::RecordInvalid)
       end
     end
   end
 
-  describe '#accepted?' do
+  describe "#accepted?" do
     subject { user.accepted? }
 
-    context 'with accepted' do
+    context "with accepted" do
       let(:user) { build :user, :accepted }
       it { is_expected.to eq true }
     end
 
-    context 'with not_accepted' do
+    context "with not_accepted" do
       let(:user) { build :user, :not_accepted }
       it { is_expected.to eq false }
     end
   end
 
-  describe '#accept' do
+  describe "#accept" do
     let(:user) { build :user }
 
     subject { user.tap(&:accept) }
@@ -144,7 +145,7 @@ RSpec.describe User do
     it { is_expected.to be_accepted }
   end
 
-  describe '#accept!' do
+  describe "#accept!" do
     let(:user) { build :user }
 
     subject { user.tap(&:accept!) }
@@ -153,21 +154,21 @@ RSpec.describe User do
     it { is_expected.to be_persisted }
   end
 
-  describe '#banned?' do
+  describe "#banned?" do
     subject { user.banned? }
 
-    context 'with banned' do
+    context "with banned" do
       let(:user) { build :user, :banned }
       it { is_expected.to eq true }
     end
 
-    context 'with not_banned' do
+    context "with not_banned" do
       let(:user) { build :user, :not_banned }
       it { is_expected.to eq false }
     end
   end
 
-  describe '#ban' do
+  describe "#ban" do
     let(:user) { build :user }
 
     subject { user.tap(&:ban) }
@@ -175,7 +176,7 @@ RSpec.describe User do
     it { is_expected.to be_banned }
   end
 
-  describe '#ban!' do
+  describe "#ban!" do
     let(:user) { build :user }
 
     subject { user.tap(&:ban!) }

data/spec/rails_helper.rb

@@ -1,9 +1,9 @@
-require 'spec_helper'
-ENV['RAILS_ENV'] ||= 'test'
-require File.expand_path('../dummy/config/environment', __FILE__)
-require 'rspec/rails'
-require 'capybara/rails'
-require 'capybara/rspec'
+require "spec_helper"
+ENV["RAILS_ENV"] ||= "test"
+require File.expand_path("../dummy/config/environment", __FILE__)
+require "rspec/rails"
+require "capybara/rails"
+require "capybara/rspec"
 
 RSpec.configure do |config|
   config.include FactoryGirl::Syntax::Methods
@@ -44,6 +44,6 @@ OmniAuth.config.test_mode = true
 
 def stub_omniauth(email: Faker::Internet.email)
   OmniAuth.config.mock_auth[:google] = OmniAuth::AuthHash.new(
-    "provider" => "google", "info" => { "email" => email }
+    "provider" => "google", "info" => { "email" => email },
   )
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: genkan
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - Yoshiyuki Hirano
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-09 00:00:00.000000000 Z
+date: 2017-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails