swetrix 4.0.0 → 4.2.0

package/tests/utils.test.ts

@@ -1,35 +1,32 @@
 import * as utils from '../src/utils'
+import { setLocation } from './testUtils'
 
 describe('Utility Functions', () => {
   beforeEach(() => {
-    Object.defineProperty(window, 'location', {
-      value: {
-        hostname: 'example.com',
-        pathname: '/test-page',
-        hash: '',
-        search: '',
-      },
-      writable: true,
-    })
+    setLocation({ hostname: 'example.com', pathname: '/test-page' })
 
     Object.defineProperty(document, 'referrer', {
       value: 'https://google.com',
       writable: true,
+      configurable: true,
     })
 
     Object.defineProperty(navigator, 'language', {
       value: 'en-US',
       writable: true,
+      configurable: true,
     })
 
     Object.defineProperty(navigator, 'languages', {
       value: ['en-US', 'en'],
       writable: true,
+      configurable: true,
     })
 
     Object.defineProperty(navigator, 'webdriver', {
       value: false,
       writable: true,
+      configurable: true,
     })
   })
 
@@ -38,50 +35,28 @@ describe('Utility Functions', () => {
   })
 
   test('isLocalhost should detect localhost', () => {
-    // Arrange
-    Object.defineProperty(window, 'location', {
-      value: {
-        hostname: 'localhost',
-      },
-      writable: true,
-    })
-
-    // Act & Assert
+    setLocation({ hostname: 'localhost' })
     expect(utils.isLocalhost()).toBe(true)
 
-    // Test 127.0.0.1
-    Object.defineProperty(window, 'location', {
-      value: {
-        hostname: '127.0.0.1',
-      },
-      writable: true,
-    })
+    setLocation({ hostname: '127.0.0.1' })
     expect(utils.isLocalhost()).toBe(true)
 
-    // Test non-localhost
-    Object.defineProperty(window, 'location', {
-      value: {
-        hostname: 'example.com',
-      },
-      writable: true,
-    })
+    setLocation({ hostname: 'example.com' })
     expect(utils.isLocalhost()).toBe(false)
   })
 
   test('isAutomated should detect webdriver', () => {
-    // Arrange
     Object.defineProperty(navigator, 'webdriver', {
       value: true,
       writable: true,
+      configurable: true,
     })
-
-    // Act & Assert
     expect(utils.isAutomated()).toBe(true)
 
-    // Test non-automated
     Object.defineProperty(navigator, 'webdriver', {
       value: false,
       writable: true,
+      configurable: true,
     })
     expect(utils.isAutomated()).toBe(false)
   })
@@ -89,129 +64,77 @@ describe('Utility Functions', () => {
   test('getLocale should return the browser language', () => {
     expect(utils.getLocale()).toBe('en-US')
 
-    // Test with navigator.languages undefined
     const originalLanguages = navigator.languages
-    // Instead of deleting, set to undefined
     Object.defineProperty(navigator, 'languages', {
       value: undefined,
       writable: true,
+      configurable: true,
     })
-    expect(utils.getLocale()).toBe('en-US') // Should use navigator.language as fallback
+    expect(utils.getLocale()).toBe('en-US')
 
-    // Restore the original value
     Object.defineProperty(navigator, 'languages', {
       value: originalLanguages,
       writable: true,
+      configurable: true,
     })
   })
 
   test('getReferrer should return the document referrer', () => {
     expect(utils.getReferrer()).toBe('https://google.com')
 
-    // Test with empty referrer
     Object.defineProperty(document, 'referrer', {
       value: '',
       writable: true,
+      configurable: true,
     })
     expect(utils.getReferrer()).toBeUndefined()
   })
 
-  test('getPath should handle different URL formats', () => {
-    // Arrange
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/test-page',
-        hash: '',
-        search: '',
-      },
-      writable: true,
-    })
+  test('getQueryString should return the URL query string without the leading ?', () => {
+    setLocation({ pathname: '/landing', search: '?fbclid=AbCdEf123&utm_source=newsletter' })
+    expect(utils.getQueryString()).toBe('fbclid=AbCdEf123&utm_source=newsletter')
 
-    // Act & Assert - basic path
+    setLocation({ pathname: '/landing' })
+    expect(utils.getQueryString()).toBeUndefined()
+
+    // Hash-routed SPAs sometimes carry the query string after the `#`.
+    setLocation({ pathname: '/', hash: '#/landing?gclid=xyz' })
+    expect(utils.getQueryString()).toBe('gclid=xyz')
+  })
+
+  test('getPath should handle different URL formats', () => {
+    setLocation({ pathname: '/test-page' })
     expect(utils.getPath({})).toBe('/test-page')
 
-    // Test with hash
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/test-page',
-        hash: '#section1',
-        search: '',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/test-page', hash: '#section1' })
     expect(utils.getPath({ hash: true })).toBe('/test-page#section1')
 
-    // Test with search
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/test-page',
-        hash: '',
-        search: '?param=value',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/test-page', search: '?param=value' })
     expect(utils.getPath({ search: true })).toBe('/test-page?param=value')
 
-    // Test with both hash and search
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/test-page',
-        hash: '#section1',
-        search: '?param=value',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/test-page', hash: '#section1', search: '?param=value' })
     expect(utils.getPath({ hash: true, search: true })).toBe('/test-page#section1?param=value')
 
-    // Test with search in hash
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/test-page',
-        hash: '#section1?param=value',
-        search: '',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/test-page', hash: '#section1?param=value' })
     expect(utils.getPath({ hash: true, search: true })).toBe('/test-page#section1?param=value')
   })
 
   test('getUTM* functions should extract UTM parameters', () => {
-    // Arrange
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/landing',
-        hash: '',
-        search: '?utm_source=google&utm_medium=cpc&utm_campaign=summer&utm_term=analytics&utm_content=ad1',
-      },
-      writable: true,
+    setLocation({
+      pathname: '/landing',
+      search: '?utm_source=google&utm_medium=cpc&utm_campaign=summer&utm_term=analytics&utm_content=ad1',
     })
 
-    // Act & Assert
     expect(utils.getUTMSource()).toBe('google')
     expect(utils.getUTMMedium()).toBe('cpc')
     expect(utils.getUTMCampaign()).toBe('summer')
     expect(utils.getUTMTerm()).toBe('analytics')
     expect(utils.getUTMContent()).toBe('ad1')
 
-    // Test with 'ref' and 'source' parameters as well
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/landing',
-        hash: '',
-        search: '?ref=twitter',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/landing', search: '?ref=twitter' })
     expect(utils.getUTMSource()).toBe('twitter')
 
-    Object.defineProperty(window, 'location', {
-      value: {
-        pathname: '/landing',
-        hash: '',
-        search: '?source=newsletter',
-      },
-      writable: true,
-    })
+    setLocation({ pathname: '/landing', search: '?source=newsletter' })
     expect(utils.getUTMSource()).toBe('newsletter')
   })
 })

package/.github/funding.yml

@@ -1,2 +0,0 @@
-github: swetrix
-ko_fi: andriir

package/.github/workflows/test.yml

@@ -1,32 +0,0 @@
-name: Tests
-
-on:
-  push:
-    branches: [main]
-  pull_request:
-    branches: [main]
-    types: [opened, labeled, synchronize, ready_for_review]
-
-jobs:
-  test:
-    name: 🧪 Unit Tests
-    runs-on: ubuntu-latest
-
-    strategy:
-      matrix:
-        node-version: [22.x]
-
-    steps:
-      - uses: actions/checkout@v4
-
-      - name: Setup Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v4
-        with:
-          node-version: ${{ matrix.node-version }}
-          cache: 'npm'
-
-      - name: Install dependencies
-        run: npm ci
-
-      - name: Run tests
-        run: npm test