baza_models 0.0.13 → 0.0.14

data/lib/baza_models/query/inspector.rb

@@ -36,6 +36,7 @@ private
 
     relationship_pair = @model.relationships.detect { |key, _value| key == argument }
     raise "Could not find a relationship on #{@model.name} by that name: #{argument}" unless relationship_pair
+
     relationship = relationship_pair[1]
 
     table_name = relationship.fetch(:table_name)
@@ -60,6 +61,7 @@ private
 
       relationship_pair = @model.relationships.detect { |relationship_name, _relationship| relationship_name == key }
       raise "Could not find a relationship on #{@model.name} by that name: #{value}" unless relationship_pair
+
       relationship = relationship_pair[1]
 
       BazaModels::Query::Inspector.new(

data/lib/baza_models/query/pagination.rb

@@ -1,10 +1,8 @@
 module BazaModels::Query::Pagination
   def current_page
-    if @page
-      @page
-    else
-      raise "Page has not been set"
-    end
+    raise "Page has not been set" unless @page
+
+    @page
   end
 
   def out_of_bounds?
@@ -44,11 +42,11 @@ module BazaModels::Query::Pagination
   end
 
   def total_pages
-    pages_count = (count.to_f / per.to_f)
+    pages_count = (count.to_f / per)
 
     pages_count = 1 if pages_count.nan? || pages_count == Float::INFINITY
     pages_count = pages_count.ceil
-    pages_count = 1 if pages_count == 0
+    pages_count = 1 if pages_count.zero?
     pages_count
   end
 end

data/lib/baza_models/query/sql_generator.rb

@@ -9,7 +9,7 @@ class BazaModels::Query::SqlGenerator
     end
   end
 
-  def to_sql
+  def to_sql # rubocop:disable Metrics/CyclomaticComplexity, Metrics/AbcSize, Metrics/PerceivedComplexity
     sql = "SELECT "
 
     if @selects.empty?
@@ -68,9 +68,9 @@ class BazaModels::Query::SqlGenerator
         end
 
         if @reverse_order
-          if order =~ /\s+desc/i
+          if /\s+desc/i.match?(order)
             order = order.gsub(/\s+desc/i, " ASC")
-          elsif order =~ /\s+asc/i
+          elsif /\s+asc/i.match?(order)
             order = order.gsub(/\s+asc/i, " DESC")
           else
             order = "#{order} DESC"

data/lib/baza_models/ransacker.rb

@@ -37,8 +37,10 @@ private
       elsif key.to_s == "s"
         match = value.to_s.match(/\A([A-z_\d]+)\s+(asc|desc)\Z/)
         raise "Couldn't sort-match: #{value}" unless match
+
         sort_by(column_name: match[1], sort_mode: match[2])
-      elsif ransackable_scopes && ransackable_scopes.include?(key.to_s)
+      elsif
+ransackable_scopes&.include?(key.to_s)
         @query = @query.__send__(key, value)
       end
     end

data/lib/baza_models/ransacker/relationship_scanner.rb

@@ -29,6 +29,7 @@ private
 
     loop do
       break if @name_parts.empty?
+
       name_part = @name_parts.shift
       current_name << name_part
 
@@ -71,6 +72,7 @@ private
 
     loop do
       break if join_parts.empty?
+
       join_part = join_parts.shift
 
       if join_parts.length == 1
@@ -96,6 +98,7 @@ private
     case @mode
     when :cont
       return if @value.empty?
+
       add_query_with_symbol("LIKE", "%#{@klass.db.esc(@value)}%")
     when :eq
       add_query_with_symbol("=")

data/lib/baza_models/validators/confirmation_validator.rb

@@ -3,9 +3,7 @@ class BazaModels::Validators::ConfirmationValidator < BazaModels::Validators::Ba
     confirmation_attribute_name = "#{attribute_name}_confirmation"
     confirmation_value = model.__send__(confirmation_attribute_name)
 
-    if value && !confirmation_value
-      model.errors.add(attribute_name, "hasn't been confirmed")
-    end
+    model.errors.add(attribute_name, "hasn't been confirmed") if value && !confirmation_value
 
     model.errors.add(attribute_name, "was not the same as the confirmation") if value && confirmation_value && confirmation_value != value
   end

data/lib/baza_models/validators/uniqueness_validator.rb

@@ -2,10 +2,8 @@ class BazaModels::Validators::UniquenessValidator < BazaModels::Validators::Base
   def validate(model, value)
     query_same = model.class.where(attribute_name => value)
 
-    if scope
-      scope.each do |scope_part|
-        query_same = query_same.where(scope_part => model.__send__(scope_part))
-      end
+    scope&.each do |scope_part|
+      query_same = query_same.where(scope_part => model.__send__(scope_part))
     end
 
     model.errors.add(attribute_name, "isn't unique") if query_same.any?

data/peak_flow.yml

@@ -1,3 +1,4 @@
+rvm: true
 script:
   - bundle exec rspec
   - bundle exec rake best_practice_project:run

data/spec/baza_models/autoloader_spec.rb

@@ -1,6 +1,6 @@
 require "spec_helper"
 
-describe BazaModels::Autoloader do
+describe BazaModels::Autoloader do # rubocop:disable RSpec/MultipleMemoizedHelpers
   include DatabaseHelper
 
   let!(:organization) { Organization.create! }

data/spec/baza_models/baza_orm_adapter_spec.rb

@@ -47,6 +47,6 @@ describe BazaModels::Query do
   end
 
   it "#column_names" do
-    expect(User.to_adapter.column_names).to eq %w(id organization_id email email_confirmation created_at updated_at admin)
+    expect(User.to_adapter.column_names).to eq %w[id organization_id email email_confirmation created_at updated_at admin]
   end
 end

data/spec/baza_models/model/has_many_relations_spec.rb

@@ -45,7 +45,7 @@ describe BazaModels::Model::HasManyRelations do
     end
   end
 
-  context "#<<" do
+  describe "#<<" do
     it "adds models to persisted parent" do
       organization.save!
       organization.users << User.create!(email: "test@example.com")

data/spec/baza_models/model/manipulation_spec.rb

@@ -10,17 +10,17 @@ describe BazaModels::Model::Manipulation do
   it "#created_at" do
     expect(user.created_at).to eq nil
     user.save!
-    expect(user.created_at).to_not eq nil
+    expect(user.created_at).not_to eq nil
   end
 
   it "#updated_at" do
     expect(user.updated_at).to eq nil
     user.save!
-    expect(user.updated_at).to_not eq nil
+    expect(user.updated_at).not_to eq nil
     old_updated_at = user.updated_at
     sleep 1
     user.email = "test2@example.com"
     user.save!
-    expect(user.updated_at).to_not eq old_updated_at
+    expect(user.updated_at).not_to eq old_updated_at
   end
 end

data/spec/baza_models/model_spec.rb

@@ -22,8 +22,8 @@ describe "BazaModels::Model" do
     expect(user.id).to eq nil
     expect(user.to_param).to eq nil
     user.save!
-    expect(user.id).to_not eq nil
-    expect(user.to_param).to_not eq nil
+    expect(user.id).not_to eq nil
+    expect(user.to_param).not_to eq nil
     expect(user.to_param).to eq user.id.to_s
   end
 
@@ -62,12 +62,30 @@ describe "BazaModels::Model" do
     end
   end
 
+  it "#update" do
+    user.save!
+    expect(user.update(email: "newemail@example.com")).to eq true
+    expect(user.email).to eq "newemail@example.com"
+  end
+
+  it "#update!" do
+    user.save!
+    user.update!(email: "newemail@example.com")
+    expect(user.email).to eq "newemail@example.com"
+  end
+
   it "#update_attributes" do
     user.save!
     expect(user.update_attributes(email: "newemail@example.com")).to eq true
     expect(user.email).to eq "newemail@example.com"
   end
 
+  it "#update_attributes!" do
+    user.save!
+    user.update_attributes!(email: "newemail@example.com")
+    expect(user.email).to eq "newemail@example.com"
+  end
+
   it "#before_save, #after_save" do
     expect(user.before_save_called).to eq nil
     expect(user.after_save_called).to eq nil
@@ -162,7 +180,7 @@ describe "BazaModels::Model" do
   end
 
   it "#attribute_names" do
-    expect(User.attribute_names).to eq %w(id organization_id email email_confirmation created_at updated_at admin)
+    expect(User.attribute_names).to eq %w[id organization_id email email_confirmation created_at updated_at admin]
   end
 
   it "#columns" do
@@ -212,7 +230,7 @@ describe "BazaModels::Model" do
   end
 
   describe "#to_a" do
-    it "should not respond to it, because it will fuck up Array(ModelClass)" do
+    it "does not respond to it, because it will fuck up Array(ModelClass)" do
       expect { User.to_a }.to raise_error(NoMethodError)
       expect { User.to_ary }.to raise_error(NoMethodError)
     end

data/spec/baza_models/query_spec.rb

@@ -7,7 +7,7 @@ describe BazaModels::Query do
   let(:role_user) { Role.new(user: user, role: "user") }
   let(:role_admin) { Role.new(user: user, role: "administrator") }
 
-  context "#average" do
+  describe "#average" do
     it "calculates the average" do
       5.times do |n|
         User.create! id: n + 1, email: "user#{n}@example.com"
@@ -18,7 +18,7 @@ describe BazaModels::Query do
     end
   end
 
-  context "#where" do
+  describe "#where" do
     before do
       user.save!
     end
@@ -29,6 +29,14 @@ describe BazaModels::Query do
       expect(query.to_a).to eq [user]
     end
 
+    it "supports hashes with nils" do
+      user.update!(email_confirmation: nil)
+
+      query = User.where(users: {email_confirmation: nil})
+      expect(query.to_sql).to eq "SELECT `users`.* FROM `users` WHERE `users`.`email_confirmation` IS NULL"
+      expect(query.to_a).to eq [user]
+    end
+
     it "supports strings" do
       query = User.where("email = 'test@example.com'")
       expect(query.to_sql).to eq "SELECT `users`.* FROM `users` WHERE (email = 'test@example.com')"
@@ -49,7 +57,7 @@ describe BazaModels::Query do
     end
   end
 
-  context "#ids" do
+  describe "#ids" do
     it "returns the ids of the models" do
       5.times do |n|
         User.create! id: n + 1, email: "user#{n}@example.com"
@@ -60,7 +68,7 @@ describe BazaModels::Query do
     end
   end
 
-  context "#joins" do
+  describe "#joins" do
     before do
       user.save!
       role_admin.save!
@@ -94,7 +102,7 @@ describe BazaModels::Query do
     end
   end
 
-  context "#group, #order" do
+  describe "#group, #order" do
     before do
       user
       role_user
@@ -109,7 +117,7 @@ describe BazaModels::Query do
       roles = Role.group(:role).order(:role).to_a
 
       expect(roles.length).to eq 2
-      expect(roles.map(&:role)).to eq %w(administrator user)
+      expect(roles.map(&:role)).to eq %w[administrator user]
     end
   end
 

data/spec/spec_helper.rb

@@ -15,7 +15,7 @@ end
 
 # Requires supporting files with custom matchers and macros, etc,
 # in ./support/ and its subdirectories.
-Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].sort.each { |f| require f }
 
 RSpec.configure do |config|
   config.include FactoryBot::Syntax::Methods

data/spec/support/database_helper.rb

@@ -53,7 +53,8 @@ module DatabaseHelper
         {name: :updated_at, type: :datetime},
         {name: :admin, type: :tinyint}
       ],
-      indexes: [:organization_id, :email])
+      indexes: [:organization_id, :email]
+    )
 
     @db.tables.create(
       :user_passports,
@@ -64,7 +65,8 @@ module DatabaseHelper
       ],
       indexes: [
         :user_id
-      ])
+      ]
+    )
 
     @db.tables.create(
       :persons,
@@ -72,7 +74,8 @@ module DatabaseHelper
         {name: :id, type: :int, primarykey: true, autoincr: true},
         {name: :user_id, type: :int}
       ],
-      indexes: [:user_id])
+      indexes: [:user_id]
+    )
 
     @db.tables.create(
       :roles,

data/spec/test_classes/user.rb

@@ -24,11 +24,12 @@ class User < BazaModels::Model
   # Used to test callbacks.
   BazaModels::Model::CALLBACK_TYPES.each do |callback_type|
     attr_reader "#{callback_type}_called"
+
     __send__(callback_type, :add_callback, callback_type)
   end
 
   def self.ransackable_scopes(_auth_object = nil)
-    %i(created_at_since)
+    %i[created_at_since]
   end
 
 private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: baza_models
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.14
 platform: ruby
 authors:
 - kaspernj
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-18 00:00:00.000000000 Z
+date: 2021-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: array_enumerator
@@ -220,6 +220,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -315,7 +343,6 @@ files:
 - spec/baza_models/validators/format_validator_spec.rb
 - spec/baza_models/validators/length_validator_spec.rb
 - spec/baza_models/validators/uniqueness_validator_spec.rb
-- spec/baza_models_spec.rb
 - spec/factories/organization.rb
 - spec/factories/user.rb
 - spec/spec_helper.rb

data/spec/baza_models_spec.rb

@@ -1,4 +0,0 @@
-require "spec_helper"
-
-describe "BazaModels" do
-end