ci: consistent management of DOMAIN variable between backend and frontend

app.py

@@ -9,7 +9,7 @@ from fastapi.responses import FileResponse, JSONResponse
 from fastapi.staticfiles import StaticFiles
 import uvicorn
 
-from my_ghost_writer.constants import ALLOWED_ORIGIN_LIST, IS_TESTING, LOG_LEVEL, STATIC_FOLDER, app_logger
+from my_ghost_writer.constants import ALLOWED_ORIGIN_LIST, DOMAIN, IS_TESTING, LOG_LEVEL, PORT, STATIC_FOLDER, app_logger
 from my_ghost_writer.type_hints import RequestTextFrequencyBody
 
 
@@ -87,7 +87,7 @@ def index() -> FileResponse:
 
 if __name__ == "__main__":
     try:
-        uvicorn.run("app:app", host="0.0.0.0", port=7860, reload=bool(IS_TESTING))
+        uvicorn.run("app:app", host=DOMAIN, port=PORT, reload=bool(IS_TESTING))
     except Exception as ex:
         print(f"fastapi/gradio application {fastapi_title}, exception:{ex}!")
         app_logger.exception(f"fastapi/gradio application {fastapi_title}, exception:{ex}!")

my_ghost_writer/constants.py

@@ -7,7 +7,9 @@ from my_ghost_writer import session_logger
 
 PROJECT_ROOT_FOLDER = Path(__file__).parent.parent
 STATIC_FOLDER = PROJECT_ROOT_FOLDER / "static"
-ALLOWED_ORIGIN_LIST = os.getenv('ALLOWED_ORIGIN', 'http://localhost:7860').split(",")
+DOMAIN=os.getenv("DOMAIN", "localhost")
+PORT=os.getenv("PORT", 7860)
+ALLOWED_ORIGIN_LIST = os.getenv('ALLOWED_ORIGIN', f'http://{DOMAIN}:{PORT}').split(",")
 LOG_JSON_FORMAT = bool(os.getenv("LOG_JSON_FORMAT"))
 IS_TESTING = bool(os.getenv('IS_TESTING', ""))
 LOG_LEVEL = os.getenv("LOG_LEVEL", "INFO")

static/playwright.config.ts

@@ -26,7 +26,7 @@ export default defineConfig({
   /* Shared settings for all the projects below. See https://playwright.dev/docs/api/class-testoptions. */
   use: {
     /* Base URL to use in actions like `await page.goto('/')`. */
-    baseURL: 'http://localhost:7860',
+    baseURL: process.env.DOMAIN_PORT ?? 'http://localhost:7860',
     browserName: "chromium",
     viewport: { width: 1600, height: 1200 },
     ignoreHTTPSErrors: true,

static/template.env

@@ -1,2 +1,2 @@
-DOMAIN=http://localhost:7860
+DOMAIN_PORT=http://localhost:7860
 ALLOWED_ORIGIN=http://localhost:7860,http://localhost:8000

static/tests/test-word-frequency-1.spec.ts

@@ -5,7 +5,7 @@ import { fileReader, loopOverTablesAndClickOnUrls } from './test-helper'
 test(`test: word frequency (short text input)`, async ({ page }) => {
   const wordFreqTable0AriaSnapshot1FilePath = `${import.meta.dirname}/test-word-frequency-1-table0-aria-snapshot.txt`
   const wordFreqTable13AriaSnapshot1FilePath = `${import.meta.dirname}/test-word-frequency-1-table13-aria-snapshot.txt`
-  await page.goto(process.env.DOMAIN ?? "http://localhost:7860/");
+  await page.goto(process.env.DOMAIN_PORT ?? "/");
   console.log(page.url())
 
   await page.getByRole('button', { name: 'btn4-getWordFrequency' }).click();
@@ -28,7 +28,7 @@ test(`test: word frequency (short text input)`, async ({ page }) => {
 
 test(`test: word frequency (long, multi line text input)`, async ({ page }) => {
   const testLLMTextFilePath = `${import.meta.dirname}/../../tests/events/llm_generated_story_1.txt`
-  await page.goto(process.env.DOMAIN ?? "http://localhost:7860/");
+  await page.goto(process.env.DOMAIN_PORT ?? "/");
   console.log(page.url())
 
   console.log("Let's try with a much longer, multiline text while scrolling the conteditable div on click")

static/tests/test-word-frequency-2-filtering-sorting.spec.ts

@@ -10,7 +10,7 @@ const expectedInitialCount = 14;
     {count: expectedInitialCount, filter: "", testIdx: 2},
 ].forEach(({ count, filter, testIdx }) => {
     test(`words frequency: filtering, ordering and sorting - ${testIdx}, sort/order ${sortOrder}.`, async ({ page }) => {
-        await page.goto('http://localhost:7860/');
+        await page.goto(process.env.DOMAIN_PORT ?? "/");
         await page.getByRole('button', { name: 'btn4-getWordFrequency' }).click();
         let containerTable = page.getByLabel('words-frequency', { exact: true })
         await page.waitForTimeout(100)

static/tests/test-word-frequency-3-sorting.spec.ts

@@ -5,7 +5,7 @@ import { assertTableStap } from './test-helper'
 const count = 14
 test.describe(`word frequency: sorting output with ${14} tables`, () => {
     test.beforeEach(async({page}) => {
-        await page.goto('http://localhost:7860/');
+        await page.goto(process.env.DOMAIN_PORT ?? "/");
         await page.getByRole('button', { name: 'btn4-getWordFrequency' }).click();
     });
     

template.env

@@ -1,2 +1,3 @@
-DOMAIN=http://localhost:7860
+DOMAIN=localhost
+PORT=7860
 ALLOWED_ORIGIN=http://localhost:7860,http://localhost:8000