metka 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cf64e63f73da3ffb7c87e5d3806a13e4c7d09356e63b8d52ffefd27bf7dda8df
-  data.tar.gz: 98c5808ea5d3426a15b9eb7084b6b07f7c3242ee8f6b96ca53f20cb5f9987208
+  metadata.gz: dd6f202972d558afed083a0095530a47e1e2f35028beb4a457c1b13741614b0d
+  data.tar.gz: e11832ae26b49cdb99aa7b6fd5dddb9f559b862b8460c19b492c05467ea04331
 SHA512:
-  metadata.gz: 24ac1c6bc72b5a98a879897f7cdddd721df052cb84841a0e3adaefff003d9414a0adf1c40cc663a5ed468f5e320a83af09ea76461fb66ce7625cfe7c2f8b376a
-  data.tar.gz: cf3750a9f2dccedd27afac2f967187e6e5655483fc7bd696b0da9d63308bc79279cb90c141db0e25d333dde79ce54235b674f2495f4fe904fccdeb6591419927
+  metadata.gz: ea2f237f88c8a661931db07532243ddd6c7af85741f77422cc6f6fafd3460fbfaacad205064eeecb438a29a25c3aff11157ddf6309ff6bb559d317674bbdc019
+  data.tar.gz: b9e9f74757a541c6bf8fb123ba27cf619ebafeb7b92b61ea0a71b0482b5e3a3def87bae4c5292cd159e4bc6beeeb9274811e77e7dd461736d16bb15529ec7673

data/.github/workflows/continuous-integration-workflow.yml

@@ -0,0 +1,11 @@
+name: CI
+# This workflow is triggered on pushes to the repository.
+on: [push]
+
+jobs:
+  build:
+    name: RSpec
+    # This job runs on Linux
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v1

data/.gitignore

@@ -6,6 +6,7 @@
 /pkg/
 /spec/reports/
 /tmp/
+/spec/tmp/
 
 # rspec failure tracking
 .rspec_status

data/.rubocop.yml

@@ -1,5 +1,5 @@
 inherit_gem:
-  standard: config/base.yml
+  jetrockets-standard: config/gems.yml
 
 AllCops:
   Exclude:
@@ -13,12 +13,6 @@ AllCops:
   DisplayCopNames: true
   TargetRubyVersion: 2.5
 
-Standard/SemanticBlocks:
-  Enabled: false
-
-Style/FrozenStringLiteralComment:
-  Enabled: true
-
 Style/TrailingCommaInArrayLiteral:
   EnforcedStyleForMultiline: no_comma
 

data/.travis.yml

@@ -16,7 +16,9 @@ env:
     - METKA_DB_USER=postgres
     - METKA_DB_NAME=metka
 
+before_install:
+  - gem install -v 2.0.2 bundler
+
 before_script:
   - gem update --system
-  - bundle exec rake dummy:db:create
-  - RAILS_ENV=test bundle exec rake dummy:db:create dummy:db:migrate
+  - RAILS_ENV=test bundle exec rake dummy:db:create dummy:db:schema:load

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    metka (0.1.0)
+    metka (0.1.1)
       dry-configurable
       rails (>= 4.2)
 
@@ -61,13 +61,17 @@ GEM
     dry-core (0.4.9)
       concurrent-ruby (~> 1.0)
     erubi (1.8.0)
-    faker (2.1.2)
-      i18n (>= 0.8)
+    faker (2.3.0)
+      i18n (~> 1.6.0)
     globalid (0.4.2)
       activesupport (>= 4.2.0)
     i18n (1.6.0)
       concurrent-ruby (~> 1.0)
     jaro_winkler (1.5.3)
+    jetrockets-standard (1.0.1)
+      rubocop-rails (~> 2.3.2)
+      rubocop-rspec (~> 1.35.0)
+      standard (~> 0.1.4)
     loofah (2.2.3)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
@@ -77,11 +81,11 @@ GEM
     mini_mime (1.0.2)
     mini_portile2 (2.4.0)
     minitest (5.11.3)
-    nio4r (2.4.0)
+    nio4r (2.5.1)
     nokogiri (1.10.4)
       mini_portile2 (~> 2.4.0)
     parallel (1.17.0)
-    parser (2.6.3.0)
+    parser (2.6.4.1)
       ast (~> 2.4.0)
     pg (1.1.4)
     rack (2.0.7)
@@ -142,6 +146,11 @@ GEM
       unicode-display_width (>= 1.4.0, < 1.7)
     rubocop-performance (1.4.1)
       rubocop (>= 0.71.0)
+    rubocop-rails (2.3.2)
+      rack (>= 1.1)
+      rubocop (>= 0.72.0)
+    rubocop-rspec (1.35.0)
+      rubocop (>= 0.60.0)
     ruby-progressbar (1.10.1)
     sprockets (3.7.2)
       concurrent-ruby (~> 1.0)
@@ -150,7 +159,7 @@ GEM
       actionpack (>= 4.0)
       activesupport (>= 4.0)
       sprockets (>= 3.0.0)
-    standard (0.1.2)
+    standard (0.1.4)
       rubocop (~> 0.72.0)
       rubocop-performance (~> 1.4.0)
     thor (0.20.3)
@@ -171,13 +180,13 @@ DEPENDENCIES
   ammeter
   bundler
   faker
+  jetrockets-standard (~> 1.0.1)
   metka!
   pg
   rake
   rspec
   rspec-rails
-  standard
   timecop
 
 BUNDLED WITH
-   2.0.1
+   2.0.2

data/README.md

@@ -76,7 +76,7 @@ Lets take a look at real example. We have a `notes` table with `tags` column.
 Now lets generate a migration.
 
 ```bash
-RAILS_ENV=test rails g metka:strategies:view --source-table-name=notes
+rails g metka:strategies:view --source-table-name=notes
 ```
 
 The result would be:

data/Rakefile

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new(:spec)
 
 namespace :dummy do
-  require_relative "spec/dummy/config/application"
+  require_relative 'spec/dummy/config/application'
   Dummy::Application.load_tasks
 end
 

data/lib/generators/metka/strategies/materialized_view/materialized_view_generator.rb

@@ -0,0 +1,58 @@
+# frozen_string_literal: true
+
+require 'rails/generators'
+require 'rails/generators/active_record'
+
+module Metka
+  module Generators
+    module Strategies
+      class MaterializedViewGenerator < ::Rails::Generators::Base # :nodoc:
+        include Rails::Generators::Migration
+
+        desc <<~LONGDESC
+          Generates migration to implement view strategy for Metka
+
+          > $ rails g metka:strategies:materialized_view --source-table-name=NAME_OF_TABLE_WITH_TAGS
+        LONGDESC
+
+        source_root File.expand_path('templates', __dir__)
+
+        class_option :source_table_name, type: :string, required: true,
+                                         desc: 'Name of the table that has a column with tags'
+
+        class_option :source_column_name, type: :string, default: 'tags',
+                                          desc: 'Name of the column with stored tags'
+
+        def generate_migration
+          migration_template 'migration.rb.erb', "db/migrate/#{migration_name}.rb"
+        end
+
+        no_tasks do
+          def source_table_name
+            options[:source_table_name]
+          end
+
+          def source_column_name
+            options[:source_column_name]
+          end
+
+          def view_name
+            "tagged_#{source_table_name}"
+          end
+
+          def migration_name
+            "create_#{view_name}_view"
+          end
+
+          def migration_class_name
+            migration_name.classify
+          end
+        end
+
+        def self.next_migration_number(dir)
+          ::ActiveRecord::Generators::Base.next_migration_number(dir)
+        end
+      end
+    end
+  end
+end

data/lib/generators/metka/strategies/materialized_view/templates/migration.rb.erb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+class <%= @migration_class_name %> < ActiveRecord::Migration<%= ActiveRecord::VERSION::MAJOR < 5 ? '' : '[5.0]' %>
+  def up
+    execute <<-SQL
+    CREATE 
+      OR REPLACE FUNCTION metka_refresh_<%= view_name %>_materialized_view RETURNS TRIGGER LANGUAGE plpgsql AS $$
+      BEGIN
+        IF TG_OP = 'INSERT' AND NEW.<%= source_column_name %> IS NOT NULL THEN
+          REFRESH MATERIALIZED VIEW CONCURRENTLY <%= view_name %>;
+        ELSIF TG_OP = 'UPDATE' AND OLD.<%= source_column_name %> IS NOT NULL AND NEW.<%= source_column_name %> IS NOT NULL THEN
+          REFRESH MATERIALIZED VIEW CONCURRENTLY <%= view_name %>;
+        ELSIF TG_OP = 'DELETE' AND OLD.<%= source_column_name %> IS NOT NULL THEN
+          REFRESH MATERIALIZED VIEW CONCURRENTLY <%= view_name %>;
+        END IF;
+
+      RETURN NULL;
+    END $$;
+
+    CREATE OR REPLACE MATERIALZIED VIEW <%= view_name %> AS
+
+    SELECT UNNEST
+      ( <%= source_column_name %> ) AS <%= source_column_name.singularize %>_name,
+      COUNT ( * ) AS taggings_count
+    FROM
+      <%= source_table_name %>
+    GROUP BY
+      <%= source_column_name.singularize %>_name;
+
+    CREATE TRIGGER metka_on_<%= source_table_name %>
+    BEFORE UPDATE OR INSERT OR DELETE ON <%= source_table_name %> FOR EACH ROW
+    EXECUTE PROCEDURE metka_refresh_<%= view_name %>_materialized_view();    
+  end
+
+  def down
+    execute <<-SQL
+      DROP VIEW <%= view_name %>;
+    SQL
+  end
+end

data/lib/generators/metka/strategies/view/templates/migration.rb.erb

@@ -11,7 +11,7 @@ class <%= @migration_class_name %> < ActiveRecord::Migration<%= ActiveRecord::VE
     FROM
       <%= source_table_name %>
     GROUP BY
-      name;
+      <%= source_column_name.singularize %>_name;
     SQL
   end
 

data/lib/generators/metka/strategies/view/view_generator.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "rails/generators"
-require "rails/generators/active_record"
+require 'rails/generators'
+require 'rails/generators/active_record'
 
 module Metka
   module Generators
@@ -9,22 +9,22 @@ module Metka
       class ViewGenerator < ::Rails::Generators::Base # :nodoc:
         include Rails::Generators::Migration
 
-        desc <<-LONGDESC
-Generates migration to implement view strategy for Metka
+        desc <<~LONGDESC
+          Generates migration to implement view strategy for Metka
 
-> $ rails g metka:strategies:view --source-table-name=NAME_OF_TABLE_WITH_TAGS
-LONGDESC
+          > $ rails g metka:strategies:view --source-table-name=NAME_OF_TABLE_WITH_TAGS
+        LONGDESC
 
-        source_root File.expand_path("templates", __dir__)
+        source_root File.expand_path('templates', __dir__)
 
         class_option :source_table_name, type: :string, required: true,
-          desc: 'Name of the table that has a column with tags'
+                                         desc: 'Name of the table that has a column with tags'
 
         class_option :source_column_name, type: :string, default: 'tags',
-          desc: 'Name of the column with stored tags'
+                                          desc: 'Name of the column with stored tags'
 
         def generate_migration
-          migration_template "migration.rb.erb", "db/migrate/#{migration_name}.rb"
+          migration_template 'migration.rb.erb', "db/migrate/#{migration_name}.rb"
         end
 
         no_tasks do
@@ -55,4 +55,4 @@ LONGDESC
       end
     end
   end
-end
+end

data/lib/metka.rb

@@ -1,20 +1,20 @@
 # frozen_string_literal: true
 
-require "metka/version"
+require 'metka/version'
 
-require "active_support/core_ext/module"
-require "dry-configurable"
+require 'active_support/core_ext/module'
+require 'dry-configurable'
 
 module Metka
-  require "metka/tag_list"
-  require "metka/generic_parser"
-  require "metka/query_builder"
-  require "metka/model"
+  require 'metka/tag_list'
+  require 'metka/generic_parser'
+  require 'metka/query_builder'
+  require 'metka/model'
 
   class Error < StandardError; end
 
   extend Dry::Configurable
 
   setting :parser, Metka::GenericParser
-  setting :delimiter, ","
+  setting :delimiter, ','
 end

data/lib/metka/generic_parser.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "singleton"
+require 'singleton'
 
 module Metka
   ##
@@ -16,7 +16,7 @@ module Metka
       TagList.new.tap do |tag_list|
         case value
         when String
-          tag_list.merge value.split(",").map(&:strip).reject(&:empty?)
+          tag_list.merge value.split(',').map(&:strip).reject(&:empty?)
         when Enumerable
           tag_list.merge value.reject(&:empty?)
         end

data/lib/metka/model.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "active_support/concern"
+require 'active_support/concern'
 
 module Metka
   # Extends AR model with methods to use tags
@@ -14,7 +14,7 @@ module Metka
 
         return none if tag_list.empty?
 
-        where(::Metka::QueryBuilder.new.call(self, "tags", tag_list, options))
+        where(::Metka::QueryBuilder.new.call(self, 'tags', tag_list, options))
       end
     end
 

data/lib/metka/query_builder.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require_relative "query_builder/exclude_tags_query"
-require_relative "query_builder/any_tags_query"
-require_relative "query_builder/all_tags_query"
+require_relative 'query_builder/exclude_tags_query'
+require_relative 'query_builder/any_tags_query'
+require_relative 'query_builder/all_tags_query'
 
 module Metka
   class QueryBuilder

data/lib/metka/query_builder/all_tags_query.rb

@@ -6,21 +6,21 @@ module Metka
 
     def call(model, column_name, tag_list)
       column_cast = Arel::Nodes::NamedFunction.new(
-        "CAST",
-        [model.arel_table[column_name].as("text[]")]
+        'CAST',
+        [model.arel_table[column_name].as('text[]')]
       )
 
       value = Arel::Nodes::SqlLiteral.new(
         # In Rails 5.2 and above Sanitanization moved to public level, but still we have to support 4.2 and 5.0 and 5.1
-        ActiveRecord::Base.send(:sanitize_sql_for_conditions, ["ARRAY[?]", tag_list.to_a])
+        ActiveRecord::Base.send(:sanitize_sql_for_conditions, ['ARRAY[?]', tag_list.to_a])
       )
 
       value_cast = Arel::Nodes::NamedFunction.new(
-        "CAST",
-        [value.as("text[]")]
+        'CAST',
+        [value.as('text[]')]
       )
 
-      Arel::Nodes::InfixOperation.new("@>", column_cast, value_cast)
+      Arel::Nodes::InfixOperation.new('@>', column_cast, value_cast)
     end
   end
 end

data/lib/metka/query_builder/any_tags_query.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "singleton"
+require 'singleton'
 
 module Metka
   class AnyTagsQuery
@@ -8,21 +8,21 @@ module Metka
 
     def call(model, column_name, tag_list)
       column_cast = Arel::Nodes::NamedFunction.new(
-        "CAST",
-        [model.arel_table[column_name].as("text[]")]
+        'CAST',
+        [model.arel_table[column_name].as('text[]')]
       )
 
       value = Arel::Nodes::SqlLiteral.new(
         # In Rails 5.2 and above Sanitanization moved to public level, but still we have to support 4.2 and 5.0 and 5.1
-        ActiveRecord::Base.send(:sanitize_sql_for_conditions, ["ARRAY[?]", tag_list.to_a])
+        ActiveRecord::Base.send(:sanitize_sql_for_conditions, ['ARRAY[?]', tag_list.to_a])
       )
 
       value_cast = Arel::Nodes::NamedFunction.new(
-        "CAST",
-        [value.as("text[]")]
+        'CAST',
+        [value.as('text[]')]
       )
 
-      Arel::Nodes::InfixOperation.new("&&", column_cast, value_cast)
+      Arel::Nodes::InfixOperation.new('&&', column_cast, value_cast)
     end
   end
 end

data/lib/metka/tag_list.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "set"
+require 'set'
 
 module Metka
   class TagList < Set

data/lib/metka/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Metka
-  VERSION = "0.1.1"
+  VERSION = '0.1.2'
 end

data/metka.gemspec

@@ -1,39 +1,39 @@
 # frozen_string_literal: true
 
-lib = File.expand_path("../lib", __FILE__)
+lib = File.expand_path('../lib', __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "metka/version"
+require 'metka/version'
 
 Gem::Specification.new do |spec|
-  spec.name = "metka"
+  spec.name = 'metka'
   spec.version = Metka::VERSION
-  spec.authors = ["Igor Alexandrov"]
-  spec.email = ["igor.alexandrov@gmail.com"]
+  spec.authors = ['Igor Alexandrov']
+  spec.email = ['igor.alexandrov@gmail.com']
 
-  spec.summary = "Rails tagging system based on PostgreSQL arrays"
-  spec.description = "Rails tagging system based on PostgreSQL arrays"
-  spec.homepage = "https://github.com/jetrockets/metka"
-  spec.license = "MIT"
+  spec.summary = 'Rails tagging system based on PostgreSQL arrays'
+  spec.description = 'Rails tagging system based on PostgreSQL arrays'
+  spec.homepage = 'https://github.com/jetrockets/metka'
+  spec.license = 'MIT'
 
   # 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("..", __FILE__)) do
+  spec.files = Dir.chdir(File.expand_path('..', __FILE__)) do
     `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   end
-  spec.bindir = "exe"
+  spec.bindir = 'exe'
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "dry-configurable"
-  spec.add_dependency "rails", ">= 4.2"
+  spec.add_dependency 'dry-configurable'
+  spec.add_dependency 'rails', '>= 4.2'
 
-  spec.add_development_dependency "ammeter"
-  spec.add_development_dependency "bundler"
-  spec.add_development_dependency "faker"
-  spec.add_development_dependency "pg"
-  spec.add_development_dependency "rake"
-  spec.add_development_dependency "rspec"
-  spec.add_development_dependency "rspec-rails"
-  spec.add_development_dependency "standard"
-  spec.add_development_dependency "timecop"
+  spec.add_development_dependency 'ammeter'
+  spec.add_development_dependency 'bundler'
+  spec.add_development_dependency 'faker'
+  spec.add_development_dependency 'jetrockets-standard', '~> 1.0.1'
+  spec.add_development_dependency 'pg'
+  spec.add_development_dependency 'rake'
+  spec.add_development_dependency 'rspec'
+  spec.add_development_dependency 'rspec-rails'
+  spec.add_development_dependency 'timecop'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: metka
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Igor Alexandrov
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-08-23 00:00:00.000000000 Z
+date: 2019-10-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-configurable
@@ -81,21 +81,21 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pg
+  name: jetrockets-standard
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.1
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: pg
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -109,7 +109,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -123,7 +123,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec-rails
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -137,7 +137,7 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: standard
+  name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
@@ -171,6 +171,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/continuous-integration-workflow.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
@@ -183,6 +184,8 @@ files:
 - Rakefile
 - bin/console
 - bin/setup
+- lib/generators/metka/strategies/materialized_view/materialized_view_generator.rb
+- lib/generators/metka/strategies/materialized_view/templates/migration.rb.erb
 - lib/generators/metka/strategies/view/templates/migration.rb.erb
 - lib/generators/metka/strategies/view/view_generator.rb
 - lib/metka.rb
@@ -214,7 +217,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.0.6
 signing_key: 
 specification_version: 4
 summary: Rails tagging system based on PostgreSQL arrays