Added test page logging, added favicon to prevent 404 message.

michaelsharman · May 20, 2024 · 55ebb7b · 55ebb7b
1 parent 33d42aa
commit 55ebb7b
Show file tree

Hide file tree

Showing 3 changed files with 37 additions and 5 deletions.
diff --git a/tests/api-server/src/server.js b/tests/api-server/src/server.js
@@ -2,6 +2,7 @@ const express = require('express');
 const net = require('net');
 const path = require('path');
 const signature = require('./signature');
+const { log } = require('console');
 
 const app = express();
 const PORT = process.env.PORT || 5150;
@@ -28,11 +29,15 @@ app.get('/itemsapi', (req, res) => {
     res.render('itemsapi', { signature: JSON.stringify(signatureData) });
 });
 
-function startServer() {
-    checkServer(PORT, isRunning => {
+async function startServer() {
+    checkServer(PORT, async isRunning => {
         if (!isRunning) {
+            console.time('server-start-complete');
             console.log('Starting the server...');
-            server = app.listen(PORT);
+            server = app.listen(PORT, () => {
+                console.log('Server started on port', PORT);
+                console.timeEnd('server-start-complete');
+            });
         }
     });
 }

diff --git a/tests/api-server/src/views/itemsapi.ejs b/tests/api-server/src/views/itemsapi.ejs
@@ -1,5 +1,9 @@
 <!DOCTYPE html>
 <html lang="en">
+    <head>
+        <meta charset="utf-8" />
+        <link rel="shortcut icon" href="https://demos.learnosity.com/static/images/favicon.ico" type="image/x-icon" />
+    </head>
     <body>
         <div id="flag"></div>
         <div id="learnosity_assess"></div>
@@ -15,7 +19,7 @@
             function init() {
                 // A DOM element that we can set so the test script knows the API has rendered
                 document.getElementById('flag').classList.add('has-loaded');
-                console.log('Items API ready!');
+                // console.log('Items API ready!');
                 window.launch(itemsApp);
             }
             function error(e) {

diff --git a/tests/runner.test.js b/tests/runner.test.js
@@ -6,13 +6,36 @@ describe('LT Core', () => {
     let page;
 
     beforeAll(async () => {
-        server.startServer();
+        await server.startServer();
 
         browser = await puppeteer.launch();
         page = await browser.newPage();
 
+        // Log all console messages from the test page
+        page.on('console', consoleMsg => {
+            if (!consoleMsg.text().includes('Learnosity developer version')) {
+                console.log(`== From test page console: ${consoleMsg.text()}`);
+            }
+        });
+        page.on('requestfailed', request => {
+            console.log(`${request.url()} failed to load. Reason: ${request.failure().errorText}`);
+        });
+        page.on('pageerror', error => {
+            console.log(`Page error: ${error.message}`);
+        });
+        page.on('response', response => {
+            if (!response.ok()) {
+                console.log(`HTTP error: ${response.status()} on ${response.url()}`);
+            }
+        });
+
+        console.time('page-load-complete');
         await page.goto('http://localhost:5150/itemsapi');
+        console.timeEnd('page-load-complete');
+
+        console.time('selector-lookup');
         await page.waitForSelector('.has-loaded', { timeout: 6000 });
+        console.timeEnd('selector-lookup');
     });
 
     afterAll(async () => {