Fix _validate_runner error msg for DataFlowRunner, no project/region case.

Change-Id: I85429442e715a7d9e68573eee9e181f16dbebc04

Author: cfezequiel

tfrutil/client.py

@@ -66,9 +66,9 @@ def _validate_runner(
 
   if (runner == 'DataFlowRunner') & (
       any(not v for v in [project, region])):
-    raise AttributeError('DataFlowRunner requires project region and region '
-                         'project is {} and region is {}'.format(
-                             project, region))
+    raise AttributeError(
+        'DataFlowRunner requires valid `project` and `region` to be specified.'
+        'The `project` is {} and `region` is {}'.format(project, region))
 
 # def read_image_directory(dirpath) -> pd.DataFrame:
 #   """Reads image data from a directory into a Pandas DataFrame."""

tfrutil/client_test.py

@@ -160,12 +160,16 @@ def test_gcs_path_with_dataflow_runner_missing_param(self):
     """Tests DataFlowRunner with missing required parameter."""
     df2 = self.test_df.copy()
     df2[constants.IMAGE_URI_KEY] = 'gs://' + df2[constants.IMAGE_URI_KEY]
-    with self.assertRaises(AttributeError):
-      client._validate_runner(
-          df2,
-          runner='DataFlowRunner',
-          project=None,
-          region=self.test_region)
+    for p, r in [
+        (None, self.test_region), (self.test_project, None), (None, None)]:
+      with self.assertRaises(AttributeError) as context:
+        client._validate_runner(
+            df2,
+            runner='DataFlowRunner',
+            project=p,
+            region=r)
+      self.assertTrue('DataFlowRunner requires valid `project` and `region`'
+                      in repr(context.exception))
 
 
 def _make_csv_tempfile(data: List[List[str]]) -> tempfile.NamedTemporaryFile: