vpp_papi: Do not raise before find_api_dir()

The change https://gerrit.fd.io/r/18352 has introduced a check
which raises when neither apifiles nor apidir are set.
This change removes that check, as it broke CSIT.

If such a check is intended, find_api_dir() should be removed.
Also, make sure you do not break CSIT next time.

Change-Id: I0faab7c41d199e3aa5d5bc6aab0defd1ab9e10da
Signed-off-by: Vratko Polak <vrpolak@cisco.com>

diff --git a/src/vpp-api/python/vpp_papi/vpp_papi.py b/src/vpp-api/python/vpp_papi/vpp_papi.py
index 8dcd63a..f29c250 100644 (file)
--- a/src/vpp-api/python/vpp_papi/vpp_papi.py
+++ b/src/vpp-api/python/vpp_papi/vpp_papi.py
@@ -212,9 +212,6 @@ class VPPApiClient(object):
         loglevel, if supplied, is the log level this logger is set
         to report at (from the loglevels in the logging module).
         """
-        if apifiles is None and self.__class__.apidir is None:
-            raise ValueError("Either apifiles or apidir must be specified.")
-
         if logger is None:
             logger = logging.getLogger(__name__)
             if loglevel is not None:
@@ -292,7 +289,7 @@ class VPPApiClient(object):
         :returns: A single directory name, or None if no such directory
             could be found.
         """
-        dirs = [cls.apidir]
+        dirs = [cls.apidir] if cls.apidir else []
 
         # perhaps we're in the 'src/scripts' or 'src/vpp-api/python' dir;
         # in which case, plot a course to likely places in the src tree