smartest 0.2.0.alpha4 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 944d35bc0a1be2293c8e276d16436693ff7dd6a2d37218f9f65af3b7a47f90f5
-  data.tar.gz: edd55fb6442a92e74146eaffc76a3b881463a16b4ce3c72a877734e0f6e36f52
+  metadata.gz: 9891230284d4770c57adc2ec2d28356aca9323ba07cd825c6a7f2fb219d6415a
+  data.tar.gz: 2cd3e5badccdc21d42a9b3ffd303053e75aa886dcaffefcb903cf84fc6483c90
 SHA512:
-  metadata.gz: 5406cfcb57b56c71140c98565da884a184b8d14dcd41f8d6f65f218010d5ef7672eee6c3ad874dcc22a2d415097c9f9926f3a22c31ea3f5dafcf6dd00c2c795d
-  data.tar.gz: '0666928711e5ec22a5f9d671714597eb9e0a7e98edcf11d334bf72301c42739a8ae71104dff5784fa4728c10c186188e9f5278137ebbf7d22f6e14e745964927'
+  metadata.gz: 1fe246549fe47ebb020820690b66937c6fa558be8444e8641f2c176a3ef9886370c17a260b25598cba8d3817adb37cdf331d7de7348c82561dedc993c8dc303e
+  data.tar.gz: bcb7e23e7fb872e3001d6683577981a03d5c5482d1d548d9f8f07d1e4f3b3d5766d08b4b3ccbb2c8f17be85c9c553e84c2f2983732e8ecc2966509704839f792

data/lib/smartest/helper_registry.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module Smartest
+  class HelperRegistry
+    include Enumerable
+
+    def self.validate!(helper_module)
+      unless helper_module.is_a?(Module) && !helper_module.is_a?(Class)
+        raise ArgumentError, "helper must be a module"
+      end
+    end
+
+    def initialize
+      @helper_modules = []
+    end
+
+    def add(helper_module)
+      self.class.validate!(helper_module)
+
+      @helper_modules << helper_module unless @helper_modules.include?(helper_module)
+    end
+
+    def each(&block)
+      @helper_modules.each(&block)
+    end
+
+    def to_a
+      @helper_modules.dup
+    end
+  end
+end

data/lib/smartest/hook_contexts.rb

@@ -49,6 +49,10 @@ module Smartest
       @test_run.add_matcher_module(matcher_module)
     end
 
+    def use_helper(helper_module)
+      @test_run.add_helper_module(helper_module)
+    end
+
     def skip(reason = nil)
       raise Skipped, reason
     end

data/lib/smartest/runner.rb

@@ -73,8 +73,8 @@ module Smartest
       test_run = TestRun.new(
         fixture_classes: @suite.fixture_classes,
         matcher_modules: @suite.matcher_modules
-      ) do |fixture_classes:, matcher_modules:|
-        run_test_body(test_case, fixture_classes, matcher_modules, run_state, cleanup_errors)
+      ) do |fixture_classes:, matcher_modules:, helper_modules:|
+        run_test_body(test_case, fixture_classes, matcher_modules, helper_modules, run_state, cleanup_errors)
       end
 
       begin
@@ -114,8 +114,8 @@ module Smartest
       end
     end
 
-    def run_test_body(test_case, fixture_classes, matcher_modules, run_state, cleanup_errors)
-      context = build_context(matcher_modules, run_state)
+    def run_test_body(test_case, fixture_classes, matcher_modules, helper_modules, run_state, cleanup_errors)
+      context = build_context(matcher_modules, run_state, helper_modules)
       fixture_set = nil
 
       begin
@@ -152,12 +152,22 @@ module Smartest
       )
     end
 
-    def build_context(matcher_modules = @suite.matcher_modules, run_state = TestRunState.new)
+    def build_context(matcher_modules = @suite.matcher_modules, run_state = TestRunState.new, helper_modules = [])
       ExecutionContext.new(run_state: run_state).tap do |context|
+        helper_modules.each { |helper_module| extend_helper_module(context, helper_module) }
         matcher_modules.each { |matcher_module| context.extend(matcher_module) }
       end
     end
 
+    def extend_helper_module(context, helper_module)
+      context.extend(helper_module)
+
+      helper_methods = helper_module.public_instance_methods + helper_module.protected_instance_methods
+      return if helper_methods.empty?
+
+      context.singleton_class.class_eval { private(*helper_methods) }
+    end
+
     def around_test_protocol_error?(error)
       error.is_a?(AroundTestRunError)
     end

data/lib/smartest/test_run.rb

@@ -4,13 +4,16 @@ module Smartest
   class TestRun
     attr_reader :result
 
-    def initialize(fixture_classes:, matcher_modules:, &block)
+    def initialize(fixture_classes:, matcher_modules:, helper_modules: [], &block)
       @fixture_classes = FixtureClassRegistry.new
       fixture_classes.each { |fixture_class| @fixture_classes.add(fixture_class) }
 
       @matcher_modules = MatcherRegistry.new
       matcher_modules.each { |matcher_module| @matcher_modules.add(matcher_module) }
 
+      @helper_modules = HelperRegistry.new
+      helper_modules.each { |helper_module| @helper_modules.add(helper_module) }
+
       @block = block
       @ran = false
       @result = nil
@@ -22,7 +25,8 @@ module Smartest
       @ran = true
       @result = @block.call(
         fixture_classes: @fixture_classes,
-        matcher_modules: @matcher_modules
+        matcher_modules: @matcher_modules,
+        helper_modules: @helper_modules
       )
     end
 
@@ -43,6 +47,12 @@ module Smartest
       @matcher_modules.add(matcher_module)
     end
 
+    def add_helper_module(helper_module)
+      raise AroundTestRunError, "use_helper must be called before test.run" if ran?
+
+      @helper_modules.add(helper_module)
+    end
+
     private
 
     def reject_suite_fixture_class!(klass)

data/lib/smartest/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Smartest
-  VERSION = "0.2.0.alpha4"
+  VERSION = "0.2.0"
 end

data/lib/smartest.rb

@@ -9,6 +9,7 @@ require_relative "smartest/fixture_definition"
 require_relative "smartest/fixture"
 require_relative "smartest/fixture_class_registry"
 require_relative "smartest/matcher_registry"
+require_relative "smartest/helper_registry"
 require_relative "smartest/fixture_set"
 require_relative "smartest/suite"
 require_relative "smartest/suite_run"

data/smartest/smartest_test.rb

@@ -517,6 +517,22 @@ test("rejects class matcher registrations") do
   expect(error.message).to include("matcher must be a module")
 end
 
+test("rejects non-module helper registrations") do
+  error = SmartestSelfTest.capture_error(ArgumentError) do
+    Smartest::HelperRegistry.new.add(Object.new)
+  end
+
+  expect(error.message).to include("helper must be a module")
+end
+
+test("rejects class helper registrations") do
+  error = SmartestSelfTest.capture_error(ArgumentError) do
+    Smartest::HelperRegistry.new.add(Class.new)
+  end
+
+  expect(error.message).to include("helper must be a module")
+end
+
 test("resolves keyword fixture dependencies per test") do
   calls = []
 
@@ -912,6 +928,73 @@ test("around_test can register matchers for one test run") do
   expect(status).to eq(0)
 end
 
+test("around_test can register private helpers for one test run") do
+  helper_module = Module.new do
+    def local_helper_value
+      "local"
+    end
+  end
+
+  suite = Smartest::Suite.new
+  suite.tests.add(
+    Smartest::TestCase.new(
+      name: "uses local helper",
+      metadata: {},
+      location: caller_locations(1, 1).first,
+      around_test_hooks: [
+        proc do |test_run|
+          use_helper helper_module
+          test_run.run
+        end
+      ],
+      block: proc do
+        expect(local_helper_value).to eq("local")
+        expect(private_methods).to include(:local_helper_value)
+        expect(public_methods).not_to include(:local_helper_value)
+      end
+    )
+  )
+  suite.tests.add(
+    SmartestSelfTest.test_case(
+      "without helper",
+      proc { expect(respond_to?(:local_helper_value, true)).to eq(false) }
+    )
+  )
+
+  status, = SmartestSelfTest.run_suite(suite)
+
+  expect(status).to eq(0)
+end
+
+test("around_test rejects helper registrations after test.run") do
+  helper_module = Module.new do
+    def late_helper_value
+      "late"
+    end
+  end
+
+  suite = Smartest::Suite.new
+  suite.tests.add(
+    Smartest::TestCase.new(
+      name: "late helper",
+      metadata: {},
+      location: caller_locations(1, 1).first,
+      around_test_hooks: [
+        proc do |test_run|
+          test_run.run
+          use_helper helper_module
+        end
+      ],
+      block: proc { expect(true).to eq(true) }
+    )
+  )
+
+  status, output = SmartestSelfTest.run_suite(suite)
+
+  expect(status).to eq(1)
+  expect(output).to include("Smartest::AroundTestRunError: use_helper must be called before test.run")
+end
+
 test("around_test must call test.run") do
   suite = Smartest::Suite.new
   suite.tests.add(

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smartest
 version: !ruby/object:Gem::Version
-  version: 0.2.0.alpha4
+  version: 0.2.0
 platform: ruby
 authors:
 - Yusuke Iwaki
@@ -52,6 +52,7 @@ files:
 - lib/smartest/fixture_class_registry.rb
 - lib/smartest/fixture_definition.rb
 - lib/smartest/fixture_set.rb
+- lib/smartest/helper_registry.rb
 - lib/smartest/hook_contexts.rb
 - lib/smartest/init_generator.rb
 - lib/smartest/matcher_registry.rb
@@ -90,9 +91,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '3.1'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.4.19
 signing_key: