chore: fix unit test

Author: zhoushaw

.github/workflows/build-and-test.yml

@@ -44,7 +44,7 @@ jobs:
         run: npx nx affected -t lint --parallel=7 --exclude='*,!tag:type:pkg'
 
       - name: Run Affected Test
-        run: npx nx affected -t test --parallel=3 --exclude='*,!tag:type:pkg'
+        run: npx nx affected -t test --parallel=3 --exclude='*,!tag:type:pkg' --skip-cache
 
       # - name: E2E Test for Next.js Dev
       #   run: |

packages/sdk/__tests__/dom.spec.ts

@@ -8,7 +8,7 @@ describe('createScript', () => {
   it('should create a new script element if one does not exist', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    const { script, needAttach } = createScript(url, cb);
+    const { script, needAttach } = createScript({ url, cb });
 
     expect(script.tagName).toBe('SCRIPT');
     expect(script.src).toBe(url);
@@ -19,7 +19,7 @@ describe('createScript', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
     document.body.innerHTML = `<script src="${url}"></script>`;
-    const { script, needAttach } = createScript(url, cb);
+    const { script, needAttach } = createScript({ url, cb });
 
     expect(script.tagName).toBe('SCRIPT');
     expect(script.src).toBe(url);
@@ -30,7 +30,7 @@ describe('createScript', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
     const attrs = { async: true, 'data-test': 'test' };
-    const { script } = createScript(url, cb, attrs);
+    const { script } = createScript({ url, cb, attrs });
 
     expect(script.async).toBe(true);
     expect(script.getAttribute('data-test')).toBe('test');
@@ -39,7 +39,7 @@ describe('createScript', () => {
   it('should call the callback when the script loads', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    const { script, needAttach } = createScript(url, cb);
+    const { script, needAttach } = createScript({ url, cb });
 
     if (needAttach) {
       document.body.appendChild(script);
@@ -52,7 +52,12 @@ describe('createScript', () => {
   it('should call the callback when the script times out', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    createScript(url, cb, {}, () => ({ timeout: 100 }));
+    createScript({
+      url,
+      cb,
+      attrs: {},
+      createScriptHook: () => ({ timeout: 100 }),
+    });
 
     setTimeout(() => {
       expect(cb).toHaveBeenCalled();
@@ -71,7 +76,7 @@ describe('createScript', () => {
     it('should use the default timeout of 20000ms if no timeout is specified', () => {
       const url = 'https://example.com/script.js';
       const cb = jest.fn();
-      const { script } = createScript(url, cb);
+      const { script } = createScript({ url, cb });
 
       expect(setTimeout).toHaveBeenCalledWith(expect.any(Function), 20000);
     });
@@ -80,7 +85,12 @@ describe('createScript', () => {
       const url = 'https://example.com/script.js';
       const cb = jest.fn();
       const customTimeout = 5000;
-      createScript(url, cb, {}, () => ({ timeout: customTimeout }));
+      createScript({
+        url,
+        cb,
+        attrs: {},
+        createScriptHook: () => ({ timeout: customTimeout }),
+      });
 
       expect(setTimeout).toHaveBeenCalledWith(
         expect.any(Function),
@@ -91,7 +101,7 @@ describe('createScript', () => {
     it('should clear the timeout when the script loads successfully', () => {
       const url = 'https://example.com/script.js';
       const cb = jest.fn();
-      const { script, needAttach } = createScript(url, cb);
+      const { script, needAttach } = createScript({ url, cb });
 
       if (needAttach) {
         document.body.appendChild(script);
@@ -104,7 +114,7 @@ describe('createScript', () => {
     it('should clear the timeout when the script fails to load', () => {
       const url = 'https://example.com/script.js';
       const cb = jest.fn();
-      const { script, needAttach } = createScript(url, cb);
+      const { script, needAttach } = createScript({ url, cb });
 
       if (needAttach) {
         document.body.appendChild(script);
@@ -124,7 +134,11 @@ describe('createLink', () => {
   it('should create a new link element if one does not exist', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    const { link, needAttach } = createLink(url, cb, { as: 'script' });
+    const { link, needAttach } = createLink({
+      url,
+      cb,
+      attrs: { as: 'script' },
+    });
 
     expect(link.tagName).toBe('LINK');
     expect(link.href).toBe(url);
@@ -136,9 +150,13 @@ describe('createLink', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
     document.head.innerHTML = `<link href="${url}" rel="preload" as="script">`;
-    const { link, needAttach } = createLink(url, cb, {
-      rel: 'preload',
-      as: 'script',
+    const { link, needAttach } = createLink({
+      url,
+      cb,
+      attrs: {
+        rel: 'preload',
+        as: 'script',
+      },
     });
 
     expect(link.tagName).toBe('LINK');
@@ -150,7 +168,7 @@ describe('createLink', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
     const attrs = { rel: 'preload', as: 'script', 'data-test': 'test' };
-    const { link } = createLink(url, cb, attrs);
+    const { link } = createLink({ url, cb, attrs });
 
     expect(link.rel).toBe('preload');
     expect(link.getAttribute('as')).toBe('script');
@@ -160,7 +178,11 @@ describe('createLink', () => {
   it('should call the callback when the link loads', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    const { link, needAttach } = createLink(url, cb, { as: 'script' });
+    const { link, needAttach } = createLink({
+      url,
+      cb,
+      attrs: { as: 'script' },
+    });
 
     if (needAttach) {
       document.head.appendChild(link);
@@ -173,7 +195,11 @@ describe('createLink', () => {
   it('should call the callback when the link fails to load', () => {
     const url = 'https://example.com/script.js';
     const cb = jest.fn();
-    const { link, needAttach } = createLink(url, cb, { as: 'script' });
+    const { link, needAttach } = createLink({
+      url,
+      cb,
+      attrs: { as: 'script' },
+    });
 
     if (needAttach) {
       document.head.appendChild(link);
@@ -190,7 +216,12 @@ describe('createLink', () => {
     customLink.href = url;
     customLink.rel = 'preload';
     customLink.setAttribute('as', 'script');
-    const { link } = createLink(url, cb, {}, () => customLink);
+    const { link } = createLink({
+      url,
+      cb,
+      attrs: {},
+      createLinkHook: () => customLink,
+    });
 
     expect(link).toBe(customLink);
   });