listings 0.0.3 → 0.1.0

data/spec/lib/sources/object_data_source_spec.rb

@@ -0,0 +1,231 @@
+include Listings::Sources
+
+RSpec.describe ObjectDataSource do
+
+  shared_examples "object datasource" do
+    let(:tracks) {
+      build_list(:object_album, 10).map(&:tracks).flatten
+    }
+
+    def add_tracks(ts)
+      if ts.is_a?(Array)
+        ts.each { |t| add_tracks t }
+      elsif ts.is_a?(ObjectAlbum)
+        add_tracks ts.tracks
+      elsif ts.is_a?(ObjectTrack)
+        tracks << ts
+      else
+        raise "Invalid Object #{ts}"
+      end
+    end
+
+    let(:track_title) { ds.build_field :title }
+
+    describe "DataSource factory" do
+      it "should have tracks with album" do
+        expect(tracks).to_not be_empty
+        expect(tracks.first.album).to_not be_nil
+      end
+
+      it "should create" do
+        expect(ds).to be_a(ObjectDataSource)
+      end
+
+      describe "field" do
+        let(:tracks) { [] }
+        before(:each) {
+          add_tracks build(:object_track, album: nil)
+        }
+
+        it "should deal with intermediate nils" do
+          expect(album_name.value_for(ds.items.first)).to be_nil
+        end
+
+        it "should have key" do
+          expect(album_name.key).to eq('album_name')
+        end
+      end
+
+      describe "items" do
+        it "should list all" do
+          expect(ds.items.count).to be(tracks.count)
+        end
+
+        it "should enumerate all items" do
+          expect(begin
+            c = 0
+            ds.items.each do
+              c = c + 1
+            end
+            c
+          end).to be(tracks.count)
+        end
+      end
+
+      describe "paginate" do
+        let(:page_size) { 5 }
+
+        before(:each) { ds.paginate(2, page_size) }
+
+        it "should get only paged items" do
+          expect(ds.items.count).to be(page_size)
+        end
+
+        it "should keep total_count" do
+          expect(ds.items.total_count).to be(tracks.count)
+        end
+      end
+
+      describe "field" do
+        it "should project attribute value" do
+          expect(track_title.value_for(ds.items.first)).to eq(tracks.first.title)
+        end
+
+        it "should navigate and  project attribute value" do
+          expect(album_name.value_for(ds.items.first)).to eq(tracks.first.album.name)
+        end
+      end
+
+      describe "scope" do
+        before(:each) do
+          ds.scope do |items|
+            [].tap do |res|
+              tracks.each_with_index do |item, index|
+                next if index % 2 == 1
+                res << item
+              end
+            end
+          end
+        end
+
+        it "should return scoped items" do
+          expect(ds.items.count).to be(tracks.count / 2)
+        end
+      end
+
+      describe "search" do
+        before(:each) do
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-magic-string-1')
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-magic-string-2')
+          build_list(:object_album, 2).each do |album_with_tracks_to_match|
+            add_tracks build_list(:object_track, 5, title: 'title-magic-string', album: album_with_tracks_to_match)
+          end
+
+          ds.search([track_title, album_name], 'magic')
+        end
+
+        it "should return matching items" do
+          expect(ds.items.count).to be(20)
+        end
+      end
+
+      describe "filter" do
+        before(:each) do
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-magic-string-1')
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-magic-string-2')
+
+          ds.filter(album_name, 'album-name-magic-string-1')
+        end
+
+        it "should return matching items" do
+          expect(ds.items.count).to be(5)
+        end
+      end
+
+      describe "sort" do
+        def all_track_albumns_name
+          tracks.map { |t| t.album.name }
+        end
+
+        before(:each) do
+          expect(all_track_albumns_name).to_not eq(all_track_albumns_name.sort)
+          ds.sort(album_name)
+        end
+
+        it "should return matching items" do
+          expect(ds.items.map { |e| album_name.value_for(e) }).to eq(all_track_albumns_name.sort)
+        end
+      end
+
+      describe "sort desc" do
+        def all_track_albumns_name
+          tracks.map { |t| t.album.name }
+        end
+
+        before(:each) do
+          expect(all_track_albumns_name).to_not eq(all_track_albumns_name.sort.reverse)
+          ds.sort(album_name, DataSource::DESC)
+        end
+
+        it "should return matching items" do
+          expect(ds.items.map { |e| album_name.value_for(e) }).to eq(all_track_albumns_name.sort.reverse)
+        end
+      end
+
+      describe "values_for_filter" do
+        let(:tracks) { [] } # skip default test albums
+
+        before(:each) do
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-1')
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-3')
+          add_tracks build(:object_album, tracks_count: 5, name: 'album-name-2')
+          add_tracks build(:object_album, tracks_count: 5, name: nil)
+        end
+
+        context "without search" do
+          it "should matching values" do
+            expect(ds.values_for_filter(album_name)).to eq(['album-name-1', 'album-name-2', 'album-name-3'])
+          end
+        end
+
+        context "with scope" do
+          before(:each) do
+            ds.scope do |items|
+              [].tap do |res|
+                tracks.each_with_index do |item, index|
+                  next if index % 2 == 1
+                  res << item
+                end
+              end
+            end
+          end
+
+          it "should matching values" do
+            expect(ds.values_for_filter(album_name)).to eq(['album-name-1', 'album-name-2', 'album-name-3'])
+          end
+        end
+      end
+    end
+
+  end
+
+  context "using array of objects" do
+    let(:ds) { DataSource.for(tracks) }
+    let(:album_name) { nil }
+
+    context "and hash like fields" do
+      let(:album_name) { ds.build_field album: :name }
+      it_behaves_like "object datasource"
+    end
+
+    context "and array like fields" do
+      let(:album_name) { ds.build_field [:album, :name] }
+      it_behaves_like "object datasource"
+    end
+  end
+
+  context "using array of hash" do
+    let(:ds) { DataSource.for(tracks.map(&:to_h)) }
+    let(:album_name) { nil }
+
+    context "and hash like fields" do
+      let(:album_name) { ds.build_field album: :name }
+      it_behaves_like "object datasource"
+    end
+
+    context "and array like fields" do
+      let(:album_name) { ds.build_field [:album, :name] }
+      it_behaves_like "object datasource"
+    end
+  end
+end

data/spec/listings/tracks_fixed_order_listing_spec.rb

@@ -0,0 +1,19 @@
+RSpec.describe TracksFixedOrderListing, type: :listing do
+  describe "List entities" do
+    let!(:track) { create(:album, tracks_count: 1) }
+
+    let(:listing) { query_listing :tracks_fixed_order }
+
+    it "should list all" do
+      expect(listing.items.count).to eq(1)
+    end
+
+    it "can be rendered" do
+      render_listing :tracks
+    end
+
+    it "can't be sorted" do
+      expect(listing).to_not be_sortable
+    end
+  end
+end

data/spec/listings/tracks_listing_spec.rb

@@ -0,0 +1,27 @@
+RSpec.describe TracksListing, type: :listing do
+  describe "List entities" do
+    let!(:track) { create(:album, tracks_count: 1) }
+
+    let(:listing) { query_listing :tracks }
+
+    it "should list all" do
+      expect(listing.items.count).to eq(1)
+    end
+
+    it "can be rendered" do
+      render_listing :tracks
+    end
+
+    it "should get name from model" do
+      expect(listing.column_with_key('album_name').human_name).to eq('Album Name')
+    end
+
+    it "should get name from title option for column" do
+      expect(listing.column_with_key('album_id').human_name).to eq('The Album Id')
+    end
+
+    it "should get name from title option for filter" do
+      expect(listing.filter_with_key('album_id').human_name).to eq('The Album Id')
+    end
+  end
+end

data/spec/models/album_spec.rb

@@ -0,0 +1,9 @@
+RSpec.describe Album, type: :model do
+  describe "Factory" do
+    it "can create" do
+      create(:album)
+      expect(Album.count).to eq(1)
+      expect(Album.first.tracks.count).to be > 1
+    end
+  end
+end

data/spec/models/post_spec.rb

@@ -1,6 +1,6 @@
 RSpec.describe Post, type: :model do
-  describe "Create" do
-    it "can be created" do
+  describe "Factory" do
+    it "can create" do
       create(:post)
       expect(Post.count).to eq(1)
     end

data/spec/models/track_spec.rb

@@ -0,0 +1,8 @@
+RSpec.describe Track, type: :model do
+  describe "Factory" do
+    it "can create" do
+      create(:track)
+      expect(Track.count).to eq(1)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -22,6 +22,9 @@ RSpec.configure do |config|
   # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
   config.fixture_path = "#{::Rails.root}/spec/fixtures"
 
+  config.filter_run :focus => true
+  config.run_all_when_everything_filtered = true
+
   # If you're not using ActiveRecord, or you'd prefer not to run each of your
   # examples within a transaction, remove the following line or assign false
   # instead of true.

data/spec/support/query_counter.rb

@@ -0,0 +1,29 @@
+module ActiveRecord
+  class QueryCounter
+    cattr_accessor :query_count do
+      0
+    end
+
+    IGNORED_SQL = [/^PRAGMA (?!(table_info))/, /^SELECT currval/, /^SELECT CAST/, /^SELECT @@IDENTITY/, /^SELECT @@ROWCOUNT/, /^SAVEPOINT/, /^ROLLBACK TO SAVEPOINT/, /^RELEASE SAVEPOINT/, /^SHOW max_identifier_length/]
+
+    def call(name, start, finish, message_id, values)
+      # FIXME: this seems bad. we should probably have a better way to indicate
+      # the query was cached
+      unless 'CACHE' == values[:name]
+        self.class.query_count += 1 unless IGNORED_SQL.any? { |r| values[:sql] =~ r }
+      end
+    end
+  end
+end
+
+ActiveSupport::Notifications.subscribe('sql.active_record', ActiveRecord::QueryCounter.new)
+
+module ActiveRecord
+  class Base
+    def self.count_queries(&block)
+      ActiveRecord::QueryCounter.query_count = 0
+      yield
+      ActiveRecord::QueryCounter.query_count
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: listings
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.1.0
   prerelease: 
 platform: ruby
 authors:
@@ -198,14 +198,22 @@ files:
 - config/routes.rb
 - lib/listings/action_view_extensions.rb
 - lib/listings/base.rb
+- lib/listings/base_field_descriptor.rb
+- lib/listings/base_field_view.rb
 - lib/listings/column_descriptor.rb
 - lib/listings/column_view.rb
 - lib/listings/configuration.rb
 - lib/listings/configuration_methods.rb
 - lib/listings/dynamic_binding.rb
 - lib/listings/engine.rb
+- lib/listings/filter_descriptor.rb
+- lib/listings/filter_view.rb
 - lib/listings/kaminari_helpers_tag_patch.rb
 - lib/listings/scope_descriptor.rb
+- lib/listings/sources/active_record_data_source.rb
+- lib/listings/sources/data_source.rb
+- lib/listings/sources/object_data_source.rb
+- lib/listings/sources.rb
 - lib/listings/version.rb
 - lib/listings/view_helper_methods.rb
 - lib/listings.rb
@@ -226,7 +234,13 @@ files:
 - spec/dummy/app/listings/filtered_posts_listing.rb
 - spec/dummy/app/listings/hash_listing.rb
 - spec/dummy/app/listings/posts_listing.rb
+- spec/dummy/app/listings/tracks_fixed_order_listing.rb
+- spec/dummy/app/listings/tracks_listing.rb
+- spec/dummy/app/models/album.rb
+- spec/dummy/app/models/object_album.rb
+- spec/dummy/app/models/object_track.rb
 - spec/dummy/app/models/post.rb
+- spec/dummy/app/models/track.rb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/views/posts/filtered.html.haml
 - spec/dummy/app/views/posts/index.html.haml
@@ -234,6 +248,7 @@ files:
 - spec/dummy/app/views/welcome/array.html.haml
 - spec/dummy/app/views/welcome/hash.html.haml
 - spec/dummy/app/views/welcome/index.html.haml
+- spec/dummy/app/views/welcome/tracks.html.haml
 - spec/dummy/config/application.rb
 - spec/dummy/config/boot.rb
 - spec/dummy/config/database.yml
@@ -254,6 +269,8 @@ files:
 - spec/dummy/db/migrate/20130618143317_create_posts.rb
 - spec/dummy/db/migrate/20130618195559_add_author_to_posts.rb
 - spec/dummy/db/migrate/20140728151246_add_category_to_post.rb
+- spec/dummy/db/migrate/20150611185824_create_albums.rb
+- spec/dummy/db/migrate/20150611185922_create_tracks.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/db/test.sqlite3
@@ -295,11 +312,21 @@ files:
 - spec/dummy/tmp/cache/assets/E2E/B10/sprockets%2F4e77441cbd1f4fadabea28dd0be9a977
 - spec/dummy/tmp/cache/assets/E57/F10/sprockets%2Fb7a56dbe6cb518caaa4cb8eac55df770
 - spec/dummy/tmp/pids/server.pid
+- spec/factories/albums.rb
 - spec/factories/post.rb
+- spec/factories/tracks.rb
+- spec/factories/traits.rb
 - spec/lib/filter_parser_spec.rb
+- spec/lib/sources/active_record_data_source_spec.rb
+- spec/lib/sources/object_data_source_spec.rb
 - spec/listings/posts_listing_spec.rb
+- spec/listings/tracks_fixed_order_listing_spec.rb
+- spec/listings/tracks_listing_spec.rb
+- spec/models/album_spec.rb
 - spec/models/post_spec.rb
+- spec/models/track_spec.rb
 - spec/spec_helper.rb
+- spec/support/query_counter.rb
 homepage: http://github.com/manastech/listings
 licenses: []
 post_install_message: 
@@ -314,7 +341,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1318764667279982391
+      hash: -3833283271247279800
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -323,7 +350,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1318764667279982391
+      hash: -3833283271247279800
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23.2
@@ -342,7 +369,13 @@ test_files:
 - spec/dummy/app/listings/filtered_posts_listing.rb
 - spec/dummy/app/listings/hash_listing.rb
 - spec/dummy/app/listings/posts_listing.rb
+- spec/dummy/app/listings/tracks_fixed_order_listing.rb
+- spec/dummy/app/listings/tracks_listing.rb
+- spec/dummy/app/models/album.rb
+- spec/dummy/app/models/object_album.rb
+- spec/dummy/app/models/object_track.rb
 - spec/dummy/app/models/post.rb
+- spec/dummy/app/models/track.rb
 - spec/dummy/app/views/layouts/application.html.erb
 - spec/dummy/app/views/posts/filtered.html.haml
 - spec/dummy/app/views/posts/index.html.haml
@@ -350,6 +383,7 @@ test_files:
 - spec/dummy/app/views/welcome/array.html.haml
 - spec/dummy/app/views/welcome/hash.html.haml
 - spec/dummy/app/views/welcome/index.html.haml
+- spec/dummy/app/views/welcome/tracks.html.haml
 - spec/dummy/config/application.rb
 - spec/dummy/config/boot.rb
 - spec/dummy/config/database.yml
@@ -370,6 +404,8 @@ test_files:
 - spec/dummy/db/migrate/20130618143317_create_posts.rb
 - spec/dummy/db/migrate/20130618195559_add_author_to_posts.rb
 - spec/dummy/db/migrate/20140728151246_add_category_to_post.rb
+- spec/dummy/db/migrate/20150611185824_create_albums.rb
+- spec/dummy/db/migrate/20150611185922_create_tracks.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/db/test.sqlite3
@@ -411,8 +447,18 @@ test_files:
 - spec/dummy/tmp/cache/assets/E2E/B10/sprockets%2F4e77441cbd1f4fadabea28dd0be9a977
 - spec/dummy/tmp/cache/assets/E57/F10/sprockets%2Fb7a56dbe6cb518caaa4cb8eac55df770
 - spec/dummy/tmp/pids/server.pid
+- spec/factories/albums.rb
 - spec/factories/post.rb
+- spec/factories/tracks.rb
+- spec/factories/traits.rb
 - spec/lib/filter_parser_spec.rb
+- spec/lib/sources/active_record_data_source_spec.rb
+- spec/lib/sources/object_data_source_spec.rb
 - spec/listings/posts_listing_spec.rb
+- spec/listings/tracks_fixed_order_listing_spec.rb
+- spec/listings/tracks_listing_spec.rb
+- spec/models/album_spec.rb
 - spec/models/post_spec.rb
+- spec/models/track_spec.rb
 - spec/spec_helper.rb
+- spec/support/query_counter.rb