Adding a standard unit test for each function #32
Conversation
Linux Test Results1 488 tests +220 1 347 ✅ +412 37s ⏱️ ±0s For more details on these failures, see this check. Results for commit 937ec56. ± Comparison against base commit 089e00a. ♻️ This comment has been updated with latest results. |
WinPS51 Test Results1 488 tests +220 1 347 ✅ +412 45s ⏱️ +4s For more details on these failures, see this check. Results for commit 937ec56. ± Comparison against base commit 089e00a. ♻️ This comment has been updated with latest results. |
WinPS71 Test Results1 488 tests +220 1 347 ✅ +412 45s ⏱️ +2s For more details on these failures, see this check. Results for commit 937ec56. ± Comparison against base commit 089e00a. ♻️ This comment has been updated with latest results. |
This update improves the structure of the parameter validation in the Add-FabricWorkspaceRoleAssignment tests. By moving the expected parameters setup into the BeforeDiscovery and BeforeAll blocks, we enhance readability and maintainability. Thank you!
…for Pester Help Tests This change simplifies the parameter validation by removing the default parameters that are not required for the Add-FabricWorkspaceRoleAssignment function. This helps to ensure that the tests focus on the essential parameters, improving clarity and maintainability. Thank you!
Consolidate expected parameters into a single array for better readability and maintainability. This change simplifies the parameter validation logic in the tests for the Add-FabricWorkspaceRoleAssignment function. Thank you!
This change adds "Jess'sMissingParam" to the expected parameters for the Add-FabricWorkspaceRoleAssignment tests. This ensures that the tests accurately reflect the parameters used in the command, improving the validation process. Thank you!
still need to
check output - why does it say 0 expected params for each
check functions that already had a test file
do something for functions that have no params