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

Missing Tests? #63

Open
ramarnat opened this issue May 29, 2024 · 4 comments
Open

Missing Tests? #63

ramarnat opened this issue May 29, 2024 · 4 comments

Comments

@ramarnat
Copy link

I have made some improvements to the ingester, but it looks like the way the tests work is to download the test files from a mongodb database. Is it possible to add those test files into the repo or provide it in some other way?

@jonhilgart22
Copy link

@ramarnat want to compare notes on your updates?

@ansukla-salesforce
Copy link

The files we used for testing are not shared as it depended on the specific customer use cases. It is possible to recreate these tests with your files, however I can understand that keeping these in mongodb etc. would be a pain.

@ramarnat
Copy link
Author

ramarnat commented Oct 1, 2024

@jonhilgart22 I moved some of the atomic functions to rust. I didn't want to make a a lot of other changes without the tests because the code seems to deal with a lot of edge cases, and without the tests it's hard to know what the impact of a change might be.

@ansukla-salesforce I understand why they maybe hard to share. maybe the tests can be shared under an NDA or private repo and we can run them independently before contribution. Maybe there is a way to do something via gihubactions that can run the tests without exposing the test files publicly.

@jonhilgart22
Copy link

The files we used for testing are not shared as it depended on the specific customer use cases. It is possible to recreate these tests with your files, however I can understand that keeping these in mongodb etc. would be a pain.

It would be great to see some test files to ensure existing functionality works!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants