Skip to content

Fix #1149: Correctly report number of found step files#1150

Merged
nohwnd merged 1 commit into
pester:masterfrom
fourpastmidnight:fix-#1149-correctly-output-number-of-found-step-files
Dec 14, 2018
Merged

Fix #1149: Correctly report number of found step files#1150
nohwnd merged 1 commit into
pester:masterfrom
fourpastmidnight:fix-#1149-correctly-output-number-of-found-step-files

Conversation

@fourpastmidnight
Copy link
Copy Markdown
Contributor

1. General summary of the pull request

Fixes #1149

It appears that at one point, there was a variable named $StepFiles and
it was used to report how many step files from which step definitions
were loaded when displaying Verbose output. At some later time,
$StepFiles was removed and inlined into the foreach loop.

This commit restores the $StepFiles intermediate variable so that the
proper verbose output is displayed.

@fourpastmidnight fourpastmidnight force-pushed the fix-#1149-correctly-output-number-of-found-step-files branch 2 times, most recently from 5d90551 to 5d6b80c Compare December 6, 2018 19:59
Fixes pester#1149

It appears that at one point, there was a variable named $StepFiles and
it was used to report how many step files from which step definitions
were loaded when displaying Verbose output. At some later time,
$StepFiles was removed and inlined into the foreach loop.

This commit restores the $StepFiles intermediate variable so that the
proper verbose output is displayed.
@fourpastmidnight fourpastmidnight force-pushed the fix-#1149-correctly-output-number-of-found-step-files branch from 5d6b80c to 5baec63 Compare December 7, 2018 14:50
@nohwnd nohwnd merged commit de98fc8 into pester:master Dec 14, 2018
@fourpastmidnight fourpastmidnight deleted the fix-#1149-correctly-output-number-of-found-step-files branch December 16, 2018 20:51
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