Skip to content

Import InvokePesterStub.ps1 from vscode-powershell (with history) #1513

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
Jun 28, 2021

Conversation

andyleejordan
Copy link
Member

I used git filter-repo to export the history and adjust the path of this script, so that I could then use git merge --allow-unrelated-history to merge said script in with its development history intact.

rkeithhill and others added 8 commits March 5, 2019 14:28
)

* Migrate Pester version detection into an InovkePester stub script

* Move TestName warning into script

* Improve stub script, move module loaded check to beginning

* Address Codacy issue

* Address PR feedback

* Move TestName check before LineNumber in case of multiple blocks/line
* Run single test

* Add options for code lens and verbosity

* Rename Pester options to enableLegacyCodeLens and outputVerbosity

* change to use

Co-authored-by: Tyler Leonhardt <[email protected]>
* Fix renamed setting

* Fall through when legacy option is disablebd but Pester 5 is not found
…2730)

* Support adding OutputPath and run file

* refactor all logic
* Fixed crash when running pester older than 3.4.5

Calling Invoke-Pester with the -show param was
breaking tests if running a version of pester
older than 3.4.5

Moved logic for building spalt to call
invoke-pester into a function and only added the
show parameter if running pester 3.4.5 or newer

* Update InvokePesterStub.ps1

Co-authored-by: Tyler James Leonhardt <[email protected]>

* Update InvokePesterStub.ps1

Co-authored-by: Tyler James Leonhardt <[email protected]>

Co-authored-by: Tyler James Leonhardt <[email protected]>
I used `git filter-repo` to export the history and adjust the path of this
script, so that I could then use `git merge --allow-unrelated-history` to merge
said script in with its development history intact.
@andyleejordan andyleejordan marked this pull request as ready for review June 25, 2021 19:20
@andyleejordan andyleejordan requested a review from rjmholt as a code owner June 25, 2021 19:20
@andyleejordan
Copy link
Member Author

andyleejordan commented Jun 25, 2021

Ok, verified that this build signs it properly:

| InvokePesterStub.ps1      | Pass   | 6.66KB   | 21.56          | 0.87        | 0.48        | 17.24         | 0           | 

Now checking that the downstream build picks up the packaged artifact and works.

@andyleejordan
Copy link
Member Author

Ok! All confirmed. The file is properly signed by this build, then incorporated into the module, which then uses it from the correct path and the "Run Pester Tests" command works as expected. @rjmholt

@andyleejordan andyleejordan enabled auto-merge June 25, 2021 20:07
@andyleejordan andyleejordan merged commit bffc7ae into master Jun 28, 2021
@andyleejordan andyleejordan deleted the andschwa/import-pester-stub branch June 28, 2021 19:02
dependabot bot added a commit to rjmholt/PowerShellEditorServices that referenced this pull request Oct 11, 2021
…ster-stub

Import `InvokePesterStub.ps1` from `vscode-powershell` (with history)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants