Allow ordering specs by modification time

Author: MatheusRich

features/command_line/order.md

@@ -13,6 +13,8 @@ order of groups at each level is randomized.
 
 With `rand` you can also specify a seed.
 
+The option `modification_time` will run the most recent modified files first. You can combine it with `--next-failure` to find the most recent failing spec.
+
 ## Example usage
 
 The `defined` option is only necessary when you have `--order rand` stored in a
@@ -22,4 +24,5 @@ config file (e.g. `.rspec`) and you want to override it from the command line.
 --order rand
 --order rand:123
 --seed 123 # same as --order rand:123
+--modification_time
 </code></pre>

lib/rspec/core/option_parser.rb

@@ -58,10 +58,11 @@ def parser(options)
         end
 
         parser.on('--order TYPE[:SEED]', 'Run examples by the specified order type.',
-                  '  [defined] examples and groups are run in the order they are defined',
-                  '  [rand]    randomize the order of groups and examples',
-                  '  [random]  alias for rand',
-                  '  [random:SEED] e.g. --order random:123') do |o|
+                  '  [defined]           examples and groups are run in the order they are defined',
+                  '  [rand]              randomize the order of groups and examples',
+                  '  [random]            alias for rand',
+                  '  [random:SEED]       e.g. --order random:123',
+                  '  [modification_time] run the most recently modified files first') do |o|
           options[:order] = o
         end
 

lib/rspec/core/ordering.rb

@@ -58,6 +58,14 @@ def jenkins_hash_digest(string)
         MAX_32_BIT = 4_294_967_295
       end
 
+      # @private
+      # Orders items by modification time (most recent modified first).
+      class ModificationTime
+        def order(list)
+          list.sort_by { |item| -File.mtime(item.metadata[:absolute_file_path]).to_i }
+        end
+      end
+
       # @private
       # Orders items based on a custom block.
       class Custom
@@ -78,6 +86,7 @@ def initialize(configuration)
           @strategies    = {}
 
           register(:random,  Random.new(configuration))
+          register(:modification_time,  ModificationTime.new)
 
           identity = Identity.new
           register(:defined, identity)
@@ -132,6 +141,8 @@ def order=(type)
                             :random
                           elsif order == 'defined'
                             :defined
+                          elsif order == 'modification_time'
+                            :modification_time
                           end
 
           register_ordering(:global, ordering_registry.fetch(ordering_name)) if ordering_name

spec/rspec/core/ordering_spec.rb

@@ -81,6 +81,23 @@ def order_with(seed)
         end
       end
 
+      RSpec.describe ModificationTime do
+        require 'time'
+
+        before do
+          allow(File).to receive(:mtime).with('./file_1.rb').and_return(::Time.parse('18:00'))
+          allow(File).to receive(:mtime).with('./file_2.rb').and_return(::Time.parse('18:01'))
+        end
+
+        it 'orders list by file modification time' do
+          file_1 = instance_double(Example, :metadata => { :absolute_file_path => './file_1.rb' })
+          file_2 = instance_double(Example, :metadata => { :absolute_file_path => './file_2.rb' })
+          strategy = ModificationTime.new
+
+          expect(strategy.order([file_1, file_2])).to eq([file_2, file_1])
+        end
+      end
+
       RSpec.describe Custom do
         it 'uses the block to order the list' do
           strategy = Custom.new(proc { |list| list.reverse })