Upload 17 files

tests/test_bucket_urls.py

@@ -0,0 +1,29 @@
+from src.bucket import summarize_run_bundle
+from src.view_models import build_run_view_model
+import app
+
+
+def test_bucket_artifact_tree_url_has_no_main_segment():
+    url = app._run_artifacts_url("gist-20260602-120645-a145824d", "fffiloni/space-factory-runs")
+    assert url == "https://huggingface.co/buckets/fffiloni/space-factory-runs/tree/runs/gist-20260602-120645-a145824d"
+    assert "/tree/main/runs/" not in url
+
+
+def test_run_summary_artifact_url_has_no_main_segment():
+    summary = summarize_run_bundle(
+        "run-launch",
+        {"state": {"status": "running"}, "launch": {}, "summary_file": {}},
+        bucket_source="user/space-factory-runs",
+    )
+    assert summary["artifacts_url"] == "https://huggingface.co/buckets/user/space-factory-runs/tree/runs/run-launch"
+
+
+def test_view_model_artifact_url_has_no_main_segment():
+    bundle = {
+        "summary": {"status": "running", "created_at": "2026-06-02T12:00:00+00:00"},
+        "state": {},
+        "launch": {"status": "running"},
+        "events": [],
+    }
+    view = build_run_view_model("run-launch", bundle, bucket_source="user/space-factory-runs")
+    assert view["links"]["artifacts_url"] == "https://huggingface.co/buckets/user/space-factory-runs/tree/runs/run-launch"

tests/test_custom_ui_shell.py

@@ -1,242 +1,100 @@
 from pathlib import Path
 
 
-def test_custom_ui_assets_exist():
-    root = Path(__file__).resolve().parents[1]
-    assert (root / "web" / "index.html").exists()
-    assert (root / "web" / "static" / "app.css").exists()
-    assert (root / "web" / "static" / "app.js").exists()
-    assert (root / "web" / "static" / "progress.js").exists()
-
-
-def test_app_registers_custom_routes():
-    root = Path(__file__).resolve().parents[1]
-    content = (root / "app.py").read_text()
-    assert "def register_custom_routes" in content
-    assert '"/custom"' in content
-    assert '"/api/progress/from-events"' in content
-    assert '"/api/runs"' in content
+def root() -> Path:
+    return Path(__file__).resolve().parents[1]
 
 
-def test_custom_ui_contains_core_product_sections():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    for text in [
-        "New Build",
-        "Space Test",
-        "Run Explorer",
-        "Run storage",
-        "Live job progress",
-        "Run details",
-        "Validation & metrics",
-        "Latest events",
-    ]:
-        assert text in html
+def test_custom_ui_assets_exist():
+    base = root()
+    assert (base / "web" / "index.html").exists()
+    assert (base / "web" / "static" / "app.css").exists()
+    assert (base / "web" / "static" / "app.js").exists()
+    assert (base / "web" / "static" / "progress.js").exists()
+    assert (base / "web" / "static" / "runs.js").exists()
 
 
-def test_custom_ui_bridge_endpoints_are_registered():
-    root = Path(__file__).resolve().parents[1]
-    content = (root / "app.py").read_text()
+def test_app_registers_custom_routes_and_bridge_endpoints():
+    content = (root() / "app.py").read_text(encoding="utf-8")
+    assert "def register_custom_routes" in content
     for route in [
+        '"/custom"',
         '"/api/me"',
         '"/api/bucket/status"',
         '"/api/bucket/create"',
         '"/api/build"',
         '"/api/validate"',
+        '"/api/runs"',
         '"/api/runs/{run_id}/progress"',
+        '"/api/runs/resumable"',
+        '"/api/runs/{run_id}/view"',
     ]:
         assert route in content
-    assert "_oauth_context_from_request" in content
 
 
-def test_custom_ui_calls_real_bridge_endpoints():
-    root = Path(__file__).resolve().parents[1]
-    js = (root / "web" / "static" / "app.js").read_text()
-    for endpoint in ["/api/me", "/api/build", "/api/validate", "/api/bucket/status", "/api/bucket/create"]:
-        assert endpoint in js
-    assert "Custom UI shell is ready" not in js
-    assert "functional Gradio controls" not in js
-
-
-def test_custom_ui_has_validate_form():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    for element_id in ["validateSpace", "validateApi", "expectedOutput", "testArgs", "testKwargs", "launchValidate"]:
-        assert element_id in html
-
-
-def test_custom_ui_has_run_detail_explorer():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    for element_id in ["selectedRunId", "selectedModelId", "artifactList", "reportPreview", "runDetailStatus"]:
-        assert element_id in html
-    js = (root / "web" / "static" / "runs.js").read_text()
-    assert "selectRun" in js
-    assert "renderRunDetail" in js
-    assert "/api/runs/" in js
-
-
-def test_custom_ui_renders_metrics_from_progress():
-    root = Path(__file__).resolve().parents[1]
-    js = (root / "web" / "static" / "app.js").read_text()
-    assert "function renderMetrics" in js
-    assert "generation_smoke" in js
-    assert "recommended_zerogpu_duration" in js
+def test_one_page_workspace_contains_core_sections():
+    html = (root() / "web" / "index.html").read_text(encoding="utf-8")
+    for text in [
+        "New Build",
+        "Active Run",
+        "Runs Explorer",
+        "Selected run",
+        "Space Test",
+        "Validation result",
+        "Raw events",
+        "Full report",
+        "Artifacts",
+        "Run storage",
+    ]:
+        assert text in html
 
-def test_custom_ui_has_selected_run_actions_and_manual_hardware_panel():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    for element_id in ["manualActionPanel", "manualOpenSettings", "manualGoValidate", "selectedValidateBtn", "selectedOpenSettings", "selectedBlockers"]:
-        assert element_id in html
-    app_js = (root / "web" / "static" / "app.js").read_text()
-    assert "prepareValidationFromActiveRun" in app_js
-    assert "renderManualAction" in app_js
-    runs_js = (root / "web" / "static" / "runs.js").read_text()
-    assert "renderSelectedLinks" in runs_js
-    assert "renderBlockers" in runs_js
 
-def test_custom_ui_has_run_filters_and_resilient_polling():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    runs_js = (root / "web" / "static" / "runs.js").read_text()
-    css = (root / "web" / "static" / "app.css").read_text()
-    for element_id in ["runFilters", "refreshRuns", "pollWarning"]:
+def test_one_page_workspace_keeps_required_ids_for_live_bridge():
+    html = (root() / "web" / "index.html").read_text(encoding="utf-8")
+    required_ids = [
+        "launchBuild",
+        "checkBucket",
+        "createBucket",
+        "launchValidate",
+        "progressStatus",
+        "timeline",
+        "activityFeed",
+        "runFilters",
+        "runsTable",
+        "selectedRunId",
+        "artifactList",
+        "reportPreview",
+        "eventsPanel",
+        "spaceTestPreview",
+        "metricsList",
+        "manualActionPanel",
+        "selectedValidateBtn",
+        "resumeLatestRun",
+    ]
+    for element_id in required_ids:
         assert element_id in html
-    assert "runStatusFilter" in js
-    assert "localStorage.setItem('asf.activeRun'" in js
-    assert "restoreActiveRun" in js
-    assert "Progress polling failed" in js
-    assert "renderRunsFromCache" in runs_js
-    assert "state.runsCache" in runs_js
-    assert ".run-row.selected" in css
-
-def test_custom_ui_has_oauth_status_panel_and_login_link():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    app_py = (root / "app.py").read_text()
-    assert "authPanel" in html
-    assert "renderAuthWarnings" in js
-    assert "/oauth/huggingface/login" in js
-    assert '"/api/oauth/diagnostics"' in app_py
-    assert "public_oauth_context" in app_py
-
-def test_custom_ui_v30_bucket_gate_and_zerogpu_toggle():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    assert 'id="tryZeroGpu"' in html
-    assert 'disabled /> Try ZeroGPU first' not in html
-    assert 'id="launchBuild" disabled' in html
-    assert 'buildGateText' in html
-    assert 'effectivePreferredHardware' in js
-    assert 'setBucketReady' in js
-    assert 'Create or check your private run bucket before launching a build' in js
-
-
-def test_custom_ui_v30_css_has_overflow_and_duplicate_cleanup():
-    root = Path(__file__).resolve().parents[1]
-    css = (root / "web" / "static" / "app.css").read_text()
-    assert '.app-shell{min-height:100vh;max-width:1540px' in css
-    assert '.top-tabs{display:flex' in css
-    assert '.runs-table{border:1px solid var(--line);border-radius:14px;padding:0;overflow:hidden' in css
-    assert 'font-family:ui-monospace' in css
-    assert '.auth-panel,.operation-status{' in css
-
-def test_custom_ui_v31_feedback_and_loading_states():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    css = (root / "web" / "static" / "app.css").read_text()
-    api_js = (root / "web" / "static" / "api.js").read_text()
-    progress_js = (root / "web" / "static" / "progress.js").read_text()
-    assert "operationStatus" in html
-    assert "lastPolled" in html
-    assert "setOperation" in js
-    assert "setBusy" in js
-    assert "validateBuildForm" in js
-    assert "validateValidationForm" in js
-    assert "TERMINAL_STATUSES" in js
-    assert "Run bucket is not ready" in Path(root / "app.py").read_text()
-    assert ".operation-status" in css
-    assert "parseApiError" in api_js
-    assert "lastPolled" in progress_js
-
-
-def test_v33_root_custom_ui_not_iframe_preview():
-    root = Path(__file__).resolve().parents[1]
-    app_py = (root / "app.py").read_text()
-    readme = (root / "README.md").read_text()
-    assert '@fastapi_app.get("/", response_class=HTMLResponse)' in app_py
-    assert 'attach_huggingface_oauth(fastapi_app)' in app_py
-    assert '<iframe src="/custom"' not in app_py
-    assert 'Custom UI preview' not in app_py
-    assert 'The product root (`/`) is the custom dashboard' in readme
 
 
-def test_v34_no_explicit_uvicorn_launcher():
-    root = Path(__file__).resolve().parents[1]
-    app_py = (root / "app.py").read_text()
-    assert "uvicorn.run(" not in app_py
-    assert 'if __name__ == "__main__"' not in app_py
-    assert "app = create_app()" in app_py
-
-
-def test_v37_simplified_navigation_and_artifact_path():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    assert 'data-view="build"' in html
-    assert 'data-view="validate"' in html
-    assert 'data-view="runs"' not in html
-    assert 'data-view="storage"' not in html
-    assert 'data-view="about"' not in html
-    assert 'tree/main/runs/${runId}' in js
-
-
-def test_v38_new_build_mode_and_local_run_filtering():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    runs_js = (root / "web" / "static" / "runs.js").read_text()
-    css = (root / "web" / "static" / "app.css").read_text()
-    assert "prepareNewBuild" in html
-    assert "buildModePanel" in html
-    assert "Preparing a new build" in html
-    assert "function prepareNewBuild" in js
-    assert "setBuildModeInspecting" in js
-    assert "filteredRuns" in runs_js
-    assert "apiGet(`/api/runs?limit=100" in runs_js
-    assert "status=${encodeURIComponent(status)}" not in runs_js
-    assert ".build-mode-panel" in css
-
-
-def test_v40_tabbed_ui_logout_and_compact_progress():
-    root = Path(__file__).resolve().parents[1]
-    html = (root / "web" / "index.html").read_text()
-    css = (root / "web" / "static" / "app.css").read_text()
-    js = (root / "web" / "static" / "app.js").read_text()
-    runs_js = (root / "web" / "static" / "runs.js").read_text()
-    assert 'class="top-tabs"' in html
-    assert 'data-view="progress"' in html
-    assert 'id="logoutLink"' in html
-    assert 'class="sidebar"' not in html
-    assert 'max-height:330px' not in css
-    assert 'overflow:auto;padding-right' not in css
-    assert "switchView('progress')" in js
-    assert 'inferredJobUrl' in runs_js
-
-
-def test_v43_run_view_model_contract_is_exposed_to_custom_ui():
-    root = Path(__file__).resolve().parents[1]
-    app_py = (root / "app.py").read_text()
-    html = (root / "web" / "index.html").read_text()
-    js = (root / "web" / "static" / "app.js").read_text() + (root / "web" / "static" / "progress.js").read_text()
-    assert '"/api/runs/resumable"' in app_py
-    assert '"/api/runs/{run_id}/view"' in app_py
-    assert "build_run_view_model" in app_py
-    for element_id in ["runSummaryCard", "activityFeed", "zeroGpuChecklist", "issueCard", "spaceTestPreview"]:
-        assert element_id in html
-    assert "bootstrapRunRecovery" in js
-    assert "renderRunViewModel" in js
+def test_frontend_calls_real_bridge_endpoints_and_keeps_run_logic():
+    app_js = (root() / "web" / "static" / "app.js").read_text(encoding="utf-8")
+    runs_js = (root() / "web" / "static" / "runs.js").read_text(encoding="utf-8")
+    progress_js = (root() / "web" / "static" / "progress.js").read_text(encoding="utf-8")
+    for endpoint in ["/api/me", "/api/build", "/api/validate", "/api/bucket/status", "/api/bucket/create"]:
+        assert endpoint in app_js
+    assert "selectRun" in runs_js
+    assert "renderRunDetail" in runs_js
+    assert "renderRunViewModel" in progress_js
+    assert "renderActivityFeed" in progress_js
+    assert "renderDiagnostics" in progress_js
+
+
+def test_css_contains_one_page_responsive_layout():
+    css = (root() / "web" / "static" / "app.css").read_text(encoding="utf-8")
+    for snippet in [
+        ".workspace-grid{display:grid;grid-template-columns:360px minmax(0,1fr) 340px",
+        ".space-test-grid{display:grid;grid-template-columns:minmax(0,1.15fr) 360px",
+        "word-break:normal",
+        ".run-row.selected",
+        "@media (max-width: 860px)",
+    ]:
+        assert snippet in css.replace(" ", "") if "word-break:normal" == snippet else snippet in css

tests/test_pass46_ui_regressions.py

@@ -1,16 +1,15 @@
 from pathlib import Path
 
 
-def test_selecting_run_does_not_force_tab_switch():
+def test_selecting_run_still_does_not_force_hidden_navigation():
     text = Path("web/static/runs.js").read_text(encoding="utf-8")
-    select_run = text.split("async function selectRun", 1)[1].split("function renderRunDetail", 1)[0]
+    select_run = text.split("async function selectRun", 1)[1].split("function renderSelectedLinks", 1)[0]
     assert "switchView(" not in select_run
     assert "Keep the user on the current tab" in select_run
 
 
-def test_mobile_css_prevents_letter_by_letter_wrapping():
+def test_mobile_css_prevents_letter_by_letter_wrapping_in_one_page_layout():
     css = Path("web/static/app.css").read_text(encoding="utf-8")
-    assert "Pass 46: mobile framing" in css
-    assert "word-break: normal" in css
-    assert "grid-template-columns:1fr!important" in css
-    assert ".nav-item{flex:0 0 auto" in css
+    assert "word-break:normal" in css.replace(" ", "")
+    assert "@media (max-width: 860px)" in css
+    assert ".workspace-grid,.space-test-grid,.editor-grid,.two-cols,.summary-meta-grid,.diagnostic-list,.bottom-split,.three-way,.run-storage,.test-top-row{grid-template-columns:1fr}".replace(" ", "") in css.replace(" ", "")

tests/test_pass47_simple_spec_reset.py

@@ -1,41 +1,30 @@
 from pathlib import Path
 
 
-def test_pass47_has_only_three_public_tabs():
+def test_one_page_workspace_has_no_public_tabs_anymore():
     html = Path("web/index.html").read_text(encoding="utf-8")
-    assert 'data-view="build"' in html
-    assert 'data-view="progress"' in html
-    assert 'data-view="validate"' in html
-    assert 'data-view="about-page"' not in html
-    assert html.count('class="nav-item') == 3
+    assert 'data-view="build"' not in html
+    assert 'data-view="progress"' not in html
+    assert 'data-view="validate"' not in html
+    assert "Runs Explorer" in html
+    assert "Active Run" in html
 
 
-def test_build_tab_is_launch_focused_without_visible_run_list():
+def test_new_build_panel_is_left_column_and_launch_focused():
     html = Path("web/index.html").read_text(encoding="utf-8")
-    build_section = html.split('data-section="build"', 1)[1].split('data-section="progress"', 1)[0]
-    assert "New Build" in build_section
-    assert "Run storage" in build_section
-    assert "Launch result" in build_section
-    assert "runsTableHome" not in build_section
-    assert "Run Explorer" not in build_section
+    assert 'id="newBuildCard"' in html
+    for text in ["Launch a private Space", "Run storage", "Build private Space", "Launch result"]:
+        assert text in html
 
 
-def test_runs_tab_is_the_single_run_inspection_surface():
+def test_runs_explorer_and_selected_run_are_integrated_on_same_page():
     html = Path("web/index.html").read_text(encoding="utf-8")
-    runs_section = html.split('data-section="progress"', 1)[1].split('data-section="validate"', 1)[0]
-    for required in ["Inspect existing runs", "runsTable", "timeline", "Latest events", "Report preview", "Artifacts"]:
-        assert required in runs_section
+    for text in ["Runs Explorer", "Search runs by ID, model, or space", "Selected run", "Prefill Space Test"]:
+        assert text in html
 
 
-def test_launching_does_not_force_runs_tab():
-    js = Path("web/static/app.js").read_text(encoding="utf-8")
-    set_active = js.split("function setActiveRun", 1)[1].split("async function loadAppInfo", 1)[0]
-    assert "switchView(" not in set_active
-    assert "renderLaunchResult" in js
-
-
-def test_pass47_mobile_layout_is_single_column_safe():
-    css = Path("web/static/app.css").read_text(encoding="utf-8")
-    assert "Pass 47: deliberately simple three-tab interface" in css
-    assert ".simple-build-grid,.simple-runs-grid,.simple-validate-grid,.simple-storage-card{grid-template-columns:1fr!important}" in css
-    assert "word-break: normal" in css
+def test_space_test_panel_is_inline_not_tabbed():
+    html = Path("web/index.html").read_text(encoding="utf-8")
+    assert 'id="spaceTestSection"' in html
+    assert "Validate an existing Space" in html
+    assert "Launch validation" in html

tests/test_run_bundle_details.py

@@ -42,7 +42,7 @@ def test_summarize_run_bundle_recovers_launch_only_running_job():
     assert summary["target_space"] == "alice/z-image"
     assert summary["job_id"] == "abc123"
     assert summary["job_url"].endswith("/abc123")
-    assert summary["artifacts_url"].endswith("/tree/main/runs/run-launch")
+    assert summary["artifacts_url"].endswith("/tree/runs/run-launch")
 
 
 def test_summarize_run_bundle_uses_summary_file_fallback():