User/manishkg/remove map token (#257)

* Remove configuration endpoint and update related test to expect 404

* Linting

* correctly specify build order

---------

Co-authored-by: Gustavo Hidalgo <[email protected]>

Author: srijan55

deployment/bin/lib

@@ -83,7 +83,8 @@ function cluster_login() {
     echo "Converting kubeconfig..."
     kubelogin convert-kubeconfig \
         -l azurecli \
-        --kubeconfig=kubeconfig
+        --kubeconfig=kubeconfig \
+        --yes
     export KUBECONFIG=kubeconfig
 }
 

docker-compose.dev.yml

@@ -5,6 +5,8 @@ services:
     build:
       context: .
       dockerfile: pcstac/Dockerfile.dev
+      additional_contexts:
+        pc-apis-stac: "service:stac"
     environment:
       - ENVIRONMENT=development
       - POSTGRES_USER=username
@@ -73,6 +75,8 @@ services:
     build:
       context: .
       dockerfile: pctiler/Dockerfile.dev
+      additional_contexts:
+        pc-apis-tiler: "service:tiler"
     environment:
       - ENVIRONMENT=development
       - FF_VRT="yes"

pctiler/pctiler/endpoints/configuration.py

@@ -25,14 +25,7 @@
 from pccommon.openapi import fixup_schema
 from pccommon.redis import connect_to_redis
 from pctiler.config import get_settings
-from pctiler.endpoints import (
-    configuration,
-    health,
-    item,
-    legend,
-    pg_mosaic,
-    vector_tiles,
-)
+from pctiler.endpoints import health, item, legend, pg_mosaic, vector_tiles
 from pctiler.middleware import ModifyResponseMiddleware
 
 # Get the root path if set in the environment
@@ -114,12 +107,6 @@ async def lifespan(app: FastAPI) -> AsyncGenerator:
     tags=["Collection vector tile endpoints"],
 )
 
-app.include_router(
-    configuration.configuration_router,
-    prefix=settings.configuration_endpoint_prefix,
-    tags=["Map configuration endpoints"],
-)
-
 app.include_router(health.health_router, tags=["Liveliness/Readiness"])
 
 add_timeout(app, settings.request_timeout)

pctiler/pctiler/main.py

@@ -6,5 +6,5 @@
 @pytest.mark.integration
 async def test_config_token_endpoint(client: AsyncClient) -> None:
     response = await client.get("/config/map/token")
-    assert response.status_code == 200
-    assert response.json()["token"]
+    # We expect this path to be not found
+    assert response.status_code == 404

pctiler/tests/endpoints/test_config.py

@@ -28,6 +28,6 @@ if [ "${BASH_SOURCE[0]}" = "${0}" ]; then
     flake8 pcstac/pcstac pcstac/tests
 
     echo "Running unit tests for stac..."
-    python3 -m pytest pcstac/tests
+    python3 -m pytest -vv pcstac/tests
 
 fi