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

Expose issue66 via unit test #84

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Expose issue66 via unit test #84

wants to merge 2 commits into from

Commits on Nov 17, 2022

  1. controller, tests: assert updating an attachment MAC is not handled 

    Signed-off-by: Miguel Duarte Barroso <[email protected]>
    @maiqueb
    maiqueb committed Nov 17, 2022
    Configuration menu
    Copy the full SHA
    89a958e View commit details
    Browse the repository at this point in the history

  2. controller, tests: expose bug in issue#66 

    This commit refactors the previously added test into a table that
clearly proves that updating the interface name of an existing
attachment actually causes the original attachment to be removed, since
this "update" is recognized by the controller as a sequence of:
- attach new interface (with the interface name specified)
- remove old interface

The first action fails, since that interface name is already present
within the system, while the second succeeds.

Signed-off-by: Miguel Duarte Barroso <[email protected]>
    @maiqueb
    maiqueb committed Nov 17, 2022
    Configuration menu
    Copy the full SHA
    e7e550e View commit details
    Browse the repository at this point in the history