-
Notifications
You must be signed in to change notification settings - Fork 16
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 stripe tests in CI #410
base: master
Are you sure you want to change the base?
Conversation
Important Auto Review SkippedDraft detected. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
@coderabbitai Make sure that the .env file is read before running the Stripe tests, so they are not skipped |
@emanuelen5 It appears that there are no direct references to Since the standard approach with If the Stripe tests are being skipped due to missing environment variables, you might want to ensure that the test runner or CI/CD pipeline is configured to load the If you have a specific setup or test file that should be handling this, please point me to it, and I can look into that file directly. Otherwise, you may need to manually check your test runner or CI/CD pipeline settings. Analysis chain@emanuelen5 To ensure that the The output from the script does not explicitly show the loading of a To proceed, I'll generate a script to search for any occurrences of Scripts executedThe following scripts were executed for the analysis: Script: #!/bin/bash
# Find files that are likely to be responsible for loading .env files
fd --type f --extension sh --extension js --extension json "test|setup|config" | xargs cat
Length of output: 16335 Script: #!/bin/bash
# Search for dotenv or similar environment variable management usage
rg --type js --type ts "require\(['\"]dotenv['\"]\)|import ['\"]dotenv['\"]"
Length of output: 86 |
All of the subscription tests pass now. The following test still fail, though
|
8e13767
to
6bd1a5f
Compare
^ Rebased onto origin/master |
6bd1a5f
to
e0b3741
Compare
e0b3741
to
ec7be0b
Compare
685e8ab
to
c115cd5
Compare
No description provided.