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.

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

fix: Rename test file #2784

Merged
merged 4 commits into from
Jun 13, 2024
Merged

fix: Rename test file #2784

merged 4 commits into from
Jun 13, 2024

Conversation

bohuini
Copy link
Contributor

@bohuini bohuini commented Jun 11, 2024

Reason for Change:
Fixed the naming of ut file and fixed bugs in UT.

Issue Fixed:

Requirements:

Notes:

@bohuini bohuini added bug do-not-merge cns Related to CNS. labels Jun 11, 2024
@bohuini bohuini requested a review from a team as a code owner June 11, 2024 23:12
@bohuini bohuini requested a review from tamilmani1989 June 11, 2024 23:12
@bohuini
Copy link
Contributor Author

bohuini commented Jun 11, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bohuini
Copy link
Contributor Author

bohuini commented Jun 12, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bohuini
Copy link
Contributor Author

bohuini commented Jun 12, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@shreyashastantram
Copy link
Contributor

Please ensure that these tests runs in pipeline

@bohuini bohuini enabled auto-merge June 13, 2024 18:14
@bohuini
Copy link
Contributor Author

bohuini commented Jun 13, 2024

@bohuini bohuini added this pull request to the merge queue Jun 13, 2024
Merged via the queue into Azure:master with commit 5526348 Jun 13, 2024
11 checks passed
@bohuini bohuini deleted the bohui_fixUnitTest branch June 13, 2024 22:27
paulyufan2 pushed a commit that referenced this pull request Jun 18, 2024
* Rename test file

* Modified UT

* Modified setRoutes name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug cns Related to CNS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants