activerecord-cte 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e2fbfad4eca9d8be81cde9c22367e6cf29020baad9411acfffe4abe2f51d99fe
-  data.tar.gz: 39cacb9b310c315c5e0fff63a88903e69b4a618b324f0bcabfe78f4b70530a01
+  metadata.gz: 9c5fd306f910ad7183f278dfbbf91363e7825a823f31fbb798693a8a529db23c
+  data.tar.gz: aa0dcc304d3469394b2e0eb6496ddfac22632df5febc0d43f71302fbbdbfe3d6
 SHA512:
-  metadata.gz: b1a1428a7916fd20c25d758a68cbcc9d39c3e737e462b01467226cc4a7fec68ad86fce2aadd7c2452170979cd8e7bff6fc0026e51a15a886e4bee8ea42eb6332
-  data.tar.gz: 488b14ed1cff2d0271188eeb0457432a38eb51637c44d4e2015ca1624ee646c42b426a7cb8f583b94f8baadc2e90abf2fd59d5aa3bc8d70b3c5a22746c631457
+  metadata.gz: c13b4cb8fe44d0380856fbac382f27f9881d03dcb6d9c6fb21d0cb1c5d2704d09d0e51432b165816f939d1ce46957d877ffc84bdeda837d20604d78065ccfa4b
+  data.tar.gz: a64ff3e533e71d04437f706197266782fd5ab3ec8fe4df047814cd2ffda5dc9bea24040bb93ddfb4b794c068e57df4d5f710d8b941a7cbe8384c57cc039c2db8

data/.github/workflows/rubocop.yml

@@ -0,0 +1,12 @@
+name: Rubocop
+on: [push]
+jobs:
+  Rubocop:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v2
+      - name: Setup Ruby
+        uses: ruby/setup-ruby@v1
+      - run: bundle install
+      - run: bundle exec rubocop

data/.github/workflows/test-with-mysql.yml

@@ -0,0 +1,25 @@
+name: MySql
+on: [push]
+jobs:
+  Test-With-Mysql:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        active_record: [6.0.4, 5.2.6]
+    env:
+      ACTIVE_RECORD_VERSION: ${{ matrix.active_record }}
+      DATABASE_ADAPTER: mysql
+      INSTALL_MYSQL_GEM: true
+      RAILS_ENV: test
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v2
+      - name: Start mysql
+        run: sudo service mysql start
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+      - name: Run tests
+        run: bundle exec rake test

data/.github/workflows/test-with-postgresql.yml

@@ -0,0 +1,34 @@
+name: PostgreSQL
+on: [push]
+jobs:
+  Test-With-PostgreSQL:
+    runs-on: ubuntu-latest
+    container: ruby:2.7
+    strategy:
+      fail-fast: false
+      matrix:
+        active_record: [6.0.4, 5.2.6]
+    env:
+      ACTIVE_RECORD_VERSION: ${{ matrix.active_record }}
+      DATABASE_ADAPTER: postgresql
+      INSTALL_PG_GEM: true
+      RAILS_ENV: test
+    services:
+      postgres:
+        image: postgres
+        env:
+          POSTGRES_PASSWORD: postgres
+        options: >-
+          --health-cmd pg_isready
+          --health-interval 10s
+          --health-timeout 5s
+          --health-retries 5
+        ports:
+          - 5432:5432
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v2
+      - name: Bundle dependencies
+        run: bundle install
+      - name: Run tests
+        run: bundle exec rake test

data/.github/workflows/test-with-sqlite.yml

@@ -0,0 +1,22 @@
+name: SQLite
+on: [push]
+jobs:
+  Test-With-SQLite:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        active_record: [6.0.4, 5.2.6]
+    env:
+      ACTIVE_RECORD_VERSION: ${{ matrix.active_record }}
+      DATABASE_ADAPTER: sqlite3
+      RAILS_ENV: test
+    steps:
+      - name: Check out repository code
+        uses: actions/checkout@v2
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+      - name: Run tests
+        run: bundle exec rake test

data/.rubocop.yml

@@ -1,8 +1,14 @@
 require:
+  - rubocop-minitest
   - rubocop-performance
+  - rubocop-rake
 
 AllCops:
-  TargetRubyVersion: 2.6
+  NewCops: enable
+  TargetRubyVersion: 2.7
+
+Gemspec/RequiredRubyVersion:
+  Enabled: false
 
 Layout/LineLength:
   AllowHeredoc: true
@@ -10,15 +16,30 @@ Layout/LineLength:
   IgnoreCopDirectives: true
   Max: 120
   Exclude:
+    - "activerecord-cte.gemspec"
     - "test/**/*"
 
+Layout/MultilineMethodCallIndentation:
+  EnforcedStyle: indented
+
 Metrics/AbcSize:
   Exclude:
-    - "test/**/*"
+    - "test/**/*_test.rb"
+
+Metrics/ClassLength:
+  Exclude:
+    - "test/**/*_test.rb"
 
 Metrics/CyclomaticComplexity:
   Max: 7
 
+Metrics/MethodLength:
+  Exclude:
+    - "test/**/*_test.rb"
+
+Minitest/MultipleAssertions:
+  Max: 5
+
 Style/ClassAndModuleChildren:
   Enabled: false
 

data/.ruby-version

@@ -0,0 +1 @@
+2.7.3

data/Dockerfile

@@ -1,4 +1,4 @@
-FROM ruby:2.6
+FROM ruby:2.7
 
 ENV APP_HOME /activerecord_cte
 RUN mkdir $APP_HOME
@@ -7,6 +7,7 @@ WORKDIR $APP_HOME
 ENV RAILS_ENV test
 ENV INSTALL_MYSQL_GEM true
 ENV INSTALL_PG_GEM true
+ENV MYSQL_HOST mysql
 
 # Cache the bundle install
 COPY Gemfile* $APP_HOME/

data/Gemfile

@@ -5,7 +5,7 @@ source "https://rubygems.org"
 # Specify your gem's dependencies in activerecord-cte.gemspec
 gemspec
 
-ACTIVE_RECORD_VERSION = ENV.fetch("ACTIVE_RECORD_VERSION") { "6.0.2.1" }
+ACTIVE_RECORD_VERSION = ENV.fetch("ACTIVE_RECORD_VERSION", "6.0.4")
 
 gem "activerecord", ACTIVE_RECORD_VERSION
 

data/README.md

@@ -1,5 +1,10 @@
 # ActiveRecord::Cte
 
+![Rubocop](https://github.com/vlado/activerecord-cte/actions/workflows/rubocop.yml/badge.svg)
+![MySQL](https://github.com/vlado/activerecord-cte/actions/workflows/test-with-mysql.yml/badge.svg)
+![PostgreSQL](https://github.com/vlado/activerecord-cte/actions/workflows/test-with-postgresql.yml/badge.svg)
+![SQLite](https://github.com/vlado/activerecord-cte/actions/workflows/test-with-sqlite.yml/badge.svg)
+
 Adds Common Table Expression support to ActiveRecord (Rails).
 
 It adds `.with` query method and makes it super easy to build and chain complex CTE queries. Let's explain it using simple example.
@@ -196,11 +201,27 @@ I decided to wait a bit :)
 
 After checking out the repo, run `bin/setup` to install dependencies.
 
+### Running Rubocop
+
+```
+bundle exec rubocop
+```
+
 ### Running tests
 
-Run `rake test` to run the tests using SQLite adapter and latest version on Rails.
+To run the tests using SQLite adapter and latest version on Rails run
+
+```
+bundle exec rake test
+```
+
+GitHub Actions will run the test matrix with multiple ActiveRecord versions and database adapters. You can also run the matrix locally with
+
+```
+bundle exec rake test:matrix
+```
 
-To run tests using all supportted database adapters and ActiveRecord versions run `rake test:matrix`. This will build Docker image with all dependencies and run alll tests in it. See `bin/test` for more info.
+This will build Docker image with all dependencies and run all tests in it. See `bin/test` for more info.
 
 ### Console
 
@@ -212,7 +233,7 @@ To install this gem onto your local machine, run `bundle exec rake install`. To
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/[USERNAME]/activerecord-cte. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
+Bug reports and pull requests are welcome on GitHub at https://github.com/vlado/activerecord-cte. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
 
 ## License
 
@@ -220,4 +241,4 @@ The gem is available as open source under the terms of the [MIT License](https:/
 
 ## Code of Conduct
 
-Everyone interacting in the Activerecord::Cte project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/[USERNAME]/activerecord-cte/blob/master/CODE_OF_CONDUCT.md).
+Everyone interacting in the Activerecord::Cte project’s codebases, issue trackers, chat rooms and mailing lists is expected to follow the [code of conduct](https://github.com/vlado/activerecord-cte/blob/master/CODE_OF_CONDUCT.md).

data/Rakefile

@@ -12,7 +12,10 @@ end
 task default: :test
 
 namespace :test do
+  desc "Will run the tests in all db adapters - AR version combinations"
   task :matrix do
+    require "English"
     system("docker-compose build && docker-compose run lib bin/test")
+    exit($CHILD_STATUS.exitstatus) unless $CHILD_STATUS.success?
   end
 end

data/activerecord-cte.gemspec

@@ -11,6 +11,7 @@ Gem::Specification.new do |spec|
   spec.email         = ["vladocingel@gmail.com"]
 
   spec.summary       = "Brings Common Table Expressions support to ActiveRecord and makes it super easy to build and chain complex CTE queries"
+  spec.description   = spec.summary
   spec.homepage      = "https://github.com/vlado/activerecord-cte"
   spec.license       = "MIT"
 
@@ -19,6 +20,7 @@ Gem::Specification.new do |spec|
   spec.metadata["homepage_uri"] = spec.homepage
   spec.metadata["source_code_uri"] = "https://github.com/vlado/activerecord-cte"
 
+  spec.required_ruby_version = ">= 1.9.2"
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
   spec.files = Dir.chdir(File.expand_path(__dir__)) do
@@ -33,7 +35,9 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 2.0"
   spec.add_development_dependency "minitest", "~> 5.0"
   spec.add_development_dependency "rake", "~> 13.0.1"
-  spec.add_development_dependency "rubocop", "~> 0.80.1"
-  spec.add_development_dependency "rubocop-performance", "~> 1.5.2"
+  spec.add_development_dependency "rubocop", "~> 1.17.0"
+  spec.add_development_dependency "rubocop-minitest", "~> 0.13.0"
+  spec.add_development_dependency "rubocop-performance", "~> 1.11.3"
+  spec.add_development_dependency "rubocop-rake", "~> 0.5.1"
   spec.add_development_dependency "sqlite3"
 end

data/bin/test

@@ -1,26 +1,52 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
+require "English"
+
 ADAPTERS = %w[
   mysql
   postgresql
   sqlite3
-]
-AR_VERSIONS = %w[
-  6.0.2.1
-  5.2.4.1
-]
-
-ORIGINAL_AR_VERSION = `bundle show activerecord`.split("-").last.strip
-
-AR_VERSIONS.each do |ar_version|
-  puts "----> Running tests with ActiveRecord #{ar_version}"
-  system("ACTIVE_RECORD_VERSION=#{ar_version} bundle update activerecord") unless ar_version == ORIGINAL_AR_VERSION
-  ADAPTERS.each do |adapter|
-    puts "----> Running tests with #{adapter} adapter"
-    system("DATABASE_ADAPTER=#{adapter} ACTIVE_RECORD_VERSION=#{ar_version} bundle exec rake")
+].freeze
+
+ACTIVE_RECORD_VERSIONS = %w[
+  6.0.4
+  5.2.6
+].freeze
+
+class Matrix
+  def self.run
+    new.run
+  end
+
+  def initialize
+    @exit_status_code = 0
+  end
+
+  def run # rubocop:disable Metrics/MethodLength
+    original_ar_version = `bundle show activerecord`.split("-").last.strip
+    ACTIVE_RECORD_VERSIONS.each do |ar_version|
+      puts "----> Switching ActiveRecord to version #{ar_version}"
+      cmd("ACTIVE_RECORD_VERSION=#{ar_version} bundle update activerecord")
+
+      ADAPTERS.each do |adapter|
+        puts "----> Running tests with ActiveRecord #{ar_version} and #{adapter} adapter"
+        cmd("DATABASE_ADAPTER=#{adapter} ACTIVE_RECORD_VERSION=#{ar_version} bundle exec rake test")
+      end
+    end
+
+    puts "----> Reverting back to original ActiveRecord version (#{original_ar_version})"
+    cmd("ACTIVE_RECORD_VERSION=#{original_ar_version} bundle update activerecord")
+
+    exit(@exit_status_code) unless @exit_status_code.zero?
+  end
+
+  private
+
+  def cmd(cmd)
+    system(cmd)
+    @exit_status_code = $CHILD_STATUS.exitstatus unless $CHILD_STATUS.success?
   end
 end
 
-puts "----> Reverting back to original ActiveRecord version (#{ORIGINAL_AR_VERSION})"
-system("ACTIVE_RECORD_VERSION=#{ORIGINAL_AR_VERSION} bundle update activerecord")
+Matrix.run

data/docker-compose.yml

@@ -15,9 +15,9 @@ services:
     restart: always
     environment:
       MYSQL_DATABASE: activerecord_cte_test
-      MYSQL_USER: user
-      MYSQL_PASSWORD: secret
-      MYSQL_ROOT_PASSWORD: secret
+      MYSQL_USER: root
+      MYSQL_PASSWORD: root
+      MYSQL_ROOT_PASSWORD: root
     ports:
       - 3306:3306
     expose:
@@ -28,8 +28,8 @@ services:
     restart: always
     environment:
       POSTGRES_DB: activerecord_cte_test
-      POSTGRES_USER: user
-      POSTGRES_PASSWORD: secret
+      POSTGRES_USER: postgres
+      POSTGRES_PASSWORD: postgres
     ports:
       - 5432:5432
     expose:

data/lib/activerecord/cte/core_ext.rb

@@ -4,12 +4,8 @@ module ActiveRecord
   module Querying
     delegate :with, to: :all
   end
-  
-  module WithMerger
-    def normal_values
-      super + %i[with]
-    end
 
+  module WithMerger
     def merge
       super
       merge_withs
@@ -56,7 +52,7 @@ module ActiveRecord
       arel
     end
 
-    def build_with(arel) # rubocop:disable Metrics/MethodLength
+    def build_with(arel) # rubocop:disable Metrics/MethodLength, Metrics/CyclomaticComplexity
       return if with_values.empty?
 
       recursive = with_values.delete(:recursive)

data/lib/activerecord/cte/version.rb

@@ -2,6 +2,6 @@
 
 module Activerecord
   module Cte
-    VERSION = "0.1.3"
+    VERSION = "0.1.4"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activerecord-cte
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Vlado Cingel
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-05-30 00:00:00.000000000 Z
+date: 2021-06-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -72,28 +72,56 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.80.1
+        version: 1.17.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.80.1
+        version: 1.17.0
+- !ruby/object:Gem::Dependency
+  name: rubocop-minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.13.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.13.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.2
+        version: 1.11.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.5.2
+        version: 1.11.3
+- !ruby/object:Gem::Dependency
+  name: rubocop-rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.5.1
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.5.1
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -108,15 +136,21 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: 
+description: Brings Common Table Expressions support to ActiveRecord and makes it
+  super easy to build and chain complex CTE queries
 email:
 - vladocingel@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/rubocop.yml"
+- ".github/workflows/test-with-mysql.yml"
+- ".github/workflows/test-with-postgresql.yml"
+- ".github/workflows/test-with-sqlite.yml"
 - ".gitignore"
 - ".rubocop.yml"
+- ".ruby-version"
 - ".travis.yml"
 - ".vimrc"
 - CODE_OF_CONDUCT.md
@@ -148,14 +182,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 1.9.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Brings Common Table Expressions support to ActiveRecord and makes it super