server-lib bugfix: need scopingRoot

Author: samouri

src/test/htmlelement/querySelectorAll.test.ts

@@ -39,6 +39,23 @@ test('test Element.querySelectorAll on id selectors', (t) => {
   t.deepEqual(document.querySelectorAll(`#${DIV_ID}`), [div]);
 });
 
+// TODO: create better solution for parameterizing server tests.
+test.serial.only('server=true: test Element.querySelectorAll on id selectors', (t) => {
+  process.env.SERVER = 'true';
+  try {
+    const document = createTestingDocument();
+    const testEl = document.createElement('test-el');
+    const id = 'TEST_EL_ID';
+    testEl.setAttribute('id', id);
+    document.appendChild(testEl);
+
+    const result = document.querySelectorAll(`#${id}`);
+    t.deepEqual(result, [testEl]);
+  } finally {
+    delete process.env.SERVER;
+  }
+});
+
 test('test Element.querySelectorAll on class selectors', (t) => {
   const { document, div } = t.context;
 

src/worker-thread/dom/Node.ts

@@ -52,11 +52,11 @@ export abstract class Node {
     this.nodeType = nodeType;
     this.nodeName = nodeName;
     this.ownerDocument = ownerDocument || this;
+    this[TransferrableKeys.scopingRoot] = this;
     if (process.env.SERVER) {
       return;
     }
 
-    this[TransferrableKeys.scopingRoot] = this;
     this[TransferrableKeys.index] = overrideIndex ? storeOverrideNodeMapping(this, overrideIndex) : storeNodeMapping(this);
     this[TransferrableKeys.transferredFormat] = [this[TransferrableKeys.index]];
   }