Skip to content

[request parser] Add unit tests for request parser to parse multiple values#1766

Merged
shi-su merged 1 commit intosonic-net:masterfrom
shi-su:parser_test
Jun 14, 2021
Merged

[request parser] Add unit tests for request parser to parse multiple values#1766
shi-su merged 1 commit intosonic-net:masterfrom
shi-su:parser_test

Conversation

@shi-su
Copy link
Contributor

@shi-su shi-su commented Jun 3, 2021

What I did
Add unit tests for request parser to parse multiple values.

Why I did it
Add unit tests for the feature added in PR #1758.

How I verified it
Run unit tests locally and confirm the tests passes with the given conditions and constraints.

Details if related

@shi-su shi-su requested a review from prsunny June 3, 2021 18:48
@shi-su
Copy link
Contributor Author

shi-su commented Jun 7, 2021

/Azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@shi-su
Copy link
Contributor Author

shi-su commented Jun 14, 2021

/Azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@shi-su shi-su merged commit ec96602 into sonic-net:master Jun 14, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
sonic-net#1766)

What I did
Add unit tests for request parser to parse multiple values.

Why I did it
Add unit tests for the feature added in PR sonic-net#1758.

How I verified it
Run unit tests locally and confirm the tests pass with the given conditions and constraints.
qiluo-msft pushed a commit that referenced this pull request Dec 20, 2021
#1766)

What I did
Add unit tests for request parser to parse multiple values.

Why I did it
Add unit tests for the feature added in PR #1758.

How I verified it
Run unit tests locally and confirm the tests pass with the given conditions and constraints.
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
…sonic-net#1766)

When systemd services finished, the systemd status could running
or degraded. Therefore, compare system status with starting directly.

Here is what is happening on kvm switch. Some services cannot be started.
Then, the system state is in degraded state.

```
admin@ARISTA01T1:~$ systemctl is-system-running
starting
admin@ARISTA01T1:~$ systemctl is-system-running
degraded
```

Signed-off-by: Guohan Lu <[email protected]>
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
sonic-net#1766)

What I did
Add unit tests for request parser to parse multiple values.

Why I did it
Add unit tests for the feature added in PR sonic-net#1758.

How I verified it
Run unit tests locally and confirm the tests pass with the given conditions and constraints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants