Fix format

Author: rickwu666666

Tools/test_plan_visualization/README.md

@@ -4,32 +4,49 @@ A web application to inspect, filter, and explore Checkbox jobs and test plans.
 
 ## Overview
 
-This application parses the [Checkbox](https://github.com/canonical/checkbox) repository and provides a local search engine for job units and test plans. It supports two views:
+This application parses the
+[Checkbox](https://github.com/canonical/checkbox) repository and provides
+a local search engine for job units and test plans. It supports two views:
 
 - **Jobs View** — browse and filter individual job units
-- **Test Plans View** — search test plans and explore their nested structure down to included jobs
+- **Test Plans View** — search test plans and explore their nested structure
+  down to included jobs
 
 ## Features
 
 - **Two-View UI**: Toggle between Jobs and Test Plans views from the header.
-- **Automated Parsing**: Scans `.pxu` files and extracts both job units (including legacy `plugin:`-style entries) and test plan units.
-- **Job Filters**: Filter by Provider, Category, Environ, Manifest Keys, and Template ID presence.
-- **Search**: Search by job ID, test plan ID, or plan name — searches across both jobs and the test plans they belong to.
-- **Test Plan Tree**: In the Test Plans view, expand plans to see nested sub-plans and directly included jobs.
-- **Exclude Support**: Jobs excluded by a test plan's `exclude:` field are shown with a strikethrough and an EXCLUDED badge — they remain visible but clearly marked.
-- **Job Details Modal**: Click Details on any job to see its full attributes and the complete test plan hierarchy it belongs to.
-- **Plan Details Modal**: Click Details on any test plan card to see all raw plan attributes, include/exclude patterns, and nested parts.
-- **Dynamic Filters**: Dropdown options update dynamically based on current selections.
-- **Provider Resolution**: Automatically resolves provider namespaces from `manage.py`.
-- **Compare Plans**: Switch to the Compare view, enter two plan IDs, and see a three-column diff — jobs only in Plan 1, jobs in both, and jobs only in Plan 2 (excludes are applied before comparing).
+- **Automated Parsing**: Scans `.pxu` files and extracts both job units
+  (including legacy `plugin:`-style entries) and test plan units.
+- **Job Filters**: Filter by Provider, Category, Environ, Manifest Keys,
+  and Template ID presence.
+- **Search**: Search by job ID, test plan ID, or plan name — searches across
+  both jobs and the test plans they belong to.
+- **Test Plan Tree**: In the Test Plans view, expand plans to see nested
+  sub-plans and directly included jobs.
+- **Exclude Support**: Jobs excluded by a test plan's `exclude:` field are
+  shown with a strikethrough and an EXCLUDED badge — they remain visible
+  but clearly marked.
+- **Job Details Modal**: Click Details on any job to see its full attributes
+  and the complete test plan hierarchy it belongs to.
+- **Plan Details Modal**: Click Details on any test plan card to see all raw
+  plan attributes, include/exclude patterns, and nested parts.
+- **Dynamic Filters**: Dropdown options update dynamically based on
+  current selections.
+- **Provider Resolution**: Automatically resolves provider namespaces
+  from `manage.py`.
+- **Compare Plans**: Switch to the Compare view, enter two plan IDs, and see
+  a three-column diff — jobs only in Plan 1, jobs in both, and jobs only in
+  Plan 2 (excludes are applied before comparing).
 
 ## Getting Started
 
 ### Option A: Run Locally
 
 **Prerequisites:** Python 3.10+, `git`
 
-1.  **Run the startup script** — it automatically creates a virtual environment, installs dependencies, clones/updates the checkbox repo, builds the database, and starts the server:
+1.  **Run the startup script** — it automatically creates a virtual
+    environment, installs dependencies, clones/updates the checkbox repo,
+    builds the database, and starts the server:
     ```bash
     ./run.sh
     ```
@@ -53,16 +70,19 @@ This application parses the [Checkbox](https://github.com/canonical/checkbox) re
     sudo docker build -t checkbox-job-db .
     ```
 
-2.  **Run the container** (the startup script runs automatically inside the container):
+2.  **Run the container** (the startup script runs automatically
+    inside the container):
     ```bash
     sudo docker run -p 8888:8888 checkbox-job-db
     ```
 
-    > If you get a "port already allocated" error, stop any existing container first:
+    > If you get a "port already allocated" error, stop any existing
+    > container first:
     > ```bash
     > sudo docker stop $(sudo docker ps -q --filter publish=8888)
     > ```
-    > If the port still appears stuck with no process using it, restart the Docker daemon:
+    > If the port still appears stuck with no process using it, restart
+    > the Docker daemon:
     > ```bash
     > sudo systemctl restart docker
     > ```
@@ -84,19 +104,27 @@ This application parses the [Checkbox](https://github.com/canonical/checkbox) re
 
 | Endpoint | Description |
 |---|---|
-| `GET /api/jobs` | List jobs with optional filters: `provider`, `category`, `environ`, `manifest`, `has_template_id`, `search` |
-| `GET /api/options` | Get available filter values for the current filter selection |
-| `GET /api/testplans?job_id=` | Get the full test plan ancestry for a given job ID |
-| `GET /api/plan-tree?search=` | Search test plans and return their full nested tree with included jobs (excluded jobs marked) |
-| `GET /api/plan-details?plan_id=` | Get all attributes, include/exclude patterns, and nested parts for a single test plan |
-| `GET /api/compare-plans?plan1=&plan2=` | Compare effective job sets of two test plans (excludes applied), returning only-in-1, in-both, only-in-2 |
+| `GET /api/jobs` | List jobs with optional filters: `provider`,
+`category`, `environ`, `manifest`, `has_template_id`, `search` |
+| `GET /api/options` | Get available filter values for the
+current filter selection |
+| `GET /api/testplans?job_id=` | Get the full test plan ancestry
+for a given job ID |
+| `GET /api/plan-tree?search=` | Search test plans and return their
+full nested tree with included jobs (excluded jobs marked) |
+| `GET /api/plan-details?plan_id=` | Get all attributes,
+include/exclude patterns, and nested parts for a single test plan |
+| `GET /api/compare-plans?plan1=&plan2=` | Compare effective job sets
+of two test plans (excludes applied), returning only-in-1,
+in-both, only-in-2 |
 
 ## Troubleshooting: Port 8888
 
 **Check what is using port 8888:**
 ```bash
 ss -tlnp sport = :8888          # shows process if owned by current user
-sudo ss -tlnp sport = :8888     # shows process for all users (including Docker/root)
+sudo ss -tlnp sport = :8888     # shows process for all users
+                                # (including Docker/root)
 ```
 
 **Kill local uvicorn process:**
@@ -126,8 +154,14 @@ sudo systemctl restart docker
 
 ## Notes
 
-- The database is rebuilt every time the server starts (both locally and in Docker).
+- The database is rebuilt every time the server starts
+  (both locally and in Docker).
 - Both `unit: job` and legacy `plugin:`-style job blocks are parsed.
-- Test plan nested hierarchy is traversed recursively; cycle detection is built in.
-- The `exclude:` field in test plans is parsed and stored. Excluded jobs are shown with a strikethrough in the plan tree and are removed from the effective job set when comparing plans.
-- Environment variables referenced in a job's `command:` field (e.g. `$SERIAL_PORTS_STATIC`) are automatically added to the Environ filter even if no explicit `environ:` field is declared.
+- Test plan nested hierarchy is traversed recursively;
+  cycle detection is built in.
+- The `exclude:` field in test plans is parsed and stored. Excluded jobs
+  are shown with a strikethrough in the plan tree and are removed from
+  the effective job set when comparing plans.
+- Environment variables referenced in a job's `command:` field
+  (e.g. `$SERIAL_PORTS_STATIC`) are automatically added to the Environ
+  filter even if no explicit `environ:` field is declared.

Tools/test_plan_visualization/app/database.py

@@ -11,33 +11,40 @@
 
 Base = declarative_base()
 
+
 class Job(Base):
     __tablename__ = "jobs"
 
     id = Column(Integer, primary_key=True, index=True)
     job_id = Column(String, index=True)
     provider = Column(String, index=True)
     category_id = Column(String, index=True)
-    environ = Column(Text) # Stored as JSON string or comma separated
-    manifest = Column(Text) # Stored as JSON string or comma separated keywords from `requires`
+    environ = Column(Text)  # Stored as JSON string or comma separated
+    manifest = Column(
+        Text
+    )  # Stored as JSON string or comma separated keywords from `requires`
     command = Column(Text)
     summary = Column(Text)
     description = Column(Text)
-    unit_type = Column(String) # 'job', 'test plan', etc.
-    data = Column(Text) # JSON string of all attributes
+    unit_type = Column(String)  # 'job', 'test plan', etc.
+    data = Column(Text)  # JSON string of all attributes
+
 
 class TestPlan(Base):
     __tablename__ = "test_plans"
 
     id = Column(Integer, primary_key=True, index=True)
-    plan_id = Column(String, index=True)   # e.g. bluetooth-cert-automated
-    full_id = Column(String, index=True)   # with namespace, e.g. com.canonical.certification::bluetooth-cert-automated
+    plan_id = Column(String, index=True)  # e.g. bluetooth-cert-automated
+    full_id = Column(
+        String, index=True
+    )  # e.g. com.canonical.certification::bluetooth-cert-automated
     provider = Column(String, index=True)
     name = Column(Text)
-    include = Column(Text)    # JSON list of raw include patterns
-    exclude = Column(Text)    # JSON list of raw exclude patterns
+    include = Column(Text)  # JSON list of raw include patterns
+    exclude = Column(Text)  # JSON list of raw exclude patterns
     nested_part = Column(Text)  # JSON list of nested test plan IDs
-    data = Column(Text)       # JSON of all raw attributes
+    data = Column(Text)  # JSON of all raw attributes
+
 
 def get_db():
     db = SessionLocal()