-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run all examples as part of E2E tests #62
Comments
I want to work on this. Please assign to me if it's OK. Thanks! |
I appreciate your dedication. However, there are a few things you need to consider.
|
It would be great to be able to run all code snippets in
It would be great to use Feel free to come up with something or ask for guidance. |
I need some guidance. I'm still learning TypeScript. The examples are excluded in /tsconfig.json" and are under "/examples/tsconfig.json". There is another "/tsconfig.examples.json" that extends "/examples/tsconfig.json" and referred by scripts in "package.json". I tried to import the examples from the tests but so far I couldn't figure out how. I probably can copy the examples in another location under "src", tweak the import statements and import from the test program in the execution of the tests but it's not nice. Is there any good way to import the example from the test program? Thanks! |
Let me clarify.
Don't import tests directly into' test`. It would be easier to run them as standalone scripts (they are scripts; they do not export anything). Look at how we implemented this in IBM Generative AI Python SDK - test examples script. |
Is your feature request related to a problem? Please describe.
The examples/agents/bee got broken multiple times in the last a few days. #55 #60
Describe the solution you'd like
Add e2e tests that runs the example so that the issue is detected in the pull request.
Describe alternatives you've considered
Run the example manually before creating the pull requests.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: