no_fly_list 0.1.0 → 0.2.1

data/lib/no_fly_list/tagging_record.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
 module NoFlyList
-  # This module provides functionality for a tag table that contains global tags for a model.
+  # This module provides functionality for a tagging table that contains tags relationship for a model.
   #
-  # This concern can be included in models that represent tags to manage global tags across different records.
+  # This concern can be included in models that represent tags to manage the relationship between the tag and the model.
   #
   # @example Usage
-  #   class User::Tag < ApplicationRecord
+  #   class User::Tagging < ApplicationRecord
   #     include NoFlyList::TaggingModel
   #   end
   module TaggingRecord

data/lib/no_fly_list/test_helper.rb

@@ -1,15 +1,95 @@
 # frozen_string_literal: true
 
 module NoFlyList
-  # = NoFlyList Test Helper
-  #
-  # Include <tt>NoFlyList::TestHelper</tt> in your test case to get access to the assertion methods.
   module TestHelper
+    # Asserts that a model class is properly set up for tagging in the given context
     def assert_taggable_record(klass, *contexts)
-      assert klass.respond_to?(:has_tags), "#{klass} does not respond to has_tags"
       contexts.each do |context|
-        assert klass.new.respond_to?(:"#{context}_list"), "#{klass} does not respond to #{context}_list"
+        assert klass._no_fly_list.tag_contexts.key?(context),
+               "#{klass} should have #{context} in its tag contexts"
+
+        assert_respond_to klass.new, "#{context}_list",
+                          "#{klass} should respond to #{context}_list"
       end
     end
+
+    # Asserts that a model class has proper tagging setup for a specific context
+    def assert_tagging_context(klass, context, polymorphic: false)
+      singular_name = context.to_s.singularize
+
+      # Check the context is registered
+      assert klass._no_fly_list.tag_contexts.key?(context),
+             "#{context} is not registered as a tag context for #{klass}"
+
+      # Check configuration
+      context_config = klass._no_fly_list.tag_contexts[context]
+      assert_equal polymorphic, context_config[:polymorphic],
+                   "#{context} should #{polymorphic ? '' : 'not '}be configured as polymorphic"
+
+      # Check tagging associations
+      assert_respond_to klass.new, "#{singular_name}_taggings",
+                        "Missing taggings association for #{context}"
+      assert_respond_to klass.new, context,
+                        "Missing tags association for #{context}"
+
+      # Check if correct classes exist
+      if polymorphic
+        assert_polymorphic_tag_classes_exist(klass, context)
+      else
+        assert_local_tag_classes_exist(klass, context)
+      end
+    end
+
+    private
+
+    def assert_polymorphic_tag_classes_exist(tags_klass, tagging_klass)
+      # Verify they include the correct modules
+      assert tags_klass.include?(NoFlyList::ApplicationTag),
+             'Polymorphic Tag should include NoFlyList::ApplicationTag'
+      assert tagging_klass.include?(NoFlyList::ApplicationTagging),
+             'Polymorphic Tagging should include NoFlyList::ApplicationTagging'
+    end
+
+    def assert_local_tag_classes_exist(klass, context)
+      context.to_s.singularize
+
+      # Check tag class exists
+      tag_class = "#{klass.name}Tag"
+      assert tag_class.safe_constantize,
+             "Tag class #{tag_class} should exist"
+
+      # Check tagging class exists
+      tagging_class = "#{klass.name}::Tagging"
+      assert tagging_class.safe_constantize,
+             "Tagging class #{tagging_class} should exist"
+
+      # Verify they include the correct modules
+      assert tag_class.constantize.include?(NoFlyList::TagRecord),
+             "#{tag_class} should include NoFlyList::TagRecord"
+      assert tagging_class.constantize.include?(NoFlyList::TaggingRecord),
+             "#{tagging_class} should include NoFlyList::TaggingRecord"
+    end
+
+    # Asserts that a specific record has a tag in a given context
+    def assert_has_tag(record, tag_name, context)
+      tag_list = record.send("#{context}_list")
+      assert_includes tag_list.to_a, tag_name,
+                      "Expected #{record.class.name} ##{record.id} to have tag '#{tag_name}' in context '#{context}'"
+    end
+
+    # Asserts that a specific record does not have a tag in a given context
+    def assert_has_no_tag(record, tag_name, context)
+      tag_list = record.send("#{context}_list")
+      refute_includes tag_list.to_a, tag_name,
+                      "Expected #{record.class.name} ##{record.id} to not have tag '#{tag_name}' in context '#{context}'"
+    end
+
+    # Asserts that a specific record has exactly the given tags in a context
+    def assert_has_exactly_tags(record, tags, context)
+      actual_tags = record.send("#{context}_list").to_a.sort
+      expected_tags = Array(tags).sort
+      assert_equal expected_tags, actual_tags,
+                   "Expected #{record.class.name} ##{record.id} to have exactly #{expected_tags.inspect} in context '#{context}', but got #{actual_tags.inspect}"
+    end
   end
 end

data/lib/no_fly_list/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module NoFlyList
-  VERSION = '0.1.0'
+  VERSION = '0.2.1'
 end

data/lib/no_fly_list.rb

@@ -6,7 +6,6 @@ require 'active_support/rails'
 require 'active_support/core_ext/numeric/time'
 require_relative 'no_fly_list/version'
 require 'no_fly_list/railtie' if defined?(Rails)
-require 'ostruct'
 
 module NoFlyList
   extend ActiveSupport::Autoload
@@ -16,9 +15,17 @@ module NoFlyList
 
   # Common tagging tables
   autoload :TaggableRecord
+
   autoload_under 'taggable_record' do
     autoload :Configuration
+    autoload :Config
+    autoload_under 'taggable_record/query' do
+      autoload :SqliteStrategy
+      autoload :MysqlStrategy
+      autoload :PostgresqlStrategy
+    end
   end
+
   autoload :TaggingRecord
   autoload :TagRecord
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: no_fly_list
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.1
 platform: ruby
 authors:
 - Abdelkader Boudih
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-06 00:00:00.000000000 Z
+date: 2024-12-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -24,20 +24,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '7.2'
-- !ruby/object:Gem::Dependency
-  name: ostruct
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 description: Tagging system for ActiveRecord models inspired by the TSA
 email:
 - terminale@gmail.com
@@ -53,7 +39,7 @@ files:
 - lib/generators/no_fly_list/templates/create_application_tagging_table.rb.erb
 - lib/generators/no_fly_list/templates/create_tagging_table.rb.erb
 - lib/generators/no_fly_list/templates/tag_model.rb.erb
-- lib/generators/no_fly_list/templates/tag_parser.rb
+- lib/generators/no_fly_list/templates/tag_transformer.rb
 - lib/generators/no_fly_list/templates/tagging_model.rb.erb
 - lib/generators/no_fly_list/transformer_generator.rb
 - lib/no_fly_list.rb
@@ -63,9 +49,14 @@ files:
 - lib/no_fly_list/railties/tasks.rake
 - lib/no_fly_list/tag_record.rb
 - lib/no_fly_list/taggable_record.rb
+- lib/no_fly_list/taggable_record/config.rb
 - lib/no_fly_list/taggable_record/configuration.rb
 - lib/no_fly_list/taggable_record/mutation.rb
 - lib/no_fly_list/taggable_record/query.rb
+- lib/no_fly_list/taggable_record/query/mysql_strategy.rb
+- lib/no_fly_list/taggable_record/query/postgresql_strategy.rb
+- lib/no_fly_list/taggable_record/query/sqlite_strategy.rb
+- lib/no_fly_list/taggable_record/tag_setup.rb
 - lib/no_fly_list/tagging_proxy.rb
 - lib/no_fly_list/tagging_record.rb
 - lib/no_fly_list/test_helper.rb