Skip to content

Add tests for acipolicygen#3

Closed
DomAyre wants to merge 19 commits intofix-scenario-layersfrom
test-arm-files
Closed

Add tests for acipolicygen#3
DomAyre wants to merge 19 commits intofix-scenario-layersfrom
test-arm-files

Conversation

@DomAyre
Copy link
Owner

@DomAyre DomAyre commented Sep 19, 2025

NOTE: Must merge after Azure#9174

Why

Most existing tests call internal functions, not the end to end functions that are exposed to the user. Also the arm templates used by existing tests are embedded in the test code, making it hard to just try the tool with that use case, we want to fix both of these issues.

These new tests caught an issue with --exclude-default-fragments being broken

How

  • Define a set of arm templates which exercise specific features
  • Add a set of policies which are the result of calling acipolicygen with various args
  • Add a set of tests which run acipolicygen and confirm the output policies match the checked in ones

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

Note, since this change is only in tests, no version bump is needed

nilo19 and others added 7 commits September 18, 2025 14:35
* feat: add acns perf options

* feat: fix issues, add tests

* chore: update history

* Update src/aks-preview/azext_aks_preview/managed_cluster_decorator.py

Co-authored-by: Copilot <[email protected]>

* Update src/aks-preview/azext_aks_preview/tests/latest/test_managed_cluster_decorator.py

Co-authored-by: Copilot <[email protected]>

* Update src/aks-preview/linter_exclusions.yml

Co-authored-by: Copilot <[email protected]>

* Update src/aks-preview/linter_exclusions.yml

Co-authored-by: Copilot <[email protected]>

* fix: address comments

* fix: address comments

* fix: switch to westcentralus

* chore: update version

* fix: undo accidental delete

* chore: update acns perf test, add recording

* fix bad merge

---------

Co-authored-by: Copilot <[email protected]>
* add pester tests for connectedk8s cli extension

* Pass the force delete param to the API call (#4)

* forcedelete

* format

* add code owner

* mypy

* fix CI testcases for nodepool image issues (#8)

* update errors for the config and connectivity issues (#11)

* update errors

* format

* style

* update python version to 3.13 (#12)

* changes to support gateway association/disassociation for api version '2025-08-01-preview' (Azure#17)

* update release notes and version for connectedk8s cli extension (Azure#18)

* remove redundant test files specific to forked repo CI checks

---------

Co-authored-by: Bavneet Singh <[email protected]>
Co-authored-by: Atchut Kumar Barli <[email protected]>
phermanov-msft and others added 12 commits September 22, 2025 11:00
…mand (Azure#9180)

* Add deliveryReport and Tag parameters to sms send command

* correct version

* update version

* Fix tests

* Fix test and recordings

* Fix recordings
* [amlfs] Add az amlfs auto-import commands

* Updating History and setup files

* Update src/amlfs/azext_amlfs/aaz/latest/amlfs/auto_import/_update.py

Co-authored-by: Copilot <[email protected]>
(cherry picked from commit 3d1f1fc)

* Updating create option conflict resolution for shorter length

* Adding re-recorded Import Job test

* Adding re-recorded AutoExportJob tests

* Adding re-recorded test results

---------

Co-authored-by: Aman Jain <[email protected]>
Co-authored-by: Copilot <[email protected]>
…zure#9153)

* update error message

* grant vnet perm

* address comments

* setup.py

* update

* update style

* release version block

* update
…inistrative-state` command as it is not supported by the API. (Azure#9182)

* updating nexusidentity - Resolves installation issue caused by Graphs Python SDK package where a long path error occured. To fix this - SDK support was removed and replaced with httpclient

* fixing a small security concern with shell

* retrigger checks

* retrigger checks

* removing unsupported operation

* removing unsupported operation

* Retrigger GitHub Actions
@DomAyre
Copy link
Owner Author

DomAyre commented Sep 22, 2025

Moved to Azure#9199

@DomAyre DomAyre closed this Sep 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants