Skip to content

Commit

Permalink
Removed a useless test
Browse files Browse the repository at this point in the history 
The remove bond is not different in the base vs. the qfx copper, no need for an extra test
  • Loading branch information
Martin Roy committed Sep 23, 2015
1 parent a62736b commit 779f6ef
Showing 1 changed file with 0 additions and 48 deletions.
48 changes: 0 additions & 48 deletions tests/adapters/switches/juniper_qfx_copper_test.py
View file
Original file line number Diff line number Diff line change
Expand Up @@ -211,54 +211,6 @@ def test_port_mode_access_with_no_port_mode_or_vlan_set_just_sets_the_port_mode(

self.switch.set_access_mode("ge-0/0/6")

def test_remove_bond_delete_slaves_and_interface_at_same_time(self):
with self.expecting_successful_transaction():

self.netconf_mock.should_receive("get_config").with_args(source="candidate", filter=is_xml("""
<filter>
<configuration>
<interfaces />
</configuration>
</filter>
""")).and_return(a_configuration("""
<interfaces>
<interface>
<name>ae10</name>
</interface>
<interface>
<name>ge-0/0/1</name>
<ether-options>
<auto-negotiation/>
<ieee-802.3ad>
<bundle>ae10</bundle>
</ieee-802.3ad>
</ether-options>
</interface>
<interface>
<name>ge-0/0/2</name>
</interface>
</interfaces>
"""))

self.netconf_mock.should_receive("edit_config").once().with_args(target="candidate", config=is_xml("""
<config>
<configuration>
<interfaces>
<interface operation="delete">
<name>ae10</name>
</interface>
<interface>
<name>ge-0/0/1</name>
<ether-options>
<ieee-802.3ad operation="delete" />
</ether-options>
</interface>
</interfaces>
</configuration>
</config>""")).and_return(an_ok_response())

self.switch.remove_bond(10)

def test_add_interface_to_bond(self):
with self.expecting_successful_transaction():

Expand Down

0 comments on commit 779f6ef

Please sign in to comment.