test: Reconcile Python namespaces for ifp and pdn in Bazel

- Add 'ifp' and 'pdn' to the list of dynamically generated Python shims in 'test/bazel_test.sh'. This allows Bazel integration tests to use the tool-specific namespaces (e.g., 'import ifp', 'import pdn') seamlessly, matching the CMake namespace behavior.
- Remove try-except namespace fallback workarounds from 'init_floorplan_flip_sites.py', 'init_floorplan_gap.py', and 'pdn_aux.py'.
- Remove redundant native 'py_test' targets and 'helpers' py_library from 'src/pdn/test/BUILD'. These duplicate targets are already 100% covered by the 'regression_test' macro targets (e.g., ':asap7_taper-py_test') which correctly utilize the test runner's shim environment.

TAG=agy
CONV=56950afb-bfb2-4074-bfcb-927c5a0f1057

Signed-off-by: Drew Lewis <cannada@google.com>

Author: calewis

src/ifp/test/init_floorplan_flip_sites.py

@@ -12,12 +12,8 @@
 
 floorplan = design.getFloorplan()
 site = floorplan.findSite("FreePDK45_38x28_10R_NP_162NW_34O")
-try:
-    flipped_sites = openroad.site_set()
-except AttributeError:
-    import ifp
-
-    flipped_sites = ifp.site_set()
+import ifp
+flipped_sites = ifp.site_set()
 flipped_sites.insert(site)
 
 floorplan.initFloorplan(

src/ifp/test/init_floorplan_gap.py

@@ -4,12 +4,8 @@
 import helpers
 import ifp_helpers as ifph
 
-try:
-    site_set = openroad.site_set
-except AttributeError:
-    import ifp
-
-    site_set = ifp.site_set
+import ifp
+site_set = ifp.site_set
 
 tech = Tech()
 tech.readLef("Nangate45/Nangate45.lef")

src/pdn/test/BUILD

@@ -141,24 +141,6 @@ COMPULSORY_TESTS = [
 
 ALL_TESTS = COMPULSORY_TESTS
 
-PY_TESTS = [
-    "asap7_taper",
-    "core_grid",
-    "core_grid_with_rings",
-    "core_grid_with_routing_obstructions",
-    "existing",
-    "macros",
-    "max_width",
-    "min_width",
-    "offgrid",
-    "power_switch",
-    "repair_vias",
-    "report",
-    "reset",
-    "ripup",
-    "widthtable",
-]
-
 filegroup(
     name = "regression_resources",
     # Dependencies could be specified more narrowly per test case,
@@ -321,30 +303,6 @@ filegroup(
     for test_name in ALL_TESTS
 ]
 
-py_library(
-    name = "helpers",
-    srcs = [
-        "helpers.py",
-        "pdn_aux.py",
-    ],
-    imports = ["."],
-    deps = [
-        "//python/openroad:openroadpy",
-    ],
-)
-
-[
-    py_test(
-        name = test_name,
-        srcs = [test_name + ".py"],
-        data = [":" + test_name + "_resources"],
-        deps = [
-            ":helpers",
-            "//python/openroad:openroadpy",
-        ],
-    )
-    for test_name in PY_TESTS
-]
 
 py_test(
     name = "pdn_man_tcl_check",

src/pdn/test/pdn_aux.py

@@ -3,10 +3,7 @@
 import odb
 from collections import defaultdict
 
-try:
-    import pdn
-except ModuleNotFoundError:
-    import openroad as pdn
+import pdn
 
 
 #  In tcl land, this lives in OpenRoad.tcl. However, it seems to be only called

test/bazel_test.sh

@@ -51,7 +51,7 @@ try:
 except ImportError:
     from openroadpy import *
 EOF
-	for module in ant cts drt grt rcx stt tap; do
+	for module in ant cts drt grt rcx stt tap ifp pdn; do
 		write_openroad_shim "${module}"
 	done