-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add tests to blitz core #873
Comments
I'd love to take a crack at this |
I checked it out, tried to use the nextjs tests as a template, but the |
Yeah, @MrLeebo I see what you mean. I was able to copy the test-utils and get that compiling properly (not that I think that's the ideal solution). But there's a ton of config/packages that goes into their integration tests that I'm not sure the right move is to copy it. Not to mention they use selenium for all their stuff and I don't know if that's the direction we want to be entrenched. I'm going to back away from this one for now. I'm going out of town in a day and it's going to take a deeper dive than I'll have time for this week to build out a testing harness. I don't want to be in the way. |
If by any chance this hasn't been taken care of when I get back home on Sunday, I will gladly start working on it! |
I started a PR (#16), but I'm sure there's more testing to be done. |
Blitz core is currently untested. We need to add comprehensive tests. This would be a good task for anyone who wants it!
Comment below if you're getting ready to work on it
The text was updated successfully, but these errors were encountered: