-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Add support for test script parameters in win_pester #58790
Conversation
@jborean93 Can you check the test failure once, I think it's not related to the change |
Seems the changes caused a stack overflow with a recursion in the return object I'm not sure what but the changes you added seem to return an object that the serialization of which are recursive or that calls to itself |
Just a thought, I can't test it right now but maybe your if is always true since the test that fails only has the path parameter |
I stand corrected, Path is an alias to script, and it seems the issue comes from the test script you added. |
Fixed the overflow issue and the test file name was wrong, fixed that as well, CI should pass now. |
@ShachafGoldstein @jborean93 Are we good here ? |
Nothing more from me |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple little things
@nitzmahone changes made as per comments. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Few minor nits in the code and some doc updates needed but looks great.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
SUMMARY
This PR is to add support for using parameters in test script.
ISSUE TYPE
COMPONENT NAME
win_pester
ADDITIONAL INFORMATION
This allows user to pass parameters like below