feat: Adds support for google.protobuf.Value pipeline parameters in the parameter_values field (#807)

* feat: Adds support for `google.protobuf.Value` pipeline parameters in the `parameter_values` field PiperOrigin-RevId: 406492661 Source-Link: googleapis/googleapis@972abb6 Source-Link: googleapis/googleapis-gen@b6774aa Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYjY3NzRhYTNiNjFkMDViZjlmMTBhYTlhZjhkNjIzMTEyNzNkMzVhOSJ9 * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: sasha-gitg <44654632+sasha-gitg@users.noreply.github.com>

Author: gcf-owl-bot[bot]

google/cloud/aiplatform_v1/services/migration_service/client.py

@@ -197,32 +197,32 @@ def parse_dataset_path(path: str) -> Dict[str, str]:
  return m.groupdict() if m else {}
 
  @staticmethod
- def dataset_path(project: str, location: str, dataset: str,) -> str:
+ def dataset_path(project: str, dataset: str,) -> str:
  """Returns a fully-qualified dataset string."""
- return "projects/{project}/locations/{location}/datasets/{dataset}".format(
- project=project, location=location, dataset=dataset,
+ return "projects/{project}/datasets/{dataset}".format(
+ project=project, dataset=dataset,
  )
 
  @staticmethod
  def parse_dataset_path(path: str) -> Dict[str, str]:
  """Parses a dataset path into its component segments."""
- m = re.match(
- r"^projects/(?P<project>.+?)/locations/(?P<location>.+?)/datasets/(?P<dataset>.+?)$",
- path,
- )
+ m = re.match(r"^projects/(?P<project>.+?)/datasets/(?P<dataset>.+?)$", path)
  return m.groupdict() if m else {}
 
  @staticmethod
- def dataset_path(project: str, dataset: str,) -> str:
+ def dataset_path(project: str, location: str, dataset: str,) -> str:
  """Returns a fully-qualified dataset string."""
- return "projects/{project}/datasets/{dataset}".format(
- project=project, dataset=dataset,
+ return "projects/{project}/locations/{location}/datasets/{dataset}".format(
+ project=project, location=location, dataset=dataset,
  )
 
  @staticmethod
  def parse_dataset_path(path: str) -> Dict[str, str]:
  """Parses a dataset path into its component segments."""
- m = re.match(r"^projects/(?P<project>.+?)/datasets/(?P<dataset>.+?)$", path)
+ m = re.match(
+ r"^projects/(?P<project>.+?)/locations/(?P<location>.+?)/datasets/(?P<dataset>.+?)$",
+ path,
+ )
  return m.groupdict() if m else {}
 
  @staticmethod

tests/unit/gapic/aiplatform_v1/test_migration_service.py

@@ -1636,20 +1636,18 @@ def test_parse_dataset_path():
 
 def test_dataset_path():
  project = "squid"
- location = "clam"
- dataset = "whelk"
- expected = "projects/{project}/locations/{location}/datasets/{dataset}".format(
- project=project, location=location, dataset=dataset,
+ dataset = "clam"
+ expected = "projects/{project}/datasets/{dataset}".format(
+ project=project, dataset=dataset,
  )
- actual = MigrationServiceClient.dataset_path(project, location, dataset)
+ actual = MigrationServiceClient.dataset_path(project, dataset)
  assert expected == actual
 
 
 def test_parse_dataset_path():
  expected = {
- "project": "octopus",
- "location": "oyster",
- "dataset": "nudibranch",
+ "project": "whelk",
+ "dataset": "octopus",
  }
  path = MigrationServiceClient.dataset_path(**expected)
 
@@ -1659,18 +1657,20 @@ def test_parse_dataset_path():
 
 
 def test_dataset_path():
- project = "cuttlefish"
- dataset = "mussel"
- expected = "projects/{project}/datasets/{dataset}".format(
- project=project, dataset=dataset,
+ project = "oyster"
+ location = "nudibranch"
+ dataset = "cuttlefish"
+ expected = "projects/{project}/locations/{location}/datasets/{dataset}".format(
+ project=project, location=location, dataset=dataset,
  )
- actual = MigrationServiceClient.dataset_path(project, dataset)
+ actual = MigrationServiceClient.dataset_path(project, location, dataset)
  assert expected == actual
 
 
 def test_parse_dataset_path():
  expected = {
- "project": "winkle",
+ "project": "mussel",
+ "location": "winkle",
  "dataset": "nautilus",
  }
  path = MigrationServiceClient.dataset_path(**expected)