address review feedback

Author: parthea

.generator/cli.py

@@ -285,15 +285,9 @@ def _read_bazel_build_py_rule(api_path: str, source: str) -> Dict:
 
     Returns:
         Dict: A dictionary containing the parsed attributes of the `_py_gapic` rule.
-
-    Raises:
-        ValueError: If the BUILD.bazel file cannot be read or a '_py_gapic' rule is not found.
     """
     build_file_path = f"{source}/{api_path}/BUILD.bazel"
-    try:
-        content = _read_text_file(build_file_path)
-    except FileNotFoundError:
-        raise ValueError(f"BUILD.bazel file not found at {build_file_path}")
+    content = _read_text_file(build_file_path)
 
     result = parse_googleapis_content.parse_content(content)
     py_gapic_entries = [key for key in result.keys() if key.endswith("_py_gapic")]

.generator/test_cli.py

@@ -309,13 +309,6 @@ def test_read_bazel_build_py_rule_success(mocker, mock_build_bazel_file):
     assert py_gapic_config["opt_args"] == ["python-gapic-namespace=google.cloud"]
 
 
-def test_read_bazel_build_py_rule_not_found(mocker):
-    """Tests failure when the BUILD.bazel file is missing."""
-    mocker.patch("cli._read_text_file", side_effect=FileNotFoundError)
-    with pytest.raises(ValueError, match="BUILD.bazel file not found"):
-        _read_bazel_build_py_rule("non/existent/v1", "source")
-
-
 def test_get_api_generator_options_all_options():
     """Tests option extraction when all relevant fields are present."""
     api_path = "google/cloud/language/v1"