From e03f8e88689225356a30e7abfd3fef06705a6cb7 Mon Sep 17 00:00:00 2001 From: gutsytechster Date: Tue, 2 Jun 2020 17:38:55 +0530 Subject: [PATCH] refactor(tests): Add helper methods for path lookups This adds helper methods to test_install_force_reinstall.py and test_uninstall_user.py --- tests/functional/test_install_force_reinstall.py | 2 +- tests/functional/test_uninstall_user.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/functional/test_install_force_reinstall.py b/tests/functional/test_install_force_reinstall.py index c312ac79f..0fbdeb276 100644 --- a/tests/functional/test_install_force_reinstall.py +++ b/tests/functional/test_install_force_reinstall.py @@ -33,7 +33,7 @@ def check_force_reinstall(script, specifier, expected): # site_packages_path is absolute, but files_created mapping uses # relative paths as key. fixed_key = os.path.relpath(to_fix, script.base_path) - assert fixed_key in result2.files_created, 'force-reinstall failed' + result2.did_create(fixed_key, message='force-reinstall failed') result3 = script.pip('uninstall', 'simplewheel', '-y') assert_all_changes(result, result3, [script.venv / 'build', 'cache']) diff --git a/tests/functional/test_uninstall_user.py b/tests/functional/test_uninstall_user.py index a367796fd..d29e088f6 100644 --- a/tests/functional/test_uninstall_user.py +++ b/tests/functional/test_uninstall_user.py @@ -62,7 +62,7 @@ class Tests_UninstallUserSite: 'install', '--user', '-e', to_install ) egg_link = script.user_site / 'FSPkg.egg-link' - assert egg_link in result1.files_created, str(result1.stdout) + result1.did_create(egg_link) # uninstall result2 = script.pip('uninstall', '-y', 'FSPkg')