traceloop / jest-opentelemetry Goto Github PK

📜 Description

It appears that jest can not properly interpret the index.js file, since it is using imports and is not in commonjs format unlike the compiled TS portions of the code base.

👟 Reproduction steps

Use jest to run *.js files instead of using ts-jest, etc. Useful, because there are some challenges with ts-jest and we can run tests on *.js after TS files are compiled down to commonjs.

👍 Expected behavior

It should not have a syntax error when parsing.

👎 Actual Behavior with Screenshots

@appbloks/storage:test: ● Test suite failed to run
@appbloks/storage:test:
@appbloks/storage:test: test/node_modules/.pnpm/@traceloop[email protected]/node_modules/@traceloop/jest-environment-otel/index.js:1
@appbloks/storage:test: import { teardown, setup } from './dist/global';
@appbloks/storage:test: ^^^^^^
@appbloks/storage:test:
@appbloks/storage:test: SyntaxError: Cannot use import statement outside a module

🤖 Node Version

v18.10.0

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

None

🔖 Feature description

Similar to how we have Postgres queries assertions that allows us to test that a write to Postgres happened in some system flow, we want to add assertions against MongoDB queries.

🎤 Why is this feature needed ?

Completeness of jest-opentelemetry

✌️ How do you aim to achieve this?

Similar to the work we've done in Postgres, this just needs to parse the appropriate MongoDB spans coming from OpenTelemetry.

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

None

🔖 Feature description

Support asserting against loading data into Snowflake. This will allow us to write assertions like

expectTrace(traceloop)
  .toReportToSnowflake()
  .withEvent('user-registered')
  .withProperties({ 'email': '[email protected]'});

🎤 Why is this feature needed ?

This is a much required feature by our users as Snowflake is widely used.

✌️ How do you aim to achieve this?

Probably the easiest way is to find and parse appropriate REST API calls to Snowflake -
https://docs.snowflake.com/en/user-guide/data-load-snowpipe-rest-load

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

None

Our project started as a Javascript project and is gradually moving to Typescript.

• Complete update of all .js files to .ts.
• Fix all outstanding ESLint issues.

🔖 Feature description

Implement support for GRPC assertions, similar to REST

🎤 Why is this feature needed ?

Support running complete test suites on Otel's Demo

✌️ How do you aim to achieve this?

We need to track specific GRPC attributes as defined by Otel
See -
https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/semantic_conventions/rpc.md#grpc-attributes

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

None

🔖 Feature description

Hi, in going through YCombinator's recent OS projects, I stumbled across traceloop, I landed on your site and I feel some changes could be effected to give it more life or make it look more active.

🎤 Why is this feature needed ?

It is paramount that there is a revamp of the landing page as I've observed that you've just crossed 100 stars on Github and more and more people are going to visit the landing page, it needs to look like an active project.

✌️ How do you aim to achieve this?

This is my first foray into open-source so pardon me for any errors in my convention. I've created a design file and I'm willing to customize and effect changes. I can also convert that to live code using Next.js, all I need is approval or consent from the team.

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

Yes I am willing to submit a PR!

🔖 Feature description

Our Postgres assertions can only validate exact queries.
So, for example, you can validate that a query like INSERT INTO users(email) VALUES ([email protected]).
The downside of this is that you need to specify the exact query that was made on the database.

But usually, you don't really want to validate the entire query syntax, you just want to validate that some fields were added to the table, but you don't care about the others.

This feature should allow writing an assertion like

expectTrace(traceloop)
  .toQueryPostgresSQL()
  .withPartialQuery(
    `INSERT INTO users(email) VALUES ([email protected])`

And it will succeed even if the actual query was
INSERT INTO users(name, request_id, email) VALUES (bla, 5934345, [email protected])

🎤 Why is this feature needed ?

Our goal with this project is to make end-to-end tests easier. Being able to write simple assertions Is core to this.

✌️ How do you aim to achieve this?

We will need to use some Typescript SQL parser to parse and compare the asserted and actual SQL queries.

🔄️ Additional Information

No response

👀 Have you spent some time to check if this feature request has been raised before?

• I checked and didn't find similar issue

Are you willing to submit PR?

None