Skip to content

Wrap pending blocks in a temporary scope.

gitlab-qa-bot requested to merge core-issue-1208 into master

Created by: xaviershay

This causes mock expectations to be verified at the end of the pending block. Most often this is the expected behaviour, since the pending scope is expected to be failing for whatever is inside of it.

Is this the right place to put this configuration? It feels kind of dirty.

Trigger for this was https://github.com/rspec/rspec-core/pull/1267

Alternatively, we could just switch the two pending specs in here to use :skip instead. Unclear how many other people would be affected by this issue though.

Merge request reports