diff mbox series

[15/41] oeqa/selftest/manifest: Remove broken/unused code

Message ID	20261115084845.701136-15-richard.purdie@linuxfoundation.org
State	New
Headers	show Received: from mail-wr1-f46.google.com (mail-wr1-f46.google.com [209.85.221.46]) by mx.groups.io with SMTP id smtpd.msgproc02-g2.4375.1794732554563772704 for <openembedded-core@lists.openembedded.org>; Sun, 15 Nov 2026 00:49:14 -0800 ip: 209.85.221.46, mailfrom: richard.purdie@linuxfoundation.org) From: Richard Purdie <richard.purdie@linuxfoundation.org> To: openembedded-core@lists.openembedded.org Subject: [PATCH 15/41] oeqa/selftest/manifest: Remove broken/unused code Date: Sun, 15 Nov 2026 08:48:18 +0000 Message-ID: <20261115084845.701136-15-richard.purdie@linuxfoundation.org> In-Reply-To: <20261115084845.701136-1-richard.purdie@linuxfoundation.org> References: <20261115084845.701136-1-richard.purdie@linuxfoundation.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset="utf-8"
Series	[01/41] oeqa/loader: Drop code for pre python 3.5 \| expand [01/41] oeqa/loader: Drop code for pre python 3.5 [02/41] lib: Simplify super() calls [03/41] oe-buildenv-internal: Correct minimum python version [04/41] oeqa/core/utils/concurrencytest: Fix losing results after class setup failures [05/41] oeqa/core/utils/concurrencytest: Only show each test's own output [06/41] oeqa/core/utils/concurrencytest: Don't buffer output in the forwarding results [07/41] oeqa/core/utils/concurrencytest: Fix the failed count in the test progress [08/41] oeqa: Drop explicit 'object' base classes [09/41] oeqa: Use OSError instead of IOError and socket.error [10/41] oeqa: Use 'is None' for None comparisons [11/41] oeqa: Drop redundant .keys() calls on dicts [12/41] oeqa/core/context: Use isinstance() for type check [13/41] oeqa: Use bare raise to re-raise exceptions [14/41] oeqa: Let subprocess decode output instead of calling .decode() [15/41] oeqa/selftest/manifest: Remove broken/unused code [16/41] oeqa/utils/testexport.py: Drop file [17/41] oeqa/runexported.py: Remove file [18/41] oeqa/oetest.py: Remove [19/41] oeqa/lib/utils/decorators.py: Remove [20/41] oeqa/targetcontrol: Drop SimpleRemoteTarget [21/41] oeqa/core/target/ssh: Fix TimeoutExpired reference [22/41] oeqa/runtime/ltp: Share a single base class and fix tearDownClass failures [23/41] oeqa/selftest/oelib/buildhistory: Fix variable removal in commit_vars() [24/41] oeqa: Remove unused code [25/41] oeqa/core/loader: Simplify the _make_failed_test override [26/41] oeqa: Remove unused imports [27/41] oeqa/core/decorator/depends: Check resolved dependency ids when running [28/41] oeqa/core/case: Tear down decorators if setUp() fails [29/41] oeqa/core/loader: Bind a copy of each decorator per test case [30/41] oeqa/core/runner: Report the status of tests with failing subtests [31/41] oeqa/buildperf: Drop sortTestMethodsUsing from the test runner [32/41] oeqa/core/tests: Add tests for the reported test result status [33/41] oeqa/core/runner: Track test results as they are added [34/41] oeqa/core/runner: Use getSuiteCases() to list tests [35/41] oeqa/core/decorator: Record decorators on the test function [36/41] oeqa/core/tests: Add tests for skipping tests and td_vars checks [37/41] oeqa/core: Use unittest's setup hook instead of patching test classes [38/41] oeqa/core/loader: Always return a flat test suite [39/41] oeqa: Drop the unused tc_kwargs list phase and dead selftest code [40/41] oeqa: Replace tc_kwargs with explicit attributes [41/41] oeqa/selftest: Use the core executor's run()

Commit Message

Richard Purdie Nov. 15, 2026, 8:48 a.m. UTC

The code was using a broken comparison operator. We always
set target anyway, so drop the broken code rather than fix it.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 meta/lib/oeqa/selftest/cases/manifest.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff mbox series

Patch

diff --git a/meta/lib/oeqa/selftest/cases/manifest.py b/meta/lib/oeqa/selftest/cases/manifest.py
index 73d262bb43b..ba9106d6cd4 100644
--- a/meta/lib/oeqa/selftest/cases/manifest.py
+++ b/meta/lib/oeqa/selftest/cases/manifest.py
@@ -40,8 +40,7 @@  class VerifyManifest(OESelftestTestCase):
 
     #this will possibly move from here
     @classmethod
-    def get_dir_from_bb_var(self, bb_var, target = None):
-        target == self.buildtarget if target == None else target
+    def get_dir_from_bb_var(self, bb_var, target):
         directory = get_bb_var(bb_var, target);
         if not directory or not os.path.isdir(directory):
             self.logger.debug("{}: {} points to {} when target = {}"\