apps: lc-compliance: Reduce test repetition
diff mbox series

Message ID 20260525091131.22793-1-kieran.bingham@ideasonboard.com
State New
Headers show
Series
  • apps: lc-compliance: Reduce test repetition
Related show

Commit Message

Kieran Bingham May 25, 2026, 9:11 a.m. UTC
The lc-compliance test suite runs many combinations of tests with a
variable number of requetss.

This can validate that a pipeline handler succeeds with early startup
and shutdown but having many different runs of the same test can be very
slow.

Reduce the set of NUMREQUESTS to continue validating the early (low
count) tests and reduce the higher quantities which provide less value,
but keep the highest as a more complete test.

Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/apps/lc-compliance/tests/capture_test.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart June 11, 2026, 2:50 p.m. UTC | #1
Hi Kieran,

Thank you for the patch.

On Mon, May 25, 2026 at 11:11:31AM +0200, Kieran Bingham wrote:
> The lc-compliance test suite runs many combinations of tests with a
> variable number of requetss.
> 
> This can validate that a pipeline handler succeeds with early startup
> and shutdown but having many different runs of the same test can be very
> slow.
> 
> Reduce the set of NUMREQUESTS to continue validating the early (low
> count) tests and reduce the higher quantities which provide less value,
> but keep the highest as a more complete test.
> 
> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
> ---
>  src/apps/lc-compliance/tests/capture_test.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/apps/lc-compliance/tests/capture_test.cpp b/src/apps/lc-compliance/tests/capture_test.cpp
> index 29d8b7f841da..d7d6f0626e38 100644
> --- a/src/apps/lc-compliance/tests/capture_test.cpp
> +++ b/src/apps/lc-compliance/tests/capture_test.cpp
> @@ -114,7 +114,7 @@ TEST_P(SimpleCapture, UnbalancedStop)
>  	capture.run(numRequests);
>  }
>  
> -const int NUMREQUESTS[] = { 1, 2, 3, 5, 8, 13, 21, 34, 55, 89 };
> +const int NUMREQUESTS[] = { 1, 2, 3, 5, 8, 89 };

Goodbye Fibonacci.

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

>  
>  const std::vector<StreamRole> SINGLEROLES[] = {
>  	{ StreamRole::Raw, },

Patch
diff mbox series

diff --git a/src/apps/lc-compliance/tests/capture_test.cpp b/src/apps/lc-compliance/tests/capture_test.cpp
index 29d8b7f841da..d7d6f0626e38 100644
--- a/src/apps/lc-compliance/tests/capture_test.cpp
+++ b/src/apps/lc-compliance/tests/capture_test.cpp
@@ -114,7 +114,7 @@  TEST_P(SimpleCapture, UnbalancedStop)
 	capture.run(numRequests);
 }
 
-const int NUMREQUESTS[] = { 1, 2, 3, 5, 8, 13, 21, 34, 55, 89 };
+const int NUMREQUESTS[] = { 1, 2, 3, 5, 8, 89 };
 
 const std::vector<StreamRole> SINGLEROLES[] = {
 	{ StreamRole::Raw, },