From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0C6B845C0A; Tue, 29 Oct 2024 13:52:29 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F12D542E52; Tue, 29 Oct 2024 13:52:28 +0100 (CET) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by mails.dpdk.org (Postfix) with ESMTP id BBA0140144 for ; Tue, 29 Oct 2024 13:52:27 +0100 (CET) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CB930113E; Tue, 29 Oct 2024 05:52:56 -0700 (PDT) Received: from [10.57.58.134] (unknown [10.57.58.134]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 655113F528; Tue, 29 Oct 2024 05:52:26 -0700 (PDT) Message-ID: <95f1df57-7653-4f82-b8aa-ed3b70080c77@arm.com> Date: Tue, 29 Oct 2024 12:52:24 +0000 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 4/5] dts: use TestSuiteSpec class imports Content-Language: en-GB To: =?UTF-8?Q?Juraj_Linke=C5=A1?= , dev@dpdk.org Cc: Honnappa Nagarahalli , Paul Szczepanek References: <20240822163941.1390326-1-luca.vizzarro@arm.com> <20240822163941.1390326-5-luca.vizzarro@arm.com> <85034ee9-09c0-4166-a546-e65ffdd0b56b@pantheon.tech> From: Luca Vizzarro In-Reply-To: <85034ee9-09c0-4166-a546-e65ffdd0b56b@pantheon.tech> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org On 17/09/2024 12:39, Juraj Linkeš wrote: >> @@ -229,139 +221,34 @@ def _get_test_suites_with_cases( > >> +            filtered_test_cases: list[TestCase] = [ >> +                test_case >> +                for test_case in test_suite_spec.test_cases >> +                if not test_suite_config.test_cases_names >> +                or test_case.name in test_suite_config.test_cases_names >> +            ] > > Ah, looks like TestSuiteSpec doesn't contain the subset we want to test. > Could we adapt it this way? I think we don't really care about test > cases we don't want to test. This is a bit out of scope, as the TestSuiteSpec only defines the specification of the test suites as they are defined.