[Spring] Start and stop test context once per scenario

[mpkorstanje]

The TestContextAdaptor would start the scenario scope once at the start of
each test, then another one to run the after test method hooks. Refactored
code to make the order of operations more obvious.

Also prevented scenario scopes from being registered more then once. Though
because CucumberScenarioScope delegates everything to CucumberTestContext
this effectively only reduces the log spam a bit at debug level.

Fixes: #2516

[codecov]

Codecov Report

Merging #2517 (633f41f) into main (3df1505) will decrease coverage by 0.02%.
The diff coverage is 92.85%.

❗ Current head 633f41f differs from pull request most recent head 6c85990. Consider uploading reports for the commit 6c85990 to get more accurate results

@@             Coverage Diff              @@
##               main    #2517      +/-   ##
============================================
- Coverage     83.80%   83.77%   -0.03%     
- Complexity     2649     2690      +41     
============================================
  Files           309      319      +10     
  Lines          9354     9475     +121     
  Branches        912      918       +6     
============================================
+ Hits           7839     7938      +99     
- Misses         1182     1200      +18     
- Partials        333      337       +4     

Impacted Files	Coverage Δ
...java/io/cucumber/spring/CucumberScenarioScope.java	46.66% <0.00%> (ø)
...n/java/io/cucumber/spring/CucumberTestContext.java	70.96% <100.00%> (+3.22%)	⬆️
...rc/main/java/io/cucumber/spring/SpringFactory.java	91.66% <100.00%> (ø)
...in/java/io/cucumber/spring/TestContextAdaptor.java	87.14% <100.00%> (+0.77%)	⬆️
.../io/cucumber/examples/wicket/main/view/Create.java	71.42% <0.00%> (ø)
.../io/cucumber/examples/wicket/main/Application.java	92.85% <0.00%> (ø)
...ucumber/examples/wicket/main/model/entity/Car.java	100.00% <0.00%> (ø)
...examples/wicket/main/model/dao/InMemoryCarDAO.java	70.58% <0.00%> (ø)
.../io/cucumber/examples/spring/application/User.java	100.00% <0.00%> (ø)
.../cucumber/examples/spring/application/Message.java	75.00% <0.00%> (ø)
... and 5 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3df1505...6c85990. Read the comment docs.