Redundant SG rule create calls in unit tests

Bug #1302091 reported by Sridhar Gaddam
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Sridhar Gaddam

Bug Description

The following test cases in TestSecurityGroups of test_extension_security_group.py file
are making multiple calls to create a SG rule.
test_create_security_group_rule_min_port_greater_max()
test_create_security_group_rule_ports_but_no_protocol()
test_create_security_group_rule_port_range_min_only()
test_create_security_group_rule_port_range_max_only()
test_create_security_group_rule_icmp_type_too_big()
test_create_security_group_rule_icmp_code_too_big()

The redundant calls can be removed.

Tags: unittest
Changed in neutron:
assignee: nobody → Sridhar Gaddam (sridhargaddam)
status: New → In Progress
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/85127

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

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

commit c49ec8b3ba10bb414a0f135d29ff3685e26d58b3
Author: sridhargaddam <email address hidden>
Date: Thu Apr 3 23:29:37 2014 +0530

    Redundant SG rule create calls in unit tests

    This patch removes the redundant calls in the test cases.

    Closes-Bug: #1302091
    Change-Id: I509843b427dc454c0e6b2fd2cbff1bbbb284337f

Changed in neutron:
status: In Progress → Fix Committed
Changed in neutron:
importance: Undecided → Low
milestone: none → icehouse-rc2
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (milestone-proposed)

Fix proposed to branch: milestone-proposed
Review: https://review.openstack.org/85507

tags: added: unittest
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (milestone-proposed)

Reviewed: https://review.openstack.org/85507
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=354bc1ab4b1490b72abf80cfd2860337f1ccb369
Submitter: Jenkins
Branch: milestone-proposed

commit 354bc1ab4b1490b72abf80cfd2860337f1ccb369
Author: sridhargaddam <email address hidden>
Date: Thu Apr 3 23:29:37 2014 +0530

    Redundant SG rule create calls in unit tests

    This patch removes the redundant calls in the test cases.

    Closes-Bug: #1302091
    Change-Id: I509843b427dc454c0e6b2fd2cbff1bbbb284337f
    (cherry picked from commit c49ec8b3ba10bb414a0f135d29ff3685e26d58b3)

Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-rc2 → 2014.1
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.