klenod-build 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3c7b7667bd38de08f8f31ebb2aecf97bcbde81d8e189dae356ac64651dd340b
-  data.tar.gz: 55d2493b7d0047d97f9fb26766f7f6a1ed07867e1b1024c04d92dac3e04088e7
+  metadata.gz: 560774e128fafb845afbff51f556e3eb30d3a1b7d280b9fbdc7ac36b48c4b741
+  data.tar.gz: 559ce8134726f2fa31084811a9dcda01987a957c073370f5ac0595c8d7a3f703
 SHA512:
-  metadata.gz: ce9d72bdc5127d334c72baa154e6d16997081c3e3a7f016074ec41088a16007b4c35c93d4bfb96a220fd950539b0e3496d7b3c9b33439c6777d5760cad2e2715
-  data.tar.gz: 156d7564ff2e42c7831585790e906598510f5b27c74de16378f06870e792baa91f15101abe932212b6669a7a368d5b852c9f1be2cdeae5cc75ffa2f951eff46f
+  metadata.gz: 77cb936bd77e35262827d9903124f1b37cec73bb11db28c1a5e497d02a20d9283e989a46524c2f674b7c0f49734632e3da02b324a4988dbccbe3fe244e591921
+  data.tar.gz: eccc054b80b674a4f304fb6ec2abbdaf3bdf244576dc0f9bfd7b0d23b7b35ac77f5a47a061ad013cd9743295606b7ca3526a50e8f2dff48cc86ba13d4dac6fc2

data/lib/klenod/build/errors.rb

@@ -161,6 +161,7 @@ module Klenod
     end
 
     class DynamicImportError < Error; end
+    class TestImportError < ResolveError; end
     class UnsupportedFileError < Error; end
 
     class ImportCycleError < Error

data/lib/klenod/build/plugins/haml_plugin.rb

@@ -30,6 +30,7 @@ module Klenod
         DEFAULT_FACTORY = ComponentDefaults::DEFAULT_FACTORY
         DEFAULT_COMPONENT_CHILDREN = :eager
         COMPONENT_CHILDREN_MODES = %i[eager lazy].freeze
+        I18N_OPTIONS = %i[class constant].freeze
         HAML_HELPER_SPECIFIER = "virtual:klenod/haml_helper"
         HAML_HELPER_MODULE_ID = ModuleId.new("#{HAML_HELPER_SPECIFIER}.rb", nil)
         STATIC_CLASS_SOURCE_PATTERN = /^[ \t]*(?:%[-:\w]+)?(?:#[\w-]+)?\.[\w-]|[({][^)}\n]*\bclass\s*=/m
@@ -47,6 +48,7 @@ module Klenod
           DEFAULT_FACTORY = HamlPlugin::DEFAULT_FACTORY
           DEFAULT_COMPONENT_CHILDREN = HamlPlugin::DEFAULT_COMPONENT_CHILDREN
           COMPONENT_CHILDREN_MODES = HamlPlugin::COMPONENT_CHILDREN_MODES
+          I18N_OPTIONS = HamlPlugin::I18N_OPTIONS
           HAML_HELPER_SPECIFIER = HamlPlugin::HAML_HELPER_SPECIFIER
           HAML_HELPER_MODULE_ID = HamlPlugin::HAML_HELPER_MODULE_ID
           STATIC_CLASS_SOURCE_PATTERN = HamlPlugin::STATIC_CLASS_SOURCE_PATTERN
@@ -64,15 +66,14 @@ module Klenod
             factory: DEFAULT_FACTORY,
             component_children: DEFAULT_COMPONENT_CHILDREN,
             variables: nil,
-            i18n_class: nil,
-            i18n_constant: nil,
+            i18n: nil,
             cache_static_subtrees: false
           )
             @component_base_class = component_base_class
             @factory = factory
             @component_children = validate_component_children(component_children)
             @variables = validate_variables(variables)
-            @i18n_class, @i18n_constant = validate_i18n_options(i18n_class, i18n_constant)
+            @i18n_class, @i18n_constant = validate_i18n_options(i18n)
             @cache_static_subtrees = cache_static_subtrees
             @transformer = Transformer.new
           end
@@ -296,15 +297,23 @@ module Klenod
             raise ArgumentError, "variables[#{kind.inspect}] must be a Ruby expression"
           end
 
-          def validate_i18n_options(i18n_class, i18n_constant)
-            return [nil, nil] if i18n_class.nil? && i18n_constant.nil?
-            raise ArgumentError, "i18n_class must be configured when i18n_constant is configured" if i18n_class.nil?
+          def validate_i18n_options(i18n)
+            return [nil, nil] if i18n.nil?
+            raise ArgumentError, "i18n must be a Hash" unless i18n.is_a?(Hash)
 
-            i18n_constant = (i18n_constant || "I18n").to_s
-            raise ArgumentError, "i18n_constant must be a Ruby constant name" unless i18n_constant.match?(/\A[A-Z]\w*\z/)
+            unknown_options = i18n.keys - I18N_OPTIONS
+            unless unknown_options.empty?
+              raise ArgumentError, "i18n contains unknown options: #{unknown_options.map(&:inspect).join(", ")}"
+            end
+
+            i18n_class = i18n[:class]
+            raise ArgumentError, "i18n[:class] must be configured" if i18n_class.nil?
+
+            i18n_constant = (i18n[:constant] || "I18n").to_s
+            raise ArgumentError, "i18n[:constant] must be a Ruby constant name" unless i18n_constant.match?(/\A[A-Z]\w*\z/)
 
             [
-              Transformer::ConstPath.parse(i18n_class, name: "i18n_class").to_s,
+              Transformer::ConstPath.parse(i18n_class, name: "i18n[:class]").to_s,
               i18n_constant
             ]
           end

data/lib/klenod/build/plugins/image_plugin.rb

@@ -191,6 +191,8 @@ module Klenod
                     super(src:, width:, height:, content_type:, aspect_ratio:, variants: variants.dup.freeze)
                   end
 
+                  alias to_s src
+
                   def srcset
                     return nil if variants.empty?
 
@@ -205,7 +207,9 @@ module Klenod
                   end
                 end
 
-              ImageVariant = Data.define(:src, :width, :height, :content_type, :aspect_ratio, :format, :descriptor, :quality)
+              ImageVariant = Data.define(:src, :width, :height, :content_type, :aspect_ratio, :format, :descriptor, :quality) do
+                alias to_s src
+              end
             RUBY
           end
 

data/lib/klenod/build/plugins/svg_plugin.rb

@@ -94,7 +94,9 @@ module Klenod
 
           def svg_runtime_source
             <<~RUBY
-              SvgMetadata = Data.define(:src, :width, :height, :content_type, :aspect_ratio)
+              SvgMetadata = Data.define(:src, :width, :height, :content_type, :aspect_ratio) do
+                alias to_s src
+              end
             RUBY
           end
 

data/lib/klenod/build/plugins/test_plugin.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+require_relative "../errors"
+require_relative "../module_id"
+require_relative "../plugin"
+
+module Klenod
+  module Build
+    module Plugins
+      module TestPlugin
+        def self.new(...)
+          Plugin.new(...)
+        end
+
+        class Plugin < Klenod::Build::Plugin
+          DEFAULT_PATTERN = "**/*.test.rb"
+
+          def initialize(pattern: DEFAULT_PATTERN)
+            @pattern = pattern.to_s
+          end
+
+          attr_reader :pattern
+
+          def discover(source_dir:)
+            root = Pathname.new(source_dir).expand_path
+
+            root
+              .glob(pattern)
+              .select(&:file?)
+              .map { |path| ModuleId.new(path.relative_path_from(root).to_s.tr("\\", "/"), nil) }
+              .sort_by(&:to_s)
+          end
+
+          def test_module_id?(module_id)
+            module_id.scheme == :app && File.fnmatch?(pattern, module_id.relative_path, File::FNM_PATHNAME | File::FNM_EXTGLOB)
+          end
+
+          def finalize(module_id, result, resolved_dependencies, _dependency_records, _context)
+            imported_test = resolved_dependencies.find { |dependency| test_module_id?(dependency.module_id) }
+            return result unless imported_test
+
+            error = TestImportError.new("Test file #{imported_test.module_id.path.inspect} cannot be imported")
+            raise error.with_resolution_context(dependency: imported_test.dependency, importer_id: module_id)
+          end
+        end
+      end
+    end
+  end
+end

data/lib/klenod/build/plugins.rb

@@ -17,6 +17,7 @@ module Klenod
       autoload :TomlPlugin, File.expand_path("plugins/data_plugin", __dir__)
       autoload :TextPlugin, File.expand_path("plugins/data_plugin", __dir__)
       autoload :RouterPlugin, File.expand_path("plugins/router_plugin", __dir__)
+      autoload :TestPlugin, File.expand_path("plugins/test_plugin", __dir__)
     end
   end
 end

data/lib/klenod/build/test_suite.rb

@@ -0,0 +1,94 @@
+# frozen_string_literal: true
+
+module Klenod
+  module Build
+    TestSelection = Data.define(:test_paths, :removed_test_paths) do
+      def empty?
+        test_paths.empty? && removed_test_paths.empty?
+      end
+    end
+
+    class TestSuite
+      def initialize(context:, plugin:)
+        @context = context
+        @plugin = plugin
+        @dependency_ids = {}
+        @failed_test_ids = Set.new
+      end
+
+      def collect
+        test_ids = discover
+        test_ids.each { |test_id| index(test_id) }
+        selection(test_ids)
+      end
+
+      def update(event)
+        previous_test_ids = @dependency_ids.keys.to_set
+        current_test_ids = discover.to_set
+        removed_test_ids = previous_test_ids - current_test_ids
+        added_test_ids = current_test_ids - previous_test_ids
+        affected_module_ids = update_module_ids(event)
+        affected_test_ids =
+          @dependency_ids.filter_map do |test_id, dependency_ids|
+            test_id if current_test_ids.include?(test_id) && dependency_ids.intersect?(affected_module_ids)
+          end
+
+        test_ids = (added_test_ids + affected_test_ids + @failed_test_ids).select { |test_id| current_test_ids.include?(test_id) }
+        removed_test_ids.each { |test_id| remove(test_id) }
+        test_ids.each { |test_id| index(test_id) }
+
+        selection(test_ids, removed_test_ids)
+      end
+
+      private
+
+      attr_reader :context, :plugin
+
+      def discover
+        plugin.discover(source_dir: context.graph.source_dir)
+      end
+
+      def index(test_id)
+        dependency_ids = Set[test_id]
+        @dependency_ids[test_id] = dependency_ids
+        collect_dependencies(test_id, dependency_ids)
+        @failed_test_ids.delete(test_id)
+      rescue
+        @failed_test_ids << test_id
+      end
+
+      def collect_dependencies(module_id, dependency_ids)
+        record = context.graph.records[module_id] || context.graph.collect_module(module_id)
+
+        record.resolved_dependencies.each do |dependency|
+          dependency_id = dependency.module_id
+          next unless dependency_ids.add?(dependency_id)
+
+          collect_dependencies(dependency_id, dependency_ids)
+        end
+      end
+
+      def remove(test_id)
+        @dependency_ids.delete(test_id)
+        @failed_test_ids.delete(test_id)
+      end
+
+      def update_module_ids(event)
+        result = event.result
+        Set.new(
+          result.changed_module_ids +
+            result.removed_module_ids +
+            result.reloaded_module_ids +
+            result.reevaluated_module_ids
+        )
+      end
+
+      def selection(test_ids, removed_test_ids = [])
+        TestSelection.new(
+          test_ids.map(&:relative_path).sort.freeze,
+          removed_test_ids.map(&:relative_path).sort.freeze
+        )
+      end
+    end
+  end
+end

data/lib/klenod/build/version.rb

@@ -2,6 +2,6 @@
 
 module Klenod
   module Build
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

data/lib/klenod/build/watcher.rb

@@ -53,10 +53,11 @@ module Klenod
         end
       end
 
-      def initialize(source_dir:, context:, debounce_interval: 0.1)
+      def initialize(source_dir:, context:, debounce_interval: 0.1, ignore: nil)
         @source_dir = source_dir
         @context = context
         @debounce_interval = debounce_interval
+        @ignore = ignore
         @graph_version = 0
         @pending_changes = PendingChanges.new
         @pending_mutex = Mutex.new
@@ -68,7 +69,7 @@ module Klenod
       def start
         @worker_thread = Thread.new { process_pending_updates }
         @listener =
-          Listen.to(@source_dir) do |modified, added, removed|
+          Listen.to(@source_dir, ignore: @ignore) do |modified, added, removed|
             enqueue_update(modified + added, removed)
           end
         @listener.start

data/lib/klenod/build.rb

@@ -9,6 +9,8 @@ module Klenod
 
   module Build
     autoload :Graphviz, "klenod/build/graphviz"
+    autoload :TestSelection, "klenod/build/test_suite"
+    autoload :TestSuite, "klenod/build/test_suite"
     autoload :UpdateEvent, "klenod/build/watcher"
     autoload :Watcher, "klenod/build/watcher"
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: klenod-build
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Andrés Alin
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.3
+        version: 0.0.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 0.0.3
+        version: 0.0.4
 - !ruby/object:Gem::Dependency
   name: async
   requirement: !ruby/object:Gem::Requirement
@@ -272,6 +272,7 @@ files:
 - lib/klenod/build/plugins/router_plugin.rb
 - lib/klenod/build/plugins/ruby_plugin.rb
 - lib/klenod/build/plugins/svg_plugin.rb
+- lib/klenod/build/plugins/test_plugin.rb
 - lib/klenod/build/profiler.rb
 - lib/klenod/build/resolution_error_formatter.rb
 - lib/klenod/build/resolver.rb
@@ -280,6 +281,7 @@ files:
 - lib/klenod/build/source_map/editor.rb
 - lib/klenod/build/source_map/map.rb
 - lib/klenod/build/source_map/vlq.rb
+- lib/klenod/build/test_suite.rb
 - lib/klenod/build/transform_result.rb
 - lib/klenod/build/version.rb
 - lib/klenod/build/watched_pattern.rb