From 248269cd48f4497038bd04baec39c31546ae229a Mon Sep 17 00:00:00 2001 From: Islam Shehata Date: Thu, 16 Nov 2023 13:45:00 +0200 Subject: [PATCH] remove un-needed todo, users have to provide config to constructor --- e2e/nextjs-app/test/ingest.spec.ts | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/e2e/nextjs-app/test/ingest.spec.ts b/e2e/nextjs-app/test/ingest.spec.ts index ad235124..85fbc98e 100644 --- a/e2e/nextjs-app/test/ingest.spec.ts +++ b/e2e/nextjs-app/test/ingest.spec.ts @@ -63,21 +63,4 @@ describe('Ingestion & query on different runtime', () => { expect(qResp.tables[0].columns[0].data.foo).toEqual('bar'); expect(qResp.tables[0].columns[1].data.bar).toEqual('baz'); }); - - // TODO: enable this test again when env vars are available on browser runtime - // it('ingest on a browser runtime should succeed', async () => { - // const startTime = new Date(Date.now()).toISOString(); - // // call route that ingests logs - // const resp = await fetch(process.env.TESTING_TARGET_URL + '/ingest'); - // expect(resp.status).toEqual(200); - - // // check dataset for ingested logs - // const qResp = await axiom.query(`['${datasetName}'] | where ['test'] == "ingest_on_browser"`, { - // startTime, - // }); - // expect(qResp.matches).toBeDefined(); - // expect(qResp.matches).toHaveLength(2); - // expect(qResp.matches![0].data.foo).toEqual('bar'); - // expect(qResp.matches![1].data.bar).toEqual('baz'); - // }); });