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.

Sign up for GitHub

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

Jump to bottom

[RouteUpdater]: Fix multi_asic mock function implementation and multi_asic variable name #186

Merged
merged 2 commits into from
Dec 23, 2020
Merged

[RouteUpdater]: Fix multi_asic mock function implementation and multi_asic variable name #186

merged 2 commits into from
Dec 23, 2020

Commits on Dec 22, 2020

  1. [RouteUpdater]: Fix multi_asic mock function implementation. 

    multi_asic mock function to return port_table from config_db
was returning empty dictionary due to which RouteUpdater class
was not completely unit-tested as one of the condition check was
never set to true.

Signed-off-by: Suvarna Meenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Dec 22, 2020
    Configuration menu
    Copy the full SHA
    a7b7b3d View commit details
    Browse the repository at this point in the history

Commits on Dec 23, 2020

  1. Minor fix as per review comments. 

    Signed-off-by: Suvarna Meenakshi <sumeenak@microsoft.com>
    @SuvarnaMeenakshi
    SuvarnaMeenakshi committed Dec 23, 2020
    Configuration menu
    Copy the full SHA
    ec0f140 View commit details
    Browse the repository at this point in the history