Skip to content

Add a unit test in sonic-cfggen to test argument "--var-json"#2906

Merged
lguohan merged 1 commit intosonic-net:masterfrom
jihaix:jihaix/test_cfggen
May 19, 2019
Merged

Add a unit test in sonic-cfggen to test argument "--var-json"#2906
lguohan merged 1 commit intosonic-net:masterfrom
jihaix:jihaix/test_cfggen

Conversation

@jihaix
Copy link
Copy Markdown
Contributor

@jihaix jihaix commented May 16, 2019

- What I did
Adding a unit test in sonic-cfggen to test argument --var-json

- How I did it
Modifying file test_cfggen.py in sonic-cfggen's unit test to test argument --var-json of sonic-cfggen

- How to verify it
Executing following commands in sequence and there is no error occurs:

1. make target/python-wheels/sonic_config_engine-1.0-py2-none-any.whl-clean
2. make target/python-wheels/sonic_config_engine-1.0-py2-none-any.whl

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented May 16, 2019

can you resolve the build issue

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented May 18, 2019

retest this please

@lguohan lguohan merged commit 6aad2da into sonic-net:master May 19, 2019
MichelMoriniaux pushed a commit to criteo-forks/sonic-buildimage that referenced this pull request May 28, 2019
mssonicbld added a commit that referenced this pull request Sep 26, 2023
…lly (#16694)

#### Why I did it
src/sonic-swss
```
* 8934b62b - (HEAD -> 202305, origin/202305) [202305][CodeQL]: Use dependencies with relevant versions in azp template. (#2906) (3 hours ago) [Nazarii Hnydyn]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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