chore(pems_streamlit): remove default placeholder app

Author: thekaveman

pems_streamlit/src/pems_streamlit/__init__.py

@@ -1,3 +0,0 @@
-import streamlit as st
-
-st.write("https://pems.dot.ca.gov")

pems_streamlit/src/pems_streamlit/utils.py

@@ -14,10 +14,6 @@ def _convert_to_pages(apps: list[Path]) -> list[StreamlitPage]:
     return list(map(_make_app_page, apps))
 
 
-def _default_app_page() -> StreamlitPage:
-    return st.Page(APP_DIR / "__init__.py", default=True, title="PeMS Streamlit apps")
-
-
 def _discover_apps() -> list[Path]:
     logger.info("Beginning streamlit app discovery")
 
@@ -51,8 +47,6 @@ def _make_app_page(app_path: Path) -> StreamlitPage:
 
 
 def discover_apps() -> list[StreamlitPage]:
-    default_app_page = _default_app_page()
     apps = _discover_apps()
     pages = _convert_to_pages(apps)
-    pages.insert(0, default_app_page)
     return pages

tests/pytest/pems_streamlit/test_utils.py

@@ -1,7 +1,6 @@
 from pathlib import Path
 
 import pytest
-from streamlit.navigation.page import StreamlitPage
 
 from pems_streamlit import utils
 
@@ -34,13 +33,6 @@ def test_convert_to_pages(mocker, app_paths):
         assert mocker.call(path) in page_factory.mock_calls
 
 
-def test_default_app_page():
-    page = utils._default_app_page()
-
-    assert page._default
-    assert page.url_path == ""
-
-
 def test__discover_apps(app_paths):
     discovered = utils._discover_apps()
 
@@ -70,12 +62,7 @@ def test_discover_apps(mocker):
     mock_discover = mocker.patch("pems_streamlit.utils._discover_apps", return_value=[])
     mock_convert = mocker.patch("pems_streamlit.utils._convert_to_pages", return_value=[])
 
-    result = utils.discover_apps()
+    utils.discover_apps()
 
     mock_discover.assert_called_once()
     mock_convert.assert_called_once()
-
-    assert len(result) == 1
-    page = result[0]
-    assert isinstance(page, StreamlitPage)
-    assert page._default