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

revup test suite #7

Open
jerry-skydio opened this issue Aug 18, 2022 · 2 comments
Open

revup test suite #7

jerry-skydio opened this issue Aug 18, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jerry-skydio
Copy link
Collaborator

We desperately need a comprehensive test suite before we can start taking more contributions

@jerry-skydio jerry-skydio added the enhancement New feature or request label Aug 18, 2022
@jerry-skydio jerry-skydio self-assigned this Aug 18, 2022
@jerry-skydio
Copy link
Collaborator Author

More thoughts

Test helper classes:

GitEnvironment - a test helper that initializes a temporary git repository and sets up a basic commit history and file structure. Includes simple methods for making and checking the contents of files, commits, and cache.

FakeGithub - inspired by https://github.com/ezyang/ghstack/blob/master/ghstack/github_fake.py. A fake github server object that can parse and return results for graphql queries. Also contains a push target that intercepts git pushes and redirects them to a git repo inside the fake server.

@gbalke
Copy link
Contributor

gbalke commented Sep 15, 2022

Github fake looks pretty cool. I don't do a lot of web dev so maybe this is not actually an issue... I have some concerns about the stability of the API and having to maintain a mock of it? I definitely think it's worth using but it'd be interesting if github already provides a mock interface to their web API 🤔 That'd be pretty sweet 🙂

Doesn't seem to be the case. I guess we want more than valid API requests (might be worth having a test for this as well to track api deprecation). We want to mock the behavior of creating real pull requests and whatnot. A github mock does seem to be the most reasonable way of doing this.

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

No branches or pull requests

2 participants