ML2 plugin needs additional unit tests

Bug #1236127 reported by Robert Kukura
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Chuck Carlino

Bug Description

The ML2 plugin needs unit tests that verify that the proper MechanismDriver calls are made for each API operation, including those involving auto-deleted resources.

Tags: unittest ml2
Changed in neutron:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Amir Sadoughi (amir-sadoughi) wrote :

From tox -e cover neutron.tests.unit.ml2; coverage report -m

neutron/plugins/ml2/plugin 411 45 85 12 89% 143-144, 182-185, 322-326, 430-435, 440-443, 448-451, 455-459, 471-475, 539-542, 546-550, 577-581, 599-603, 663-667, 676-678, 697-698

tags: added: ml2-unit-test-coverage
Kyle Mestery (mestery)
Changed in neutron:
assignee: nobody → Kyle Mestery (mestery)
Kyle Mestery (mestery)
Changed in neutron:
status: Triaged → In Progress
tags: added: ml2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/74079

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/74079
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=5fb347ca6fd70750861cf1a49808f68125288f38
Submitter: Jenkins
Branch: master

commit 5fb347ca6fd70750861cf1a49808f68125288f38
Author: Kyle Mestery <email address hidden>
Date: Tue Jan 28 20:22:29 2014 +0000

    Add additional unit tests for the ML2 plugin

    This commit adds extended unit test coverage for the ML2 plugin.py file.

    Partial-Bug: 1236127

    Change-Id: I2f4465c84810965679a585007cd9005ba8a7da71

Kyle Mestery (mestery)
Changed in neutron:
assignee: Kyle Mestery (mestery) → nobody
Chuck Carlino (ccarlino)
Changed in neutron:
assignee: nobody → Chuck Carlino (ccarlino)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/99129

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/99129
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=4cd29b7a9ebadd0eecf60aa10251ed4d8e51d4cc
Submitter: Jenkins
Branch: master

commit 4cd29b7a9ebadd0eecf60aa10251ed4d8e51d4cc
Author: Chuck Carlino <email address hidden>
Date: Mon Jun 9 23:18:11 2014 -0700

    Add test cases for plugins/ml2/plugin.py

    Test cases which generate MechanismDriverError failures and
    validate correct behavior.

    Partial-Bug: 1236127
    Change-Id: I922e842b54add786723ecb7763d6b4f330794794

Revision history for this message
Shiv Haris (shh) wrote :

chuckc: Can this bug be closed?

tags: added: unittest
removed: ml2-unit-test-coverage
Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
milestone: none → kilo-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: kilo-1 → 2015.1.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.