commit a236e34e64c9919400d8d03bd2d234785dfd50ad
author U. Artie Eoff <ullysses.a.eoff@intel.com> Wed May 01 09:48:50 2019 -0700
committer XinfengZhang <carl.zhang@intel.com> Wed May 08 20:47:02 2019 +0800
tree e5abff348e2566d79631009f5fa9913f23bfc0b5
parent d71e2604972d35e3803d9b66c7d2ed0a80c145dc

test: update vaInitialize_vaTerminate_Bad_vaSetDriverName

Since intel/libva#250, driver names are no longer restricted
to a whitelist.  Thus, any driver name is legal except for
a few corner cases (i.e. empty name and name > 255 chars).

Signed-off-by: U. Artie Eoff <ullysses.a.eoff@intel.com>

test/test_va_api_init_terminate.cpp

diff --git a/test/test_va_api_init_terminate.cpp b/test/test_va_api_init_terminate.cpp
index a4d589c..ab80a2e 100644
--- a/test/test_va_api_init_terminate.cpp
+++ b/test/test_va_api_init_terminate.cpp
@@ -74,12 +74,32 @@
 
 TEST_F(VAAPIInitTerminate, vaInitialize_vaTerminate_Bad_vaSetDriverName)
 {
-    char driver[4] = "bad";
+    int major, minor;
+
     VADisplay display = getDisplay();
     ASSERT_TRUE(display);
 
-    EXPECT_STATUS_EQ(
-        VA_STATUS_ERROR_INVALID_PARAMETER, vaSetDriverName(display, driver));
+    { // driver name length == 0 invalid
+      char driver[1] = "";
+      EXPECT_STATUS_EQ(
+          VA_STATUS_ERROR_INVALID_PARAMETER, vaSetDriverName(display, driver));
+    }
+
+    { // driver name length >= 256 invalid
+      char driver[257];
+      driver[256] = '\0';
+      std::string(256, 'x').copy(driver, 256);
+      EXPECT_STATUS_EQ(
+          VA_STATUS_ERROR_INVALID_PARAMETER, vaSetDriverName(display, driver));
+    }
+
+    { // acceptable driver name, but does not exist
+      char driver[4] = "bad";
+      EXPECT_STATUS(vaSetDriverName(display, driver));
+
+      EXPECT_STATUS_EQ(
+          VA_STATUS_ERROR_UNKNOWN, vaInitialize(display, &major, &minor));
+    }
 
     EXPECT_STATUS(vaTerminate(display));
 }