Revert^2 "Use unicast instead of broadcast as ethernet destination in `test_apf_drop_ethercat`"
This reverts commit c3eccf9ef4c2ba311e37f077fb56a6258b3d3b86.
Reason for revert: As we discussed, it might be valid for the chipset to drop BC (non-ARP, non-IPv4) packets in the hardware instead of the firmware. To ensure this test doesn't fail on those chipsets, using unicast packets for testing seems more suitable.
Change-Id: I89debf79965cf3ea62484c443e952bd137818efc
diff --git a/tests/cts/multidevices/apfv4_test.py b/tests/cts/multidevices/apfv4_test.py
index 7795be5..aa535fd 100644
--- a/tests/cts/multidevices/apfv4_test.py
+++ b/tests/cts/multidevices/apfv4_test.py
@@ -53,7 +53,7 @@
) # Declare inputs for state_str and expected_result.
def test_apf_drop_ethertype_not_allowed(self, blocked_ether_type):
# Ethernet header (14 bytes).
- packet = ETHER_BROADCAST_ADDR # Destination MAC (broadcast)
+ packet = self.client_mac_address.replace(":", "") # Destination MAC
packet += self.server_mac_address.replace(":", "") # Source MAC
packet += blocked_ether_type