[Az.Migrate] Stabilize command internals & update policy defaults
Description
Mandatory Checklist
-
Please choose the target release of Azure PowerShell. (⚠️Target release is a different concept from API readiness. Please click below links for details.)
- [x] General release
- [ ] Public preview
- [ ] Private preview
- [ ] Engineering build
- [ ] No need for a release
-
[x] Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:
- SHOULD update
ChangeLog.mdfile(s) appropriately- Update
src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.- A snippet outlining the change(s) made in the PR should be written under the
## Upcoming Releaseheader in the past tense.
- A snippet outlining the change(s) made in the PR should be written under the
- Should not change
ChangeLog.mdif no new release is required, such as fixing test case only.
- Update
- SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
- SHOULD have proper test coverage for changes in pull request.
- SHOULD NOT adjust version of module manually in pull request
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.
/azp run
Azure Pipelines successfully started running 3 pipeline(s).
Replication policy now aligns with Portal UX for default config.
Passing dev test with private module 3.0.17: https://dev.azure.com/msazure/One/_build/results?buildId=143763316&view=results
Hey @minhsuanlee, @mayada83,
- What are these changes related to?
- Why did you remove all the test cases?
@minhsuanlee, @mayada83, Is this PR still targeting 15.1?
Hey @minhsuanlee, @mayada83,
- What are these changes related to?
- Why did you remove all the test cases?
-
What are these changes related to? a. Putting back usage of $PSBoundParameters in AzLocal commands b. Update Policy default values to align with the product
-
Why did you remove all the test cases? Following the suggestion by Joyer Jin from PS team, I've created a more comprehensive unit test of Test-AzMigrateLocalEndToEnd.Recording.json to test the entire scenario of AzLocal migrate instead of testing individual commands separately