Skip to content

Commit 4d48415

Browse files
committed
test: fix e2e tests, removing some dead code
1 parent 3e04dc6 commit 4d48415

File tree

13 files changed

+69
-96
lines changed

13 files changed

+69
-96
lines changed

e2e/capture.spec.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { test, expect } from '@playwright/test'
2-
import { NEXTJS_API_PORT, NEXTJS_PORT, NUXT_PORT, SVELTE_PORT, fetchCaptures, getBaseUrl, getLatestCapture, waitMs, getApiBase } from './util'
2+
import { NEXTJS_API_PORT, NEXTJS_PORT, NUXT_PORT, SVELTE_PORT, fetchCaptures, getBaseUrl, getLatestCapture, waitMs, getApiBase, getNextConsoleLogs } from './util'
33
import { join } from 'path'
44
import { existsSync, rmSync } from 'fs'
55

@@ -70,9 +70,8 @@ test('should capture front-end bug in nextjs example', async ({ page }) => {
7070
await page.click('text=Submit')
7171

7272
// Wait for some console.log
73-
const consolePromise = await page.waitForEvent('console');
74-
const consoleLogsArgs = await consolePromise.args()
75-
expect(await consoleLogsArgs[0]?.jsonValue()).toContain(`[🐛 post] ${getApiBase()}/api/v1/captures`)
73+
const consoleLogs = await getNextConsoleLogs(page)
74+
expect(consoleLogs).toContain(`[POST] ${getApiBase()}/api/v1/captures`)
7675

7776
await waitMs(3_000);
7877

e2e/util.ts

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import { Page } from "@playwright/test";
2+
13
export const getApiBase = () => 'https://www.useflytrap.com';
24
// Capturing
35
export const NEXTJS_API_PORT = 3000;
@@ -25,3 +27,17 @@ export async function fetchCaptures(secretApiKey: string): Promise<any[]> {
2527
export function getLatestCapture(captures: any[], testStartTime: Date): any | undefined {
2628
return captures.find((capture) => new Date(capture.createdAt).getTime() > testStartTime.getTime());
2729
}
30+
31+
export async function getNextConsoleLogs(page: Page) {
32+
const consolePromise = await page.waitForEvent('console');
33+
const consoleLogsArgs = await consolePromise.args()
34+
// consoleLogsArgs.length;
35+
36+
const consoleLogJsonValues = await Promise.all(
37+
consoleLogsArgs.map(async (c) => await c.jsonValue())
38+
)
39+
40+
return consoleLogJsonValues.join('\n');
41+
42+
// expect(await consoleLogsArgs[0]?.jsonValue()).toContain(`[🐛 post] ${getApiBase()}/api/v1/captures`)
43+
}

examples/with-express/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,5 @@ export default defineFlytrapConfig({
88
privateKey:
99
'sk_MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQChz7gmNTx5Tfh/dwTEKXvm4KMZwuRvAbrMS4jZkJWkC8ham0xnIaK84rUEzTTg4Fn+T4y8Z1nqtXjcCqAkugM6g80bOJhfTRK3W+fiHP6l56dDugyYHXI+rHu0YtK4we3BYStKEEo3Z0kEyhKZBrK3lt/RJsxuUFj41EslOcaAw3sDOQe3guS3TqWyXWbGs5V/nhOBI2EZ8p5ET/a0WrnxzCydUoc806nXP8lHlnVg54cPGAVPayE3O135HfpP5VI0A6rjoNtfd4Raeuc3hnxucC9C49r/hsqgebRlz6Uge2IfxxjdEx5ld3cdAqKUdHE0yYqEVzmttPk0prERtAI/AgMBAAECggEABDVbVD8RNhR1kJcfXBy6GLyh8VMUzuBMI/EY+5ObmLsnuFeOqqYAzgnVbG0L4YdkP82A2wuTFpwmqw5psNrXp6dy57DWoa/m+zbcNpDLfLQkMQo/NpuZc5DX3M6vLVf534wv7Iq063D3wLSivDEk58p4Qr/QFFRpo/sdgzfX9lRUksCC9xLDaCEyDNvI2v4jas4v0edSoka0MQiXnaYlyVA9NlDIcjgHJNiYYNqYhktznDe63BSWN7nE9Xghbi5Ww8zxOKvK+mrOKiGfHEnRKcKSmF9bUlraCBTsVaTKGHmcMNi+SabgBkc9GurTDoHaS2GNblCP6DBJjKOx1nGsAQKBgQDiQKI/UQMw8cPXr9yzSQuEdZnPXgStM85llhv3Bcf1sK7i5C/JA9o61M9RIrai/7qbWruOpBzY28KQdzVvbyi5aiUK5Jb9AEv+hBjgj7DZKDlJUZ0fO1s4Io8F2D/zTAz19Lp/kLaeL12ZNog8xa1xyziKW9RsMcIjuNRbbriWPwKBgQC3FhUPgL92DzID2FLvmb47s6QzTWCOQuiZvjscQcuy1ozgNL5NIPKe3TsxaXCy7lu4KejOG86ALmqoOW2NKLZyeA9IkXjph4daJDa8GXbzKgSuYVNlOYSnAL1WOxaHqV7tkFzbBb9Mv3xozaHTn1SoRERbRNKruoyEsS2A8iGUAQKBgQCfwSmO8aICNkOlkUyR31rdjbgsPuOodLMg8cwPMSrO5ZGnLZ48HpdCAS/dx1gUYeOefIOApYqk6Q9JuJKne2LTAQrk1SFzxXjjXty9+1fRDeDdT15VRjETwvsgEmCdZy6joMjq+ACqUyGQ5HtwBdiBSBBFSbAFMWE2nfR8FwMJOwKBgGZJ8RRLQyzrim75gkSZKd5jHD8JYQO6QAf9bzr3S54sHmL5NMUUvlihk3ukxDFD2HyI9aeJg5K73B0GkL1H3L8RcsCBfgNDFAz5pJbsxzdN53WhrJ0mSIWhdSbjNlAvAXlKqYY4rMttRE2tbVc1cm/cTDCNwAHv5KgR8TFUlgwBAoGAUlG+5yAVPkWtP90wT7HBhVeRFl5ZGG9PI79Sdc7u/W0ksjJS+fhI5qbrHDMW4Son2/TM4qNLLyY3nSsdV8LgRAcbYBUXUvtW9dHi68ofND4Yqcg79cxLISg2Div5ldakslYIRRIsEfBALCJTKTxSDWAyXD8AaRXiVg5mIcYIlAw=',
1010
mode: 'capture',
11-
logging: ['api-calls', 'capture', 'storage']
11+
logging: ['error', 'api-calls', 'capture', 'storage']
1212
})

examples/with-nextjs-api/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@ export default defineFlytrapConfig({
99
'sk_MIIEvgIBADANBgkqhkiG9w0BAQEFAASCBKgwggSkAgEAAoIBAQCe3rNiZqbiQz0wtvcIoeDikyaKUyYA5hMi95Vqwl1X73DxfVFKI+VbBs0SNsZ9gET6A2Az8lcwyvpOGoy+Oig5FSlsQV/C9+U3BFvBTnGLZFDuyVMCF5CGCHRAOlyS5MVOfszSUnz0/cCH2vcEEmQ+CIHiY7/zfZhkaVhkWZim5Les3Kg9OhbZVpvDvue/jPVQz3Z9dboNESMuN5MHyG0m7yp+GMpajMtvB4rsvKw6gZL3xzS8pC+Rq1rQh+PtU2tPzZT8+14Vab/Yots9JMw7bLVQQksuykEG+16BXBHPDNh066XGI81pDmYpfrt/8hk6+c7Omc9Fq+vYYyhEQGBrAgMBAAECggEAPqP/pfZkqcUqTNlviQNO6PdtVFAZpXQSKpDlZK1TYbdC2Kz5Ttrddak2VY6sb9KSH+QR3BmbIF2AGSilwuGqd2ttJEaYjAbzZWB+DzevVG9O7AAMj0AcV4Tg1WxNkQvL8jnT57/nji+5aLfA+uWHieAP2mRLTKYCXuEcnaCoLHFWIxGiQ9te/J11V+OxxEPMH3auBdx3lMQ44njJNhnEGskz/5jYc+ilRMKVFkyDer/smv0BpykdNpxXExpoqhgIOZ9N4Io5QJTaBxv5b0WZbUgoxGahH5Gzs+FPoYvb/jxjjVka50HRHNtePwmA4yAamdVJsoB4P6RDOnAV4uKXUQKBgQDK78UQ73f8jjQ82vienXSHLCXcuAr1ov2J3HA4MDWwzDjnypvr2kIHnE1p7dwJINeu4SYhctDpqPiA8DYg+2FLbUc8aU3BF2IkRiQR9aC0+j1bEjNI3fOKc18BtK0WOBzVeysKoinRfT7zQsLmxlgJjd6x1YSI1bPKxAcELpZFkQKBgQDIaS7Ho8YWO1aUpqKxiGYRJ3kbPAhm5ADKApS/mgRG7AsmdBhJ3rL77Az3mca6TLcJ9WAfxj+rxlX/HfDOAE8T/rl9k2omLOxSrCi9Oud0kvnJBquF/yhnLTzGf3vfXmeii02sWajzng5WRW57vVyPzZoLn4IJL8rERPmeSCzYOwKBgQChElgWzgstMO1zU9W7h3wDAd53ZriEKL8WGhoT7ufekOmlyGGjfJOmGMA1EK7xq9ilHF04dTouC4haRs6ftQaPaIf/AhHN7bJe2jEs9Al/O5NRYovEDS6NOG8zGmW7WgRAyUbrZLTOapt5Vwb9RnZ/9tPH0JImTYArsxVik1uAkQKBgQC8UEF5rAU87KLtaSrk4MLviYS9t4o5jDLFnulApKsyLShJCLut5cmO+H4yxlc6xU0U2XFqSfGYKfSylOKp3xZvNmH7gmIy3vFsuYJKVSgZnOWDJHZVoa3ITXSXHEE8YUvGK1lgApvD1peoLvOi6AsqwMnn9AjPMw43oIxhU3Z38wKBgE8+bku3tQvnwSSh1bwf0GEoa/gwlVXmzpEjntr66ysX+U9ysTJ/BH2DdjjBmvWbTaUpN+SwZq/FhBmIyyTFJ2LiuHAjuuZ2gWXVkl4+oOnhmHVW7xkEsN9BEpFAiGSNojkK4UH0V9pdV1ButZKCYjLz1aSPl8uBMyDFEvPZb9fo',
1010
mode: 'capture',
1111
packageIgnores: ['next/font'],
12-
logging: ['api-calls', 'capture', 'storage']
12+
logging: ['error', 'api-calls', 'capture', 'storage']
1313
})

examples/with-nextjs/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,6 @@ export default defineFlytrapConfig({
99
'sk_MIIEvAIBADANBgkqhkiG9w0BAQEFAASCBKYwggSiAgEAAoIBAQCU5XslK9na97TDCDIgImB948aBBQcEflrFKDQ0lsRW/g6RnW/V+j+r6tQq/p+EA98hNfilfIm5M3WneoAAxniTsAKrYp4OOk5Arux0QY7EzwzDGqSQiwvz+r9+mWyKS9TNmRTpODLoSK1gPZI2QyGSvWMZztjgUvDv+4JgO1y9XtnnCHKCDgrnke1TwgxTBs7OXAy30h+UwbrPLPgLgFu680rXAZ3hkSymkqAO2CcYiEQZjnmngb9WTedv36oh9IumTySPZ+xc+zXfN4jk1x6zFcZ0wtaPRIpZcIn1Qixgmx5UFNerM0ac1RkRCv9DzzWOybloXP+4waqTQZMIIhI9AgMBAAECggEABsLk9KspGWhmLPUSUaM9/nl5aiJwTyPtu0Y4GKs//zNKNhKQIYdbnNLtD4bAd541Y6yvvdQJ7P8JOLG3XlGk6A/2DxJLWxoKs6U914mikg1U5wcuseGeiVW5nGYbJ6yJ3Uw2RuDmbuXd+aDSIuSpgaBX/7QE5SrfBgbFxNjtPSwftUJ/gS5dQ4HBdDnUjzzHJv3KbiS+mxLCtcdz/OsjP77G2qT++fooB8dGhEVcbqMHoNtDgde45nFXDU9kAbfRcNsMXfq8iRKpKccKCCV9MfN18csZTOcAqg9n2ZCzRje6mb8HVwGufJQ4E7Iw2T8czNttRNC066/Hn1BBvJi0AQKBgQDPiSUDiZ3k/g88NRajwuOCY24yZytRqioLA6n1gLc+PWfE7iZk3fRHUuzceU3nC1MGtR0xXsA1INkN97e8kFkjM5XEbLiggP2pRdK72YAaNybrYnIE62r8ynoszC2UaQgzBAuNsS5XgsUeRmG0E7WP8dQjShT/tIEhmOmgVwgu9QKBgQC3qsYe03XJiUAzW6JlB8PH9uQYmas2iMEq7mwl83aFKAmUPSuZck+l2TZtQfAKB7+2LSPU65MyR9gpadacROQVHxGzUN6QWU6PqIQhEkNKCaPanpzwm5DQ6GswWIRZb77kTdTk+/70wb5f9dIxbgn8q2QzAC0DFF0nVeF9+SY5KQKBgDka4o1ro3+sDKMjdHxRFur80wiZbMkGypTHSE/eQA2nNdNxOC1PCXVKsxQjaLUnX4wb79Fy4e58gF0giXUgPva5rf8jb1XyyFl4dfjsaxxLirEAIKGL6Z9vU1jaQbxxfikTdt5hH4BC4lxnP2EefzoBb77aFpcZouKMwdQ1vnG1AoGAZJZOcst0kq9sTbQ9t/8MphL80Mw4DGHRCOYSrKYtnrdSp7Dok90a6cWQHT4jbBraT7dhDFcIAl1+cQ/ZzTOdhkPBuebsPTnr4Jl143bRC1dEOZLseIl6Qi9HpO6VbTXZ4ccOdWndM8Z3sK48M34EUjC7a4MJI8qfSc+q8wbkfCECgYA72aNRAMIJ1grpHtyBSkkwJjQUUJ78e3uhYJXjdBMshBw1ZiaBOB2iY6CQ56uFvbnHGwdPW2uxxme/1drtTGXDvmJboTYZsl215rSRhLLzWEf0MBLhN8XwUoo4W8MBFyeqr5A+oUC2RgAAEEgV/7x2lGVPRBE/WoJWwWBfPvelzA==',
1010
mode: (process?.env?.FLYTRAP_MODE as FlytrapMode) ?? 'capture',
1111
captureId: process?.env?.FLYTRAP_CAPTURE_ID ?? undefined,
12-
logging: ['api-calls', 'capture', 'storage'],
12+
logging: ['error', 'api-calls', 'capture', 'storage'],
1313
packageIgnores: ['next/font']
1414
})

examples/with-nuxt/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,6 @@ export default defineFlytrapConfig({
99
secretApiKey: 'sk_TnBe2CSBNPGE-9jDtiQDvaoef8LOJ17T_xpIvKxVcBJuskGX',
1010
mode: (process?.env?.FLYTRAP_MODE as FlytrapMode) ?? 'capture',
1111
captureId: process?.env?.FLYTRAP_CAPTURE_ID ?? undefined,
12-
logging: ['api-calls', 'capture', 'storage'],
12+
logging: ['error', 'api-calls', 'capture', 'storage'],
1313
excludeDirectories: ['.nuxt']
1414
})

examples/with-sveltekit/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,6 @@ export default defineFlytrapConfig({
1111
captureId: process?.env?.FLYTRAP_CAPTURE_ID ?? undefined,
1212
// ignore Svelte generated files
1313
excludeDirectories: ['./.svelte-kit'],
14-
logging: ['api-calls', 'capture', 'storage'],
14+
logging: ['error', 'api-calls', 'capture', 'storage'],
1515
browser: true
1616
})

examples/with-vue-3/flytrap.config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@ export default defineFlytrapConfig({
99
'sk_MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQC7x0EaJoWb9tGvQeAvo+n2b/2jD9iaTefivggX1U8YcYk5K1l5ffAKUGmE+7GY6JThn3klVjh+smujv1NyibLd2Xt1ZWL7aFHGDpvf8dVY/QVNuaXYEzYdQ0H+6gqb75TE9x1NVSEq08gWxNzLHOKAebT7fIWHt0JEo+LvYYQK+fYGDnZvEZuXsy5nMSMrNapp967wiuwjUzkKpF0f3MX+f2V7sR1rNfPuy/u02ETem2aTZV2WMLH54zahQTXXW1Pv5+U1YH3kzB02AdtjiO3vLuo2dLrhan9jpLCzOZJqMa7o5qJKK3xFwRY8yVaZC/TxY1x+IyTj4sir+9/n/iaVAgMBAAECggEACW8KvcIWr024nhnspegRvoBE2VbX9nSYhEMwy+W7CgrWLomXfquNji8ZBgXRF9IDfMxSvUX2I6NaFglsK1JwqoWdFrpHvUKx7QXZ3efb5JMdm5iKlO7uJo00+RYP/2KLclPJ5JM0zd6C7GXqg8fJN1Ql/AZ2/Oj/isxC/+zBwf1Trnu8iPU/rMTadJJ6RKIT05hdg0kmuA14TkBbNyApvPWD/Kw247MYap+U33ipGsDQOgbQPFRDwoGPPQ0xZWDnCyRaacrLZ2ScFD0y48k3VzzWvkyq/dU+5D1TzEXwSedpK7fIUI84JR/YxFNEErGtjd+oKM70+ZXaR5tSX94JkQKBgQD0WLrmwX0WPOjJdRb8e3WiJCsEmyMPAwT2vEO50uVyX9pzZAFhTzmWzLizJPBcM5kwQsNe4rCn/TuFVtm9vSyE37ZW5EIdU8rM2yjj/KfohFh7m+Cb2sBgu6s5E0oX6gf5VZDnfR5TSsGU7x7RQjcJHCxAB9PzeKaKIaj2AJpD+QKBgQDEu99b10W1XYxycQnL8NdIj35wH8harjpnflaSUCnWY84Kw6jFaE0P1IRz08f8TqUk3zpKp3zEOM+Fi9aU3XlNQIZXN8tIkLFs5tutf9RMfJcQtPuCZMP0egF5DiuJmMuBoHqK/tlmYNDIyI0jab60HoT6SuoXwd1vhwzmKDQmfQKBgAZprb4Qpe/HeXSQFiJaOyQePuLID4t9UMwI9N4TouH1kg4lOcQMCD7k8JQDDCvfTs3tcqMy3+AIzV8agGxyYpC6LlbpdrK+WDw4JEKXhvCbSJc16BcvDo17X2sIDpHCGqN4k2z/46Pb+mimPfDQBBHBI4gqrG1fZT+K0pxiN6pBAoGAQXoV2MdJGBfOirg8fnfl7uxzNHYfhK7AGXne7ktn4UbOaJ6+KRWRsFcWzkYKJVs3c5IoIHil+di+mL/15w3fNWroS1byfXBK3Ofo7VHCIsLEtHbHzpoi7dc9bkDdFKqZ8Masgl5XlM9cLbcx4jtrkR8nOvZ7+vQ28suL634t/TECgYEAxfGz4MJ2O5iiXm+Cpm1H9+lmjmKKbruRcZ2/rMMXus2B1uGEoePJQY82oSxqtIrabh8AY2yKWQ4eA2NHlGVBoScs81qjRBsvkGYLeWzCmsKwfrD7mTVpz1U5rUOs6LMLAUDwkzwDtSf6/p75NpFnwadvBY1zRUAsqzJPnpHwZrg=',
1010
mode: 'capture',
1111
excludeFunctionNames: ['defineProps'],
12-
logging: ['api-calls', 'capture', 'storage']
12+
logging: ['error', 'api-calls', 'capture', 'storage']
1313
})

rollup.config.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -79,7 +79,7 @@ export default [
7979
sourcemap: true,
8080
},
8181
],
82-
}, */
82+
},
8383
{
8484
input: ["src/index.ts"],
8585
plugins: [dts()],
@@ -91,7 +91,7 @@ export default [
9191
exports: "named",
9292
},
9393
],
94-
},
94+
}, */
9595
// Build transform plugin
9696
{
9797
input: ["src/transform.ts"],

src/core/newStorage.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ import { getLimitedCaptures } from './captureLimits'
88
import { shouldIgnoreCapture } from './captureIgnores'
99
import { formatBytes } from './util'
1010
import { getUserId } from '../index'
11-
import { removeCircularsAndNonPojos, safeStringify } from './stringify'
11+
import { removeCircularsAndNonPojos, removeUnserializableValues, safeStringify } from './stringify'
1212
import { decryptCapture, encryptCapture } from './newEncryption'
1313
import { newRequest } from './requestUtils'
1414

@@ -93,6 +93,10 @@ export async function saveCapture(
9393
)
9494
}
9595

96+
// Remove unserializable values
97+
calls = removeUnserializableValues(calls)
98+
functions = removeUnserializableValues(functions)
99+
96100
// Remove circulars and non-pojos
97101
calls = removeCircularsAndNonPojos(calls)
98102
functions = removeCircularsAndNonPojos(functions)

0 commit comments

Comments
 (0)