Repository navigation
Add autoload for test modules - #3342
Conversation
c59bba3 to
dc4a286
Compare
|
cc @PowerShell/area-test for review |
There was a problem hiding this comment.
Removing the module imports just doesn't seem right to me. If you invoke-pester directly, it doesn't even tell you what is wrong.
There was a problem hiding this comment.
Adding a new helper modules and using of existing helper modules need to be documented. I believe this is enough.
Maybe Jason Shirk (@lzybkr) has something to add.
There was a problem hiding this comment.
If we are going to require everyone to use Start-PSPester which btw someone misnamed this function, it should be Invoke-PSPester since it is synchronous, it should have the -testname parameter.
There was a problem hiding this comment.
I added import of helper modules in current session too so it is not required to use Start-PSPester - it is worked well .\Out-Host.Tests.ps1
|
@PowerShell/area-test, Does anyone have any comments here. I left one comment on what Ilya (@iSazonov) should change if we take this and one about what is wrong with making this change. |
343bd42 to
6afea3f
Compare
Related PowerShell#3238 1. Add autoload for test modules 2. Move TestHostCS.psm1 to 'test\tools\Modules\' folder 3. Remove explicit load TestHostCS.psm1 from test files
6afea3f to
939e396
Compare
|
Travis Plunk (@TravisEz13) Is The PR ready to merge? |
Related PowerShell#3238 1. Add autoload for test modules 2. Move TestHostCS.psm1 to 'test\tools\Modules\' folder 3. Remove explicit load TestHostCS.psm1 from test files
Related #3238