search

Azure / azurestack-powershell

AzureStack Hub Operator Powershell Cmdlets
MIT License
16 stars 33 forks source link

Fixing\Updating AzsAdmin Compute powershell generation and tests. #122

Closed travist13 closed 3 years ago

travist13 commented 3 years ago

Open API spec PR is updated and merged. Dependent Open API Spec PR: https://github.com/Azure/azure-rest-api-specs/pull/15059

This fixes many of the issues with the module generation and the tests. I also added a test for the new computeScaleUnit API.

TheOnlyWei commented 3 years ago

There are tests that have not been implemented, will these be implemented in the future?

TheOnlyWei commented 3 years ago

/azp run

azure-pipelines[bot] commented 3 years ago
Azure Pipelines successfully started running 3 pipeline(s).
travist13 commented 3 years ago

There are tests that have not been implemented, will these be implemented in the future?

The "Feature" tests are being implemented by Miguel. For the other NotImplemented tests I will create work items to implement those in a later effort. However, these other APIs are not changed recently and are being covered in our nightly team CI runs.

TheOnlyWei commented 3 years ago

/azp run

azure-pipelines[bot] commented 3 years ago
Azure Pipelines successfully started running 3 pipeline(s).
travist13 commented 3 years ago

/azp run

azure-pipelines[bot] commented 3 years ago
Commenter does not have sufficient privileges for PR 122 in repo Azure/azurestack-powershell
bganapa commented 3 years ago

/azp run

azure-pipelines[bot] commented 3 years ago
Azure Pipelines successfully started running 3 pipeline(s).
TheOnlyWei commented 3 years ago

/azp run

azure-pipelines[bot] commented 3 years ago
Azure Pipelines successfully started running 3 pipeline(s).
bganapa commented 3 years ago

@travist13 closing this PR as your other PR is merged.