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
tools.vpm: update tests, add get_installed
test
#20028
Conversation
b40475b
to
d9626b2
Compare
… about error location in tests
…rview and searchability
d9626b2
to
c89a8fa
Compare
The occasional failure with windows msvc seems to be unrelated.
|
@@ -20,62 +20,62 @@ fn testsuite_end() { | |||
os.rmdir_all(test_path) or {} | |||
} | |||
|
|||
fn get_vmod(path string) vmod.Manifest { | |||
return vmod.from_file(os.join_path(test_path, path, 'v.mod')) or { | |||
eprintln('Failed to parse v.mod for `${path}`') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please add err: $err
in the message too
os.chdir(test_project_path)! | ||
os.write_file('v.mod', '')! | ||
if os.getenv('CI') != '' { | ||
os.execute_or_exit('git config --global user.email "v@vi.com"') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vi.com
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work.
Updates vpm tests that have increased in number, makes some minor improvements, adds a test.
Changes split into commits.
🤖[deprecated] Generated by Copilot at b40475b
The pull request enhances the vpm tool and its tests by refactoring code, improving error handling, increasing reliability, and adding more test cases and logging. It affects the files
dependency_test.v
,install_test.v
,install_version_input_test.v
,install_version_test.v
,update_test.v
, andparse.v
in thecmd/tools/vpm
directory.🤖[deprecated] Generated by Copilot at b40475b
dependency_test.v
,install_version_test.v
, andupdate_test.v
to reduce flaky test failures (link, link, link)dependency_test.v
,install_test.v
, andupdate_test.v
to improve error handling and test assertions (link, link, link, link, link, link)install_test.v
, and update the test cases ininstall_test.v
,install_version_input_test.v
, andinstall_version_test.v
to use this function and access the manifest fields instead of separate variables (link, link, link, link, link, link, link, link, link, link)install_test.v
, which checks the installation and version status of a module with different branch and tag states (link)parse.v
, for debugging purposes (link)install_version_test.v
installs from a GitLab repository, not GitHub (link)install_test.v
, to reflect the broader scope of the tests (link)