From c3eccf9ef4c2ba311e37f077fb56a6258b3d3b86 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Maciej=20=C5=BBenczykowski?= Date: Thu, 26 Sep 2024 18:11:41 +0000 Subject: [PATCH] Revert "Use unicast instead of broadcast as ethernet destination in `test_apf_drop_ethercat`" This reverts commit 46102348e0c7a92bf2f232b59b633df5e9eb8a32. Reason for revert: mistakenly submitted. Change-Id: I6072a9828a8cc0633ca35ea2ecf7707f2b3573a1 --- staticlibs/testutils/host/python/apf_test_base.py | 3 --- tests/cts/multidevices/apfv4_test.py | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/staticlibs/testutils/host/python/apf_test_base.py b/staticlibs/testutils/host/python/apf_test_base.py index 891876b7a9..9a30978420 100644 --- a/staticlibs/testutils/host/python/apf_test_base.py +++ b/staticlibs/testutils/host/python/apf_test_base.py @@ -50,9 +50,6 @@ class ApfTestBase(multi_devices_test_base.MultiDevicesTestBase): self.server_mac_address = apf_utils.get_hardware_address( self.serverDevice, self.server_iface_name ) - self.client_mac_address = apf_utils.get_hardware_address( - self.clientDevice, self.client_iface_name - ) # Enable doze mode to activate APF. adb_utils.set_doze_mode(self.clientDevice, True) diff --git a/tests/cts/multidevices/apfv4_test.py b/tests/cts/multidevices/apfv4_test.py index aa535fd3fc..7795be523a 100644 --- a/tests/cts/multidevices/apfv4_test.py +++ b/tests/cts/multidevices/apfv4_test.py @@ -53,7 +53,7 @@ class ApfV4Test(apf_test_base.ApfTestBase, parameterized.TestCase): ) # Declare inputs for state_str and expected_result. def test_apf_drop_ethertype_not_allowed(self, blocked_ether_type): # Ethernet header (14 bytes). - packet = self.client_mac_address.replace(":", "") # Destination MAC + packet = ETHER_BROADCAST_ADDR # Destination MAC (broadcast) packet += self.server_mac_address.replace(":", "") # Source MAC packet += blocked_ether_type