adds date_format to new to/from_api_repr tests

Author: chalmerlowe

google/cloud/bigquery/external_config.py

@@ -862,6 +862,7 @@ def date_format(self) -> Optional[str]:
     def date_format(self, value: Optional[str]):
         self._properties["dateFormat"] = value
 
+    @property
     def time_zone(self) -> Optional[str]:
         """Optional[str]: Time zone used when parsing timestamp values that do not
         have specific time zone information (e.g. 2024-04-20 12:34:56). The expected

google/cloud/bigquery/job/load.py

@@ -561,6 +561,7 @@ def date_format(self) -> Optional[str]:
     def date_format(self, value: Optional[str]):
         self._set_sub_prop("dateFormat", value)
 
+    @property
     def time_zone(self) -> Optional[str]:
         """Optional[str]: Default time zone that will apply when parsing timestamp
         values that have no specific time zone.
@@ -922,6 +923,7 @@ def date_format(self):
         """
         return self.configuration.date_format
 
+    @property
     def time_zone(self):
         """See
         :attr:`google.cloud.bigquery.job.LoadJobConfig.time_zone`.

tests/unit/job/test_load.py

@@ -147,7 +147,6 @@ def _verifyResourceProperties(self, job, resource):
             )
         else:
             self.assertIsNone(job.reference_file_schema_uri)
-
         if "destinationEncryptionConfiguration" in config:
             self.assertIsNotNone(job.destination_encryption_configuration)
             self.assertEqual(
@@ -156,15 +155,14 @@ def _verifyResourceProperties(self, job, resource):
             )
         else:
             self.assertIsNone(job.destination_encryption_configuration)
-        if "timeZone" in config:
-            self.assertEqual(job.time_zone, config["timeZone"])
-        else:
-            self.assertIsNone(job.time_zone)
-
         if "dateFormat" in config:
             self.assertEqual(job.date_format, config["dateFormat"])
         else:
             self.assertIsNone(job.date_format)
+        if "timeZone" in config:
+            self.assertEqual(job.time_zone, config["timeZone"])
+        else:
+            self.assertIsNone(job.time_zone)
 
     def test_ctor(self):
         client = _make_client(project=self.PROJECT)

tests/unit/job/test_load_config.py

@@ -958,6 +958,7 @@ def test_column_name_character_map_none(self):
             },
             "useAvroLogicalTypes": True,
             "writeDisposition": "WRITE_TRUNCATE",
+            "dateFormat": "%Y-%m-%d",
             "timeZone": "America/New_York",
             "parquetOptions": {"enableListInference": True},
             "columnNameCharacterMap": "V2",
@@ -999,6 +1000,7 @@ def test_from_api_repr(self):
         )
         self.assertTrue(config.use_avro_logical_types)
         self.assertEqual(config.write_disposition, WriteDisposition.WRITE_TRUNCATE)
+        self.assertEqual(config.date_format, "%Y-%m-%d")
         self.assertEqual(config.time_zone, "America/New_York")
         self.assertTrue(config.parquet_options.enable_list_inference)
         self.assertEqual(config.column_name_character_map, ColumnNameCharacterMap.V2)
@@ -1033,6 +1035,7 @@ def test_to_api_repr(self):
         )
         config.use_avro_logical_types = True
         config.write_disposition = WriteDisposition.WRITE_TRUNCATE
+        config.date_format = "%Y-%m-%d"
         config.time_zone = "America/New_York"
         parquet_options = ParquetOptions()
         parquet_options.enable_list_inference = True