diff mbox

[i-g-t,2/2] tests/kms_force_connect: skip if a VGA connector is not available

Message ID 1412869853-7689-2-git-send-email-thomas.wood@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Wood Oct. 9, 2014, 3:50 p.m. UTC 
Signed-off-by: Thomas Wood <thomas.wood@intel.com>
---
 tests/kms_force_connector.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Chris Wilson Oct. 9, 2014, 3:53 p.m. UTC | #1 
On Thu, Oct 09, 2014 at 04:50:53PM +0100, Thomas Wood wrote:
> Signed-off-by: Thomas Wood <thomas.wood@intel.com>
> ---
>  tests/kms_force_connector.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/kms_force_connector.c b/tests/kms_force_connector.c
> index 96881c7..361bf84 100644
> --- a/tests/kms_force_connector.c
> +++ b/tests/kms_force_connector.c
> @@ -54,7 +54,7 @@ main (int argc, char **argv)
>  		connector = NULL;
>  	}
>  
> -	igt_assert(connector);
> +	igt_require(connector);

If you rename this vga_connector, the error message will be immediately
understandable.
-Chris
diff mbox

Patch

diff --git a/tests/kms_force_connector.c b/tests/kms_force_connector.c
index 96881c7..361bf84 100644
--- a/tests/kms_force_connector.c
+++ b/tests/kms_force_connector.c
@@ -54,7 +54,7 @@  main (int argc, char **argv)
 		connector = NULL;
 	}
 
-	igt_assert(connector);
+	igt_require(connector);
 
 	/* force the connector on and check the reported values */
 	kmstest_force_connector(drm_fd, connector, FORCE_CONNECTOR_ON);