spectre-core 2.1.1 → 2.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 903623ea7a511af71aebe1919c91b0b298fed6819310d60a38dc8ad3fd74127d
-  data.tar.gz: 52e66bd3838d220bd3ab57fc91f81d3eff5c7c9c85c5651611a75e99ed133c44
+  metadata.gz: 3e75cee429c0765480965f1e1177c85f5ecf769651332e7167b1be5db8601cda
+  data.tar.gz: 6ad3c26cf386b69b57db775bec8fb8e1ae011a8d6abfb21a1aa250fec6198104
 SHA512:
-  metadata.gz: 97ac252558b5acbc8ad88995759632f69adf7a5b8dd38e373b1f8cf176a047e89019fd568ebd02d0b6a556e6d210499f25ea3f3d7ff74ccaa00f9796d0619c09
-  data.tar.gz: e12befe6472a287a0d94755853e123da29066aab9aeaa8e60e5366019776638730e274f9a0e9f4d3fd87023faaa9961b51fbf9ffc5cfb66edc524f68ec54d7ce
+  metadata.gz: '09f0c500ce8ed1416a418b75af0716a6f7bd1083abf3b2d402c0adf381c0ce60bf51e8cb8e06f6f26bb5d19c1c3e8d8034361bcc320821f82835d39b427189f5'
+  data.tar.gz: 23d0113096cb9b69f78f40b1ae5ad7c6aa9201593fa9ac3e885f95ad14f53a5343673cbd932aca8603904638b3ef10afea6f81a5e55d47167b604836dc5e310a

data/lib/spectre/assertion.rb

@@ -88,7 +88,7 @@ module Spectre
         @var_name = file_content
           .lines[location.lineno - 1]
           .strip
-          .match(/[\s\(]([^\s]+|\[.*\]|{.*})\.(to|not_to)[\s\(]/)
+          .match(/[\s(]([^\s]+|\[.*\]|{.*})\.(to|not_to)[\s(]/)
           .captures
           .first
           .strip

data/lib/spectre/helpers.rb

@@ -122,7 +122,7 @@ end
 module Spectre
   module Helpers
     class << self
-      def uuid length = niL
+      def uuid length = nil
         return SecureRandom.hex(length / 2) if length
 
         SecureRandom.uuid

data/lib/spectre/version.rb

@@ -1,3 +1,3 @@
 module Spectre
-  VERSION = '2.1.1'.freeze
+  VERSION = '2.1.3'.freeze
 end

data/lib/spectre.rb

@@ -258,13 +258,14 @@ module Spectre
       counter = 0
 
       specs
-        .group_by { |x| x.parent.root }
+        .group_by { |x| x.parent.root.name }
         .each_value do |spec_group|
-          spec_group.sort_by!(&:name)
-          spec_group.each do |spec|
-            spec_id = "[#{spec.name}]".send(@colors[counter % @colors.length])
-            @out.puts "#{spec_id} #{spec.full_desc} #{spec.tags.map { |x| "##{x}" }.join(' ').cyan}"
-          end
+          spec_group
+            .sort
+            .each do |spec|
+              spec_id = "[#{spec.name}]".send(@colors[counter % @colors.length])
+              @out.puts "#{spec_id} #{spec.full_desc} #{spec.tags.map { |x| "##{x}" }.join(' ').cyan}"
+            end
 
           counter += 1
         end
@@ -273,15 +274,15 @@ module Spectre
     ##
     # Outputs all the specs for all contexts
     #
-    def describe contexts, level = 0
-      contexts.each do |context|
+    def describe contexts, level = 0, parent: nil
+      contexts.select { |x| x.parent == parent }.each do |context|
         @out.puts("#{'  ' * level}#{context.desc.send(level.positive? ? :magenta : :blue)}")
 
         context.specs.each do |spec|
           @out.puts("#{'  ' * (level + 1)}#{spec.desc}")
         end
 
-        describe(context.children, level + 1)
+        describe(contexts, level + 1, parent: context)
       end
     end
 
@@ -802,7 +803,6 @@ module Spectre
     DEFAULT_ASYNC_NAME = 'default'
 
     @@current = nil
-    @@location_cache = {}
     @@skip_count = 0
 
     ##
@@ -1071,10 +1071,7 @@ module Spectre
     def await name = DEFAULT_ASYNC_NAME
       return unless @threads.key? name
 
-      threads = @threads[name].map(&:join)
-
-      @threads.delete(name)
-
+      threads = @threads.delete(name)
       threads.map(&:join)
     end
 
@@ -1185,6 +1182,19 @@ module Spectre
       @full_desc = "#{@parent.full_desc} #{@desc}"
     end
 
+    def <=>(other)
+      # Split on dash: text part and numeric part
+      self_parts = @name.split('-')
+      other_parts = other.name.split('-')
+
+      text_compare = self_parts[0] <=> other_parts[0]
+
+      # If the text parts are already different, we can return here
+      return text_compare unless text_compare.zero?
+
+      self_parts[1].to_i <=> other_parts[1].to_i
+    end
+
     ##
     # Creates a new +RunContext+ and executes the spec,
     # +before+ and +after+ blocks
@@ -1217,13 +1227,13 @@ module Spectre
   class DefinitionContext
     include Delegate
 
-    attr_reader :id, :name, :desc, :parent, :full_desc, :children, :specs, :file
+    attr_reader :id, :name, :desc, :parent, :full_desc, :specs, :file
 
-    def initialize desc, file, parent = nil
+    def initialize desc, file, engine, parent = nil
+      @engine = engine
       @parent = parent
       @desc = desc
       @file = file
-      @children = []
       @specs = []
 
       @setups = []
@@ -1236,6 +1246,8 @@ module Spectre
       @name = @parent.name + '-' + @name unless @parent.nil?
 
       @full_desc = @parent.nil? ? @desc : "#{@parent.full_desc} #{@desc}"
+
+      @engine.contexts << self
     end
 
     ##
@@ -1245,13 +1257,6 @@ module Spectre
       @parent ? @parent.root : self
     end
 
-    ##
-    # A flattened list of all specs including those of child contexts.
-    #
-    def all_specs
-      @specs + @children.map(&:all_specs).flatten
-    end
-
     ##
     # Creates a new sub context with the given block.
     #
@@ -1261,8 +1266,7 @@ module Spectre
         .gsub(/:in .*/, '')
         .gsub(Dir.pwd, '.')
 
-      context = DefinitionContext.new(desc, file, self)
-      @children << context
+      context = DefinitionContext.new(desc, file, @engine, self)
       context.instance_eval(&)
     end
 
@@ -1313,8 +1317,14 @@ module Spectre
 
       with ||= [nil]
 
-      with.each_with_index do |data, _index|
-        spec_index = root.all_specs.count + 1
+      initial_index = @engine
+        .contexts
+        .select { |x| x.root.name == root.name }
+        .flat_map(&:specs)
+        .count + 1
+
+      with.each_with_index do |data, index|
+        spec_index = initial_index + index
         name = "#{root.name}-#{spec_index}"
 
         spec = Specification.new(self, name, desc, tags, data, file, block)
@@ -1324,23 +1334,23 @@ module Spectre
     end
 
     # :nodoc:
-    def run engine, specs
+    def run specs
       runs = []
 
-      return runs unless all_specs.any? { |x| specs.include? x }
-
       selected = @specs.select { |x| specs.include? x }
 
-      engine.formatter.scope(@desc, self) do
+      return runs if selected.empty?
+
+      @engine.formatter.scope(@desc, self) do
         if selected.any?
           setup_bag = nil
 
           if @setups.any?
-            setup_run = RunContext.new(engine, self, :setup) do |run_context|
+            setup_run = RunContext.new(@engine, self, :setup) do |run_context|
               @setups.each do |block|
-                engine.formatter.scope('setup', :setup) do
-                  engine.logger.correlate do
-                    engine.logger.debug("setup \"#{@desc}\"")
+                @engine.formatter.scope('setup', :setup) do
+                  @engine.logger.correlate do
+                    @engine.logger.debug("setup \"#{@desc}\"")
                     run_context.execute(nil, &block)
                   end
                 end
@@ -1355,18 +1365,18 @@ module Spectre
           # Only run specs if setup was successful
           if runs.all? { |x| x.status == :success }
             runs += selected.map do |spec|
-              engine.logger.correlate do
-                spec.run(engine, @befores, @afters, setup_bag)
+              @engine.logger.correlate do
+                spec.run(@engine, @befores, @afters, setup_bag)
               end
             end
           end
 
           if @teardowns.any?
-            runs << RunContext.new(engine, self, :teardown, setup_bag) do |run_context|
+            runs << RunContext.new(@engine, self, :teardown, setup_bag) do |run_context|
               @teardowns.each do |block|
-                engine.formatter.scope('teardown', :teardown) do
-                  engine.logger.correlate do
-                    engine.logger.debug("teardown \"#{@desc}\"")
+                @engine.formatter.scope('teardown', :teardown) do
+                  @engine.logger.correlate do
+                    @engine.logger.debug("teardown \"#{@desc}\"")
                     run_context.execute(nil, &block)
                   end
                 end
@@ -1375,9 +1385,12 @@ module Spectre
           end
         end
 
-        @children.each do |context|
-          engine.logger.correlate do
-            runs += context.run(engine, specs)
+        @engine
+          .contexts
+          .select { |x| x.parent == self }
+          .each do |context|
+          @engine.logger.correlate do
+            runs += context.run(specs)
           end
         end
       end
@@ -1554,9 +1567,11 @@ module Spectre
         resource_files = Dir.glob File.join(resource_path, '**/*')
 
         resource_files.each do |file|
-          file.slice! resource_path
-          file = file[1..]
-          @resources[file] = File.expand_path File.join(resource_path, file)
+          relative_file = file
+            .delete_prefix(resource_path)
+            .delete_prefix('/')
+
+          @resources[relative_file] = File.expand_path(file)
         end
       end
 
@@ -1601,8 +1616,7 @@ module Spectre
       tag_filter = config['tags'] || []
 
       @contexts
-        .map(&:all_specs)
-        .flatten
+        .flat_map(&:specs)
         .select do |spec|
           (spec_filter.empty? and tag_filter.empty?) or
             spec_filter.any? { |x| spec.name.match?("^#{x.gsub('*', '.*')}$") } or
@@ -1624,10 +1638,9 @@ module Spectre
 
       list
         .group_by { |x| x.parent.root }
-        .map do |context, specs|
-          context.run(self, specs)
+        .flat_map do |context, specs|
+          context.run(specs)
         end
-        .flatten
     rescue Interrupt
       # Do nothing here
     end
@@ -1661,20 +1674,15 @@ module Spectre
     ##
     # Describe a test subject
     #
-    def describe(name, &)
+    def describe(desc, &)
       file = caller
         .first
         .gsub(/:in .*/, '')
         .gsub(Dir.pwd, '.')
 
-      main_context = @contexts.find { |x| x.desc == name }
-
-      if main_context.nil?
-        main_context = DefinitionContext.new(name, file)
-        @contexts << main_context
-      end
-
-      main_context.instance_eval(&)
+      DefinitionContext
+        .new(desc, file, self)
+        .instance_eval(&)
     end
 
     ##

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spectre-core
 version: !ruby/object:Gem::Version
-  version: 2.1.1
+  version: 2.1.3
 platform: ruby
 authors:
 - Christian Neubauer
@@ -13,72 +13,86 @@ dependencies:
   name: debug
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: ectoplasm
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.4'
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fiddle
+  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'
 - !ruby/object:Gem::Dependency
   name: logger
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: ostruct
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '0.1'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: stringio
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '0'
 description: A DSL and command line tool to describe and run automated tests
 email:
 - christian.neubauer@ionos.com