Skip to content

Fix selector ordering issue#1375

Merged
leonard84 merged 2 commits intospockframework:masterfrom
marcphilipp:selector_order_issue
Oct 3, 2021
Merged

Fix selector ordering issue#1375
leonard84 merged 2 commits intospockframework:masterfrom
marcphilipp:selector_order_issue

Conversation

@marcphilipp
Copy link
Member

Prior to this commit the callback that is called when an entire test
class is selected enabled all feature methods on the wrong SpecInfo
instance which was part of the closure.

leonard84 and others added 2 commits September 29, 2021 20:08
Prior to this commit the callback that is called when an entire test
class is selected enabled all feature methods on the wrong `SpecInfo`
instance which was part of the closure.

Co-authored-by: Leonard Brünings <lbruenings@gradle.com>
@marcphilipp marcphilipp requested a review from leonard84 October 3, 2021 12:01
@marcphilipp marcphilipp self-assigned this Oct 3, 2021
@codecov
Copy link

codecov bot commented Oct 3, 2021

Codecov Report

Merging #1375 (f91cf88) into master (04690f7) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1375   +/-   ##
=========================================
  Coverage     79.26%   79.26%           
+ Complexity     3950     3948    -2     
=========================================
  Files           402      402           
  Lines         12290    12290           
  Branches       1625     1626    +1     
=========================================
  Hits           9742     9742           
  Misses         1965     1965           
  Partials        583      583           
Impacted Files Coverage Δ
.../spockframework/runtime/ClassSelectorResolver.java 96.15% <100.00%> (ø)

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 04690f7...f91cf88. Read the comment docs.

@leonard84 leonard84 merged commit 6cf09c6 into spockframework:master Oct 3, 2021
@marcphilipp marcphilipp deleted the selector_order_issue branch October 4, 2021 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants