chore: moving unit tests (#943)

Author: patrickhousley

.eslintrc.js

@@ -58,7 +58,7 @@ module.exports = {
       }
     },
     {
-      files: ['src/**/*.test.js', 'src/**/__mocks__/**/*', 'tests/components/**/*'],
+      files: ['src/**/__mocks__/**/*', 'tests/unit/**/*', 'tests/components/**/*'],
       env: {
         browser: true,
         node: true,

babel.config.js

@@ -14,7 +14,6 @@ module.exports = function (api, ...args) {
   process.env.RRWEB_VERSION = pkg.dependencies.rrweb
 
   const ignore = [
-    '**/*.test.js',
     '**/__mocks__/*.js'
   ]
   const presets = [

jest.config.js

@@ -15,8 +15,6 @@ module.exports = {
   coverageDirectory: 'coverage',
   collectCoverageFrom: [
     'src/**/*.js',
-    '!src/**/*.test.js',
-    '!src/**/*.component-test.js',
     '!src/index.js',
     '!src/cdn/**/*.js',
     '!src/features/*/index.js',
@@ -27,7 +25,7 @@ module.exports = {
     {
       ...commonConfig,
       displayName: 'unit',
-      testMatch: ['<rootDir>/src/**/?(*.)+(test).[tj]s?(x)']
+      testMatch: ['<rootDir>/tests/unit/**/?(*.)+(test).[tj]s?(x)']
     },
     {
       ...commonConfig,

src/loaders/browser-agent.test.js

@@ -1,4 +1,4 @@
-import { Aggregator } from './aggregator.js'
+import { Aggregator } from '../../../../src/common/aggregate/aggregator.js'
 
 let aggregator
 beforeEach(() => {

src/common/aggregate/aggregator.test.js renamed to tests/unit/common/aggregate/aggregator.test.js

@@ -1,4 +1,4 @@
-import { getModeledObject } from './configurable'
+import { getModeledObject } from '../../../../../src/common/config/state/configurable'
 
 test('if either params are not objects, function fails', () => {
   let result = getModeledObject(123, {})

src/common/config/state/configurable.test.js renamed to tests/unit/common/config/state/configurable.test.js

@@ -1,7 +1,7 @@
 let isValid, getInfo, setInfo
 beforeEach(async () => {
   jest.resetModules()
-  ;({ isValid, getInfo, setInfo } = await import('./info.js'))
+  ;({ isValid, getInfo, setInfo } = await import('../../../../../src/common/config/state/info.js'))
 })
 
 test('set/getInfo should throw on an invalid agent id', () => {

src/common/config/state/info.test.js renamed to tests/unit/common/config/state/info.test.js

@@ -1,7 +1,7 @@
 let getConfiguration, setConfiguration, getConfigurationValue
 beforeEach(async () => {
   jest.resetModules()
-  ;({ getConfiguration, setConfiguration, getConfigurationValue } = await import('./init.js'))
+  ;({ getConfiguration, setConfiguration, getConfigurationValue } = await import('../../../../../src/common/config/state/init.js'))
 })
 
 test('set/getConfiguration should throw on an invalid agent id', () => {

src/common/config/state/init.test.js renamed to tests/unit/common/config/state/init.test.js

@@ -1,7 +1,7 @@
 let getLoaderConfig, setLoaderConfig
 beforeEach(async () => {
   jest.resetModules()
-  ;({ getLoaderConfig, setLoaderConfig } = await import('./loader-config.js'))
+  ;({ getLoaderConfig, setLoaderConfig } = await import('../../../../../src/common/config/state/loader-config.js'))
 })
 
 test('set/getLoaderConfig should throw on an invalid agent id', () => {

src/common/config/state/loader-config.test.js renamed to tests/unit/common/config/state/loader-config.test.js

@@ -1,7 +1,7 @@
 let getRuntime, setRuntime
 beforeEach(async () => {
   jest.resetModules()
-  ;({ getRuntime, setRuntime } = await import('./runtime.js'))
+  ;({ getRuntime, setRuntime } = await import('../../../../../src/common/config/state/runtime.js'))
 })
 
 test('set/getRuntime should throw on an invalid agent id', () => {

src/common/config/state/runtime.test.js renamed to tests/unit/common/config/state/runtime.test.js

@@ -1,4 +1,4 @@
-import * as env from './env.cdn'
+import * as env from '../../../../src/common/constants/env.cdn'
 
 test('should default environment variables to CDN values', async () => {
   expect(env.VERSION).toMatch(/\d{1,3}\.\d{1,3}\.\d{1,3}/)

src/common/constants/env.cdn.test.js renamed to tests/unit/common/constants/env.cdn.test.js

@@ -1,4 +1,4 @@
-import * as env from './env.npm'
+import * as env from '../../../../src/common/constants/env.npm'
 
 test('should default environment variables to NPM values', () => {
   expect(env.VERSION).toMatch(/\d{1,3}\.\d{1,3}\.\d{1,3}/)

src/common/constants/env.npm.test.js renamed to tests/unit/common/constants/env.npm.test.js

@@ -1,4 +1,4 @@
-import * as env from './env'
+import * as env from '../../../../src/common/constants/env'
 
 test('should default environment variables to NPM values', () => {
   expect(env.VERSION).toMatch(/\d{1,3}\.\d{1,3}\.\d{1,3}/)

src/common/constants/env.test.js renamed to tests/unit/common/constants/env.test.js

@@ -28,7 +28,7 @@ test('should indicate agent is running in a browser scope', async () => {
     }
   }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   delete global.window
 
@@ -43,7 +43,7 @@ test('should indicate agent is running in a worker scope using global self varia
   const mockedGlobalSelf = global.self = new global.WorkerGlobalScope()
   mockedGlobalSelf.navigator = new global.WorkerNavigator()
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   delete global.WorkerGlobalScope
   delete global.WorkerNavigator
@@ -62,7 +62,7 @@ test('should indicate agent is running in a worker scope using global self varia
   Object.defineProperties(global.globalThis, Object.getOwnPropertyDescriptors(cachedGlobalThis))
   global.globalThis.navigator = new WorkerNavigator()
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   delete global.WorkerGlobalScope
   delete global.WorkerNavigator
@@ -88,7 +88,7 @@ test('should store the initial page location', async () => {
     }
   }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
   mockedWindow.location.href = faker.internet.url()
 
   delete global.window
@@ -108,7 +108,7 @@ test.each([
   global.navigator.userAgent = userAgent
   global.window = { navigator: global.navigator, document: true }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   expect(runtime.isiOS).toEqual(expected)
   delete global.window
@@ -131,7 +131,7 @@ test.each([
   global.navigator.userAgent = userAgent
   global.window = { navigator: global.navigator, document: true }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   delete global.SharedWorker
 
@@ -150,7 +150,7 @@ test.each([
   global.navigator.userAgent = userAgent
   global.window = { navigator: global.navigator, document: true }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   expect(runtime.ffVersion).toEqual(expected)
   delete global.window
@@ -160,7 +160,7 @@ test('should set supportsSendBeacon to false', async () => {
   // Ensure we don't have a sendBeacon function
   delete global.navigator.sendBeacon
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   expect(runtime.supportsSendBeacon).toEqual(false)
 })
@@ -169,7 +169,7 @@ test('should set supportsSendBeacon to true', async () => {
   global.navigator.sendBeacon = jest.fn()
   global.window = { navigator: global.navigator, document: true }
 
-  const runtime = await import('./runtime')
+  const runtime = await import('../../../../src/common/constants/runtime')
 
   expect(runtime.supportsSendBeacon).toEqual(true)
   delete global.window

src/common/constants/runtime.test.js renamed to tests/unit/common/constants/runtime.test.js

@@ -1,10 +1,10 @@
 jest.enableAutomock()
-jest.unmock('./deny-list')
+jest.unmock('../../../../src/common/deny-list/deny-list')
 
 let denyListModule
 
 beforeEach(async () => {
-  denyListModule = await import('./deny-list')
+  denyListModule = await import('../../../../src/common/deny-list/deny-list')
 })
 
 afterEach(() => {

src/common/deny-list/deny-list.test.js renamed to tests/unit/common/deny-list/deny-list.test.js

@@ -1,10 +1,10 @@
-import { dispatchGlobalEvent } from './global-event'
+import { dispatchGlobalEvent } from '../../../../src/common/dispatch/global-event'
 
 let globalScope, handler
 
 describe('dispatchGlobalEvent', () => {
   beforeEach(async () => {
-    globalScope = (await import('../../common/constants/runtime')).globalScope
+    globalScope = (await import('../../../../src/common/constants/runtime')).globalScope
   })
   afterEach(() => {
     jest.resetAllMocks()

src/common/dispatch/global-event.test.js renamed to tests/unit/common/dispatch/global-event.test.js

@@ -1,4 +1,4 @@
-import { isValidSelector } from './query-selector'
+import { isValidSelector } from '../../../../src/common/dom/query-selector'
 describe('query selector tests', () => {
   test('handles nullish values', () => {
     expect(isValidSelector(null)).toEqual(false)

src/common/dom/query-selector.test.js renamed to tests/unit/common/dom/query-selector.test.js

@@ -1,8 +1,8 @@
 let registerDrain, drain, ee
 beforeEach(async () => {
   jest.resetModules()
-  ;({ registerDrain, drain } = await import('./drain'))
-  ;({ ee } = await import('../event-emitter/contextual-ee'))
+  ;({ registerDrain, drain } = await import('../../../../src/common/drain/drain'))
+  ;({ ee } = await import('../../../../src/common/event-emitter/contextual-ee'))
 })
 
 test('can register a feat and drain it', () => {

src/common/drain/drain.test.js renamed to tests/unit/common/drain/drain.test.js

@@ -1,6 +1,6 @@
 import { faker } from '@faker-js/faker'
 
-jest.mock('./contextual-ee', () => ({
+jest.mock('../../../../src/common/event-emitter/contextual-ee', () => ({
   __esModule: true,
   ee: {
     buffer: jest.fn(),
@@ -18,8 +18,8 @@ afterEach(() => {
 })
 
 test('it should create and use a default event-emitter', async () => {
-  const { ee } = await import('./contextual-ee')
-  const { handle } = await import('./handle')
+  const { ee } = await import('../../../../src/common/event-emitter/contextual-ee')
+  const { handle } = await import('../../../../src/common/event-emitter/handle')
 
   const handleEE = jest.mocked(ee.get).mock.results[0].value
   const eventType = faker.string.uuid()
@@ -34,8 +34,8 @@ test('it should create and use a default event-emitter', async () => {
 })
 
 test('it should use the provided scoped event-emitter', async () => {
-  const { ee } = await import('./contextual-ee')
-  const { handle } = await import('./handle')
+  const { ee } = await import('../../../../src/common/event-emitter/contextual-ee')
+  const { handle } = await import('../../../../src/common/event-emitter/handle')
   const scopedEE = {
     buffer: jest.fn(),
     emit: jest.fn()

src/common/event-emitter/handle.test.js renamed to tests/unit/common/event-emitter/handle.test.js

@@ -1,6 +1,6 @@
 import { faker } from '@faker-js/faker'
 
-jest.mock('./handle', () => ({
+jest.mock('../../../../src/common/event-emitter/handle', () => ({
   __esModule: true,
   handleEE: {}
 }))
@@ -11,8 +11,8 @@ afterEach(() => {
 })
 
 test('should default group to "feature"', async () => {
-  const { handleEE } = await import('./handle')
-  const { registerHandler } = await import('./register-handler')
+  const { handleEE } = await import('../../../../src/common/event-emitter/handle')
+  const { registerHandler } = await import('../../../../src/common/event-emitter/register-handler')
 
   const eventType = faker.string.uuid()
   const eventHandler = jest.fn()
@@ -27,8 +27,8 @@ test('should default group to "feature"', async () => {
 })
 
 test('should use the provided group', async () => {
-  const { handleEE } = await import('./handle')
-  const { registerHandler } = await import('./register-handler')
+  const { handleEE } = await import('../../../../src/common/event-emitter/handle')
+  const { registerHandler } = await import('../../../../src/common/event-emitter/register-handler')
 
   const eventType = faker.string.uuid()
   const eventGroup = faker.string.uuid()
@@ -44,7 +44,7 @@ test('should use the provided group', async () => {
 })
 
 test('should use the provided event-emitter', async () => {
-  const { registerHandler } = await import('./register-handler')
+  const { registerHandler } = await import('../../../../src/common/event-emitter/register-handler')
 
   const scopedEE = {}
   const eventType = faker.string.uuid()

src/common/event-emitter/register-handler.test.js renamed to tests/unit/common/event-emitter/register-handler.test.js

@@ -1,13 +1,13 @@
 import { faker } from '@faker-js/faker'
 
-import * as submitData from '../util/submit-data'
-import { subscribeToEOL } from '../unload/eol'
-import { Harvest } from './harvest'
+import * as submitData from '../../../../src/common/util/submit-data'
+import { subscribeToEOL } from '../../../../src/common/unload/eol'
+import { Harvest } from '../../../../src/common/harvest/harvest'
 
-import { HarvestScheduler } from './harvest-scheduler'
+import { HarvestScheduler } from '../../../../src/common/harvest/harvest-scheduler'
 
 jest.enableAutomock()
-jest.unmock('./harvest-scheduler')
+jest.unmock('../../../../src/common/harvest/harvest-scheduler')
 jest.useFakeTimers()
 
 let harvestSchedulerInstance

src/common/harvest/harvest-scheduler.test.js renamed to tests/unit/common/harvest/harvest-scheduler.test.js

@@ -1,15 +1,15 @@
 import { faker } from '@faker-js/faker'
 
-import * as encodeModule from '../url/encode'
-import * as submitDataModule from '../util/submit-data'
-import * as configModule from '../config/config'
-import { warn } from '../util/console'
-import { applyFnToProps } from '../util/traverse'
+import * as encodeModule from '../../../../src/common/url/encode'
+import * as submitDataModule from '../../../../src/common/util/submit-data'
+import * as configModule from '../../../../src/common/config/config'
+import { warn } from '../../../../src/common/util/console'
+import { applyFnToProps } from '../../../../src/common/util/traverse'
 
-import { Harvest } from './harvest'
+import { Harvest } from '../../../../src/common/harvest/harvest'
 
 jest.enableAutomock()
-jest.unmock('./harvest')
+jest.unmock('../../../../src/common/harvest/harvest')
 let harvestInstance
 
 beforeEach(() => {

src/common/harvest/harvest.test.js renamed to tests/unit/common/harvest/harvest.test.js

@@ -1,7 +1,7 @@
 let idFn
 
 beforeEach(async () => {
-  idFn = (await import('./id')).id
+  idFn = (await import('../../../../src/common/ids/id')).id
 })
 
 afterEach(() => {