Restore reported run time to begin after files are loaded.

- was causing problems for spork users and others.
- closes rspec#475.

This reverts commit fd04952, with
a couple of minor changes/additions.

Changelog.md

@@ -13,6 +13,9 @@
 * Bug fixes
   * Make sure the `bar` in `--tag foo:bar` makes it to DRb (Aaron Gibralter)
   * Fix bug where full descriptions of groups nested 3 deep  were repeated.
+  * Restore report of time to run to start after files are loaded.
+    * fixes bug where run times were cumalitive in spork
+    * fixes compatibility with time-series metrics
 
 * Deprecations
   * :alias option on `configuration.add_setting`. Use `:alias_with` on the
@@ -50,6 +53,7 @@ As of 2.7.0, you must explicity `require "rspec/autorun"` unless you use the
   * Add `--no-drb` option (Iain Hecker)
   * Provide more accurate run time by registering start time before code
     is loaded (David Chelimsky)
+    * reverted in 2.8.0
   * Rake task default pattern finds specs in symlinked dirs (Kelly Felkins)
   * Rake task no longer does anything to invoke bundler since Bundler already
     handles it for us. Thanks to Andre Arko for the tip.

lib/rspec/core.rb

@@ -1,4 +1,3 @@
-$rspec_start_time ||= Time.now
 require 'rspec/core/dsl'
 require 'rspec/core/extensions'
 require 'rspec/core/load_path'

lib/rspec/core/reporter.rb

@@ -3,7 +3,7 @@ class Reporter
     def initialize(*formatters)
       @formatters = formatters
       @example_count = @failure_count = @pending_count = 0
-      @duration = nil
+      @duration = @start = nil
     end
 
     def report(count, seed)
@@ -31,6 +31,7 @@ def finish(seed)
     alias_method :abort, :finish
 
     def start(expected_example_count)
+      @start = Time.now
       notify :start, expected_example_count
     end
 
@@ -66,7 +67,7 @@ def example_pending(example)
     end
 
     def stop
-      @duration = Time.now - $rspec_start_time
+      @duration = Time.now - @start if @start
       notify :stop
     end