Use RSpec example block argument instead of the global RSpec.current_example. This allows to run tests with the async-rspec gem.

[ArturT]

Use RSpec example from RSpec each hook instead of global RSpec.current_example. It allows running tests with async-rspec gem.

Related

#116

details

I noticed when I downgrade RSpec to 2.x version in the example repo https://github.com/KnapsackPro/rails-app-with-knapsack then we can't record tests due to bug:

rails-app-with-knapsack rspec-current-example-group! ruby-3.0.1
$ KNAPSACK_GENERATE_REPORT=true bundle exec rspec spec
Knapsack report generator started!
/Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/hooks.rb:512:in `extract_scope_from': You must explicitly give a scope (:each, :all, or :suite) when using symbols as metadata for a hook. (ArgumentError)
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/hooks.rb:505:in `scope_and_options_from'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/hooks.rb:483:in `register_hook'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/hooks.rb:328:in `prepend_before'
	from /Users/artur/Documents/github/knapsack-pro/knapsack/lib/knapsack/adapters/rspec_adapter.rb:9:in `block in bind_time_tracker'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core.rb:154:in `configure'
	from /Users/artur/Documents/github/knapsack-pro/knapsack/lib/knapsack/adapters/rspec_adapter.rb:8:in `bind_time_tracker'
	from /Users/artur/Documents/github/knapsack-pro/knapsack/lib/knapsack/adapters/base_adapter.rb:19:in `bind'
	from /Users/artur/Documents/github/knapsack-pro/knapsack/lib/knapsack/adapters/base_adapter.rb:10:in `bind'
	from /Users/artur/Documents/github/knapsack-pro/rails-app-with-knapsack/spec/spec_helper.rb:8:in `<top (required)>'
	from /Users/artur/Documents/github/knapsack-pro/rails-app-with-knapsack/spec/rails_helper.rb:3:in `require'
	from /Users/artur/Documents/github/knapsack-pro/rails-app-with-knapsack/spec/rails_helper.rb:3:in `<top (required)>'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/configuration.rb:1036:in `require'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/configuration.rb:1036:in `block in setup_load_path_and_require'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/configuration.rb:1036:in `each'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/configuration.rb:1036:in `setup_load_path_and_require'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/configuration_options.rb:25:in `configure'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/command_line.rb:17:in `run'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/runner.rb:103:in `run'
	from /Users/artur/.rvm/gems/ruby-3.0.1/gems/rspec-core-2.99.2/lib/rspec/core/runner.rb:17:in `block in autorun'

It looks like we introduced breaking change for RSpec 2.x when we added context hook in PR #107

changes

• As part of this PR we drop support for RSpec 2.x
• we added support for async-rspec gem

[shadre]

I propose to add similar lingo changes as in the correspondend PR from knapsack_pro