Track time in before and after :context

[caalberts]

This PR changes the start and end of the timer to be done in prepend_before(:context) and append_after(:context) so that the time tracked by Knapsack includes time spent in before(:context) and after(:context).

Closes #106

Given a spec file as follows:

RSpec.describe 'Foo' do
  context 'when something' do
    before(:context) do
      sleep(5)
    end

    before do
      sleep(0.1)
    end

    it 'test_1' do
      sleep(0.2)
      expect(true).to be_truthy
    end

    it 'test_2' do
      sleep(0.3)
      expect(true).to be_truthy
    end
  end
end

Finished in 5.72 seconds (files took 0.74649 seconds to load)
2 examples, 0 failures

The test takes a total of 5.7 seconds (5 + 2*0.1 + 0.2 + 0.3), as recorded by RSpec. Knapsack should also track this file as 5.7 seconds.

[caalberts]

@ArturT Could you review please?

[ArturT]

Nice improvement with ordered. I did not know about it
https://relishapp.com/rspec/rspec-mocks/docs/setting-constraints/message-order

[ArturT]

@caalberts Thank you for the PR. I've released a new version of knapsack 1.21.0.

[ArturT]

Note

context hooks seems to be introduced in RSpec 3.0
https://relishapp.com/rspec/rspec-core/v/3-0/docs/hooks/before-and-after-hooks

There is no info about context in RSpec 2.99
https://relishapp.com/rspec/rspec-core/v/2-99/docs/hooks/before-and-after-hooks

Just info for myself that this change might not work for legacy RSpec 2.x. But that's fine. RSpec 2.x was released last time in 2014 (long time ago).
https://github.com/rspec/rspec-core/blob/main/Changelog.md#2992--2014-08-19

[ArturT]

This PR introduced a new bug #108

I'm working on solving that.

[ArturT]

It's fixed and knapsack 1.21.1 has been released.