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

Run yang validation in unit test #3087

Merged
merged 1 commit into from
Dec 18, 2023

Commits on Dec 18, 2023

  1. Run yang validation in unit test

    <!--
        Please make sure you've read and understood our contributing guidelines:
        https://github.com/Azure/SONiC/blob/gh-pages/CONTRIBUTING.md
    
        ** Make sure all your commits include a signature generated with `git commit -s` **
    
        If this is a bug fix, make sure your description includes "closes #xxxx",
        "fixes #xxxx" or "resolves #xxxx" so that GitHub automatically closes the related
        issue when the PR is merged.
    
        If you are adding/modifying/removing any command or utility script, please also
        make sure to add/modify/remove any unit tests from the tests
        directory as appropriate.
    
        If you are modifying or removing an existing 'show', 'config' or 'sonic-clear'
        subcommand, or you are adding a new subcommand, please make sure you also
        update the Command Line Reference Guide (doc/Command-Reference.md) to reflect
        your changes.
    
        Please provide the following information:
    -->
    
    #### What I did
    Add unit test for db_migrator.py: ConfigDB passing yang validation.
    Microsoft ADO: 24657445
    Pending on sonic-net/sonic-buildimage#16974
    
    #### How I did it
    Add yang validation for unit test, and fix test data to pass yang validation.
    
    #### How to verify it
    Run sonic-utilities end to end test.
    
    #### Previous command output (if the output of a command-line utility has changed)
    
    #### New command output (if the output of a command-line utility has changed)
    ganglyu authored Dec 18, 2023
    Configuration menu
    Copy the full SHA
    a2cb58b View commit details
    Browse the repository at this point in the history