Skip to content
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

False positive integration test failures #43

Open
schuylermartin45 opened this issue May 1, 2024 · 1 comment
Open

False positive integration test failures #43

schuylermartin45 opened this issue May 1, 2024 · 1 comment
Labels
infrastructure Issues related to automated testing and CI/CD tasks to triage Needs additional investigation

Comments

@schuylermartin45
Copy link
Collaborator

rattler-build will fail a dry-run build if environment variables referenced by env.get() are not present.

For our purposes, this is analogous to a false-positive and potentially obfuscates other test failures as this check appears to happen before some other validation checks.

We should find a way to mitigate this issue. At the very least, this issue affects all R recipe files.

@schuylermartin45 schuylermartin45 added infrastructure Issues related to automated testing and CI/CD tasks to triage Needs additional investigation labels Sep 20, 2024
@schuylermartin45
Copy link
Collaborator Author

I think we addressed this at one point? I think we might pre-fill some environment variables for the dry-run build? I don't remember at this current time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infrastructure Issues related to automated testing and CI/CD tasks to triage Needs additional investigation
Projects
Status: Backlog
Development

No branches or pull requests

1 participant