Merge pull request #5396 from blink1073/fix-browser-test

Fix browser test

buildutils/src/ensure-package.ts

@@ -133,8 +133,9 @@ export async function ensurePackage(
     if (!fs.existsSync(target)) {
       return;
     }
-    references[name] = path.relative(path.join(pkgPath, 'src'), locals[name]);
-    references[name] = path.join(references[name], 'src');
+    let ref = path.relative(path.join(pkgPath, 'src'), locals[name]);
+    ref = path.join(ref, 'src');
+    references[name] = ref.split(path.sep).join('/');
   });
   if (
     data.name.indexOf('test-') === -1 &&

dev_mode/index.js

@@ -162,12 +162,12 @@ function main() {
 
   // Handle a browser test.
   var browserTest = PageConfig.getOption('browserTest');
-  var el = document.createElement('div');
-  el.id = 'browserTest';
-  document.body.appendChild(el);
-  el.textContent = '[]';
-
   if (browserTest.toLowerCase() === 'true') {
+    var el = document.createElement('div');
+    el.id = 'browserTest';
+    document.body.appendChild(el);
+    el.textContent = '[]';
+    el.style.display = 'none';
     var errors = [];
     var reported = false;
     var timeout = 25000;

jupyterlab/staging/index.js

@@ -162,12 +162,12 @@ function main() {
 
   // Handle a browser test.
   var browserTest = PageConfig.getOption('browserTest');
-  var el = document.createElement('div');
-  el.id = 'browserTest';
-  document.body.appendChild(el);
-  el.textContent = '[]';
-
   if (browserTest.toLowerCase() === 'true') {
+    var el = document.createElement('div');
+    el.id = 'browserTest';
+    document.body.appendChild(el);
+    el.textContent = '[]';
+    el.style.display = 'none';
     var errors = [];
     var reported = false;
     var timeout = 25000;