remove un-needed todo, users have to provide config to constructor

axiomhq · Nov 16, 2023 · 248269c · 248269c
1 parent f4fc53e
commit 248269c
Showing 1 changed file with 0 additions and 17 deletions.
diff --git 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');
-  // });
 });