slugable 0.0.4 → 1.0.0

data/spec/slugable/slugable_builder/caching_tree_ancestry_spec.rb

@@ -0,0 +1,38 @@
+require 'spec_helper'
+require 'support/set_up_models'
+
+describe Slugable::SlugBuilder::CachingTreeAncestry do
+  let(:cache) { double(:cache) }
+
+  subject {
+    Slugable::SlugBuilder::CachingTreeAncestry.new(
+        slug_column: :slug,
+        formatter: lambda { |string| "#{string}-formatted" },
+        cache: cache,
+    )
+  }
+
+  let(:model) { TreeCategory }
+
+  describe '#to_slug' do
+    it 'returns array of slugs column from record path that are loaded from cache' do
+      root = model.create!(slug: 'root')
+      child = model.create!(slug: 'child', parent: root)
+
+      allow(cache).to receive(:read_slug).with(:slug, root.id).and_return('root-cached')
+      allow(cache).to receive(:read_slug).with(:slug, child.id).and_return('child-cached')
+
+      expect(subject.to_slug(child)).to eq ['root-cached', 'child-cached']
+    end
+
+    it 'removes blank values from array of record path' do
+      root = model.create!(slug: 'root')
+      child = model.create!(slug: 'child', parent: root)
+
+      allow(cache).to receive(:read_slug).with(:slug, root.id).and_return('')
+      allow(cache).to receive(:read_slug).with(:slug, child.id).and_return(nil)
+
+      expect(subject.to_slug(child)).to eq []
+    end
+  end
+end

data/spec/slugable/slugable_builder/flat_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+require 'support/set_up_models'
+
+describe Slugable::SlugBuilder::Flat do
+  subject {
+    Slugable::SlugBuilder::Flat.new(
+        slug_column: :slug,
+        formatter: lambda { |string| "#{string}-formatted" }
+    )
+  }
+
+  let(:model) { FlatItem }
+
+  describe '#to_slug' do
+    it 'returns value from record column' do
+      record = model.new(slug: 'value')
+      expect(subject.to_slug(record)).to eq 'value'
+
+      record = model.create!(slug: 'value')
+      expect(subject.to_slug(record)).to eq 'value'
+    end
+  end
+
+  describe '#to_slug_was' do
+    it 'returns old value from record'do
+      record = model.create!(slug: 'value')
+      record.slug = 'new-value'
+      expect(subject.to_slug_was(record)).to eq 'value'
+    end
+  end
+
+  describe '#to_slug_will' do
+    it 'returns future value of record' do
+      record = model.new(slug: 'value')
+      expect(subject.to_slug_will(record)).to eq 'value-formatted'
+
+      record = model.create!(slug: 'value')
+      record.slug = 'new-value'
+      expect(subject.to_slug_will(record)).to eq 'new-value-formatted'
+    end
+  end
+end

data/spec/slugable/slugable_builder/tree_ancestry_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+require 'support/set_up_models'
+
+describe Slugable::SlugBuilder::TreeAncestry do
+  subject {
+    Slugable::SlugBuilder::TreeAncestry.new(
+        slug_column: :slug,
+        formatter: lambda { |string| "#{string}-formatted" }
+    )
+  }
+
+  let(:model) { TreeCategory }
+
+  describe '#to_slug' do
+    it 'returns array of slugs column from record path' do
+      root = model.create!(slug: 'root')
+      child = model.create!(slug: 'child', parent: root)
+      expect(subject.to_slug(child)).to eq ['root', 'child']
+    end
+
+    it 'removes blank values from array of record path' do
+      child = model.new(slug: '')
+      expect(subject.to_slug(child)).to eq []
+
+      child = model.new(slug: nil)
+      expect(subject.to_slug(child)).to eq []
+    end
+  end
+
+  describe '#to_slug_was' do
+    it 'returns old value of slug using old ancestry and slug_was from record' do
+      root = model.create!(slug: 'root')
+      child = model.create!(slug: 'child', parent: root)
+
+      new_root = model.create!(slug: 'new-root')
+      child.parent = new_root
+      child.slug = 'new-child'
+
+      expect(subject.to_slug_was(child)).to eq ['root', 'child']
+    end
+  end
+
+  describe '#to_slug_will' do
+    it 'returns future value of slugs in record path' do
+      root = model.create!(slug: 'root')
+      child = model.create!(slug: 'child', parent: root)
+
+      new_root = model.create!(slug: 'new-root')
+      child.parent = new_root
+      child.slug = 'new-child'
+
+      expect(subject.to_slug_will(child)).to eq ['new-root', 'new-child-formatted']
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,12 +1,32 @@
-require "slugable"
-require "sqlite3"
-require "ancestry"
+if ENV['COVERAGE'] == '1'
+  require 'simplecov'
+  SimpleCov.start do
+    load_profile 'test_frameworks'
+    add_group 'Libraries", "lib'
+  end
+end
+
+if ENV['CI'] == 'true' && ENV['TRAVIS_RUBY_VERSION'] == '2.2.2' && ENV['BUNDLE_GEMFILE'] =~ /activerecord_4\.2/
+  require 'codeclimate-test-reporter'
+  CodeClimate::TestReporter.start
+end
 
-ActiveRecord::Base.establish_connection :adapter => "sqlite3", :database => ":memory:"
+require 'slugable'
+require 'sqlite3'
+require 'ancestry'
+require 'pry'
 
-load "db/schema.rb"
+ActiveRecord::Base.send :extend, Slugable::HasSlug
+
+ActiveRecord::Base.establish_connection adapter: 'sqlite3', database: ':memory:'
+
+load 'db/schema.rb'
 
 RSpec.configure do |config|
+  config.filter_run focus: true
+  config.filter_run_excluding skip: true
+  config.run_all_when_everything_filtered = true
+
   config.around do |example|
     ActiveRecord::Base.transaction do
       example.run

data/spec/support/hash_cache_storage.rb

@@ -0,0 +1,27 @@
+class HashCacheStorage
+  attr_reader :store
+
+  def initialize
+    @store = {}
+  end
+
+  def fetch(key, options = nil, &block)
+    store.fetch(key) {
+      value = block.call
+      store[key] = value
+      value
+    }
+  end
+
+  def clear(options = nil)
+    @store = {}
+  end
+
+  def read(key, options = nil)
+    store[key]
+  end
+
+  def write(key, value, options = nil)
+    store[key] = value
+  end
+end

data/spec/support/set_up_models.rb

@@ -0,0 +1,27 @@
+require 'support/simple_to_slug'
+
+class FlatItem < ActiveRecord::Base
+  has_slug
+end
+
+class FlatPage < ActiveRecord::Base
+  has_slug from: :title, to: :seo_url
+end
+
+class TreeCategory < ActiveRecord::Base
+  has_ancestry
+  has_slug tree_cache_storage: nil
+end
+
+class TreeItem < ActiveRecord::Base
+  has_ancestry
+  has_slug tree_cache_storage: Slugable.configuration.tree_cache_storage
+end
+
+class FlatProduct < ActiveRecord::Base
+  has_slug formatter: lambda { |string| 'hello-all-the-time' }
+end
+
+class FlatNews < ActiveRecord::Base
+  has_slug to_slug_builder: SimpleToSlug.new
+end

data/spec/support/simple_to_slug.rb

@@ -0,0 +1,13 @@
+class SimpleToSlug
+  def to_slug(record)
+    "to_slug_#{record.id}"
+  end
+
+  def to_slug_was(record)
+    "to_slug_was_#{record.id}"
+  end
+
+  def to_slug_will(record)
+    "to_slug_will_#{record.id}"
+  end
+end

metadata

@@ -1,128 +1,153 @@
 --- !ruby/object:Gem::Specification
 name: slugable
 version: !ruby/object:Gem::Version
-  version: 0.0.4
-  prerelease: 
+  version: 1.0.0
 platform: ruby
 authors:
 - Miroslav Hettes
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-30 00:00:00.000000000 Z
+date: 2015-11-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    none: false
-    requirements:
-    - - ! '>='
-      - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '3.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 - !ruby/object:Gem::Dependency
-  name: wnm_support
+  name: appraisal
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.4
-  type: :runtime
+        version: 2.1.0
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.0.4
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.11.0
+        version: 3.4.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.11.0
+        version: 3.4.0
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.9.2.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 0.9.2.2
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: 1.3.11
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.3.6
+        version: 1.3.11
 - !ruby/object:Gem::Dependency
   name: ancestry
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ~>
+    - - ">="
       - !ruby/object:Gem::Version
         version: 1.3.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  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: simplecov
+  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: codeclimate-test-reporter
+  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'
 description: Add dsl method for automatic storing seo friendly url in database column
 email:
 - hettes@webynamieru.sk
@@ -130,51 +155,70 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .rspec
+- ".gitignore"
+- ".rspec"
+- ".rvmrc"
+- ".travis.yml"
+- Appraisals
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - changelog.md
 - db/schema.rb
+- gemfiles/activerecord_3.2.gemfile
+- gemfiles/activerecord_4.0.gemfile
+- gemfiles/activerecord_4.1.gemfile
+- gemfiles/activerecord_4.2.gemfile
 - lib/slugable.rb
+- lib/slugable/cache_layer.rb
+- lib/slugable/formatter/parameterize.rb
 - lib/slugable/has_slug.rb
 - lib/slugable/railtie.rb
+- lib/slugable/slug_builder/caching_tree_ancestry.rb
+- lib/slugable/slug_builder/flat.rb
+- lib/slugable/slug_builder/tree_ancestry.rb
 - lib/slugable/version.rb
 - slugable.gemspec
+- spec/slugable/cache_layer_spec.rb
 - spec/slugable/has_slug_spec.rb
+- spec/slugable/slugable_builder/caching_tree_ancestry_spec.rb
+- spec/slugable/slugable_builder/flat_spec.rb
+- spec/slugable/slugable_builder/tree_ancestry_spec.rb
 - spec/spec_helper.rb
+- spec/support/hash_cache_storage.rb
+- spec/support/set_up_models.rb
+- spec/support/simple_to_slug.rb
 homepage: https://github.com/mirrec/slugable
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -494734928332233680
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-      segments:
-      - 0
-      hash: -494734928332233680
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.4.5
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: Storing seo friendly url in column
 test_files:
+- spec/slugable/cache_layer_spec.rb
 - spec/slugable/has_slug_spec.rb
+- spec/slugable/slugable_builder/caching_tree_ancestry_spec.rb
+- spec/slugable/slugable_builder/flat_spec.rb
+- spec/slugable/slugable_builder/tree_ancestry_spec.rb
 - spec/spec_helper.rb
+- spec/support/hash_cache_storage.rb
+- spec/support/set_up_models.rb
+- spec/support/simple_to_slug.rb