Skip to content

Conversation

@jliusan
Copy link
Member

@jliusan jliusan commented Nov 28, 2025

Copilot AI review requested due to automatic review settings November 28, 2025 03:40
Copilot finished reviewing on behalf of jliusan November 28, 2025 03:42
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds comprehensive management plane testing documentation to the testing.md file. The changes include examples for both data plane and management plane testing workflows, specifically using the armchaos service as a reference implementation.

  • Adds structured documentation for management plane test setup including asset.json configuration and test file structure
  • Reorganizes the table of contents with corrected indentation
  • Provides step-by-step examples for writing and running management plane tests

#### example about `management plane`

A simple test for `armchaos` is shown below:
##### The first step is to download prepared scripts to genrated asset.json in the path and create file utils_test.go
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The heading structure is inconsistent. A level 5 heading (##### ) follows a level 4 heading (####) without an intermediate level. Consider using #### or restructuring the headings to maintain a proper hierarchy.

Suggested change
##### The first step is to download prepared scripts to genrated asset.json in the path and create file utils_test.go
#### The first step is to download prepared scripts to genrated asset.json in the path and create file utils_test.go

Copilot uses AI. Check for mistakes.

```

##### Then you can add the test file
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The heading structure is inconsistent. A level 5 heading (##### ) follows a level 4 heading (####) without an intermediate level. Consider using #### or restructuring the headings to maintain a proper hierarchy.

Suggested change
##### Then you can add the test file
### Then you can add the test file

Copilot uses AI. Check for mistakes.

```

##### At last, you can run test via command `go test -run TestOperationsTestSuite`
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The heading structure is inconsistent. A level 5 heading (##### ) follows a level 4 heading (####) without an intermediate level. Consider using #### or restructuring the headings to maintain a proper hierarchy.

Suggested change
##### At last, you can run test via command `go test -run TestOperationsTestSuite`
#### At last, you can run test via command `go test -run TestOperationsTestSuite`

Copilot uses AI. Check for mistakes.

1. Run the following PowerShell commands to download necessary scripts:

```ps
Copy link

Copilot AI Nov 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code fence should use 'powershell' as the language identifier instead of 'ps' for better syntax highlighting and clarity.

Suggested change
```ps
```powershell

Copilot uses AI. Check for mistakes.
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.

2 participants