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 test coverage for update-checkout --stash
option
#72523
Comments
Hi MaxDesiatov, I am new to the open-source contribution. If no one is currently working on this issue, I would like to work on it. If that is the case, could you assign it to me? Thanks a lot! |
My concern here is that this is quite a long-running operation for real repositories. Maybe creating some mock repositories on the file system in a temp directory could be a sufficient workaround for that. |
Using the mock remotes that are already set up for each test, of course. See |
author Fabrice Renard <[email protected]> 1711509329 -0400 committer Fabrice Renard <[email protected]> 1712183881 -0400 parent dea09e1 author Fabrice Renard <[email protected]> 1711509329 -0400 committer Fabrice Renard <[email protected]> 1712183868 -0400 Complete update-checkout stash test apple#72523
update-checkout
has a Python unit test suite atutils/update_checkout/tests
. You can run it usingutils/update_checkout/run_tests.py
orlit
+validation-test/Python/update_checkout.swift
. The test suite is centered around an intermediaryunittest.TestCase
subclass from which all the effective test classes inherit. This base class is responsible for setting up and tearing down a mock-up environment consisting of a directory of local remotes and a source root — a directory for their clones. Subclasses define test functions that runupdate-checkout
commands against these mock repositories and perform state checks.I think a single test function that
update-checkout
update-checkout
with--stash
would good enough for now (the update/clone tests, if any, are really basic anyway). I would place this function in a new subclass in its own file.
Originally posted by @AnthonyLatsis in #71178 (comment)
The text was updated successfully, but these errors were encountered: