Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update the data in the test cases #18496

Merged
merged 2 commits into from
Jun 16, 2022
Merged

Update the data in the test cases #18496

merged 2 commits into from
Jun 16, 2022

Commits on Jun 13, 2022

  1. Fix the unit tests. 

    - Parsing the command parameter requires the module to be loaded. On the machine (e.g. build machine) where the Az modules are not installed, the
  parsing doesn't return the parameters names as expected. Some test cases thus fail. The fix is to remove Az cmdlets from the test cases. Instead, we
  use the cmdlet from the built-in modules in the test cases.
    @kceiw
    kceiw committed Jun 13, 2022
    Configuration menu
    Copy the full SHA
    d01b56e View commit details
    Browse the repository at this point in the history

  2. Replace the way of getting the data. 

    - We used to zip the whole model and read from it. Since we don't need to use Az cmdlet in the test cases, we don't really need to zip the model.
  So we use the json file with crafted built-in PowerShell cmdlet as the data used in the test cases. With that, it's also clear to show the changes
  to the test cases and the data.
    @kceiw
    kceiw committed Jun 13, 2022
    Configuration menu
    Copy the full SHA
    5e5b0b8 View commit details
    Browse the repository at this point in the history