-
Notifications
You must be signed in to change notification settings - Fork 84
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
[RHELC-1169] Add a log message to check if the user has a back up of their system #1235
Commits on Oct 18, 2024
-
Added the log message for the user system backup
* This commit adds to the print_data_collection() to make sure the user knows to back up there system before running convert2rhel analyze
Configuration menu - View commit details
-
Copy full SHA for c98f96e - Browse repository at this point
Copy the full SHA c98f96eView commit details -
Moved the log outside the print_data_collection()
* it is better to have this outside of the print_data_collection() as it has nothing to do with that
Configuration menu - View commit details
-
Copy full SHA for 7239776 - Browse repository at this point
Copy the full SHA 7239776View commit details -
Moved the confirmation of user backup
* the confirmation for the users backup is now its own function and is seperated from the print_data_collection() as this has noting to do with that function * Added the call to the new function to be right after the print_data_collection()
Configuration menu - View commit details
-
Copy full SHA for fc4c425 - Browse repository at this point
Copy the full SHA fc4c425View commit details -
Made a function for the user back up check
* now the prompt to ask the user if they have a back up of there system is in a function * draft the unit test for this function
Configuration menu - View commit details
-
Copy full SHA for 04a92e5 - Browse repository at this point
Copy the full SHA 04a92e5View commit details -
Moved the confirm_user_backup() to main.py
*Moved the function and test to main.py, and main_test.py as this check doesn't align with what breadcrumbs.py is doing
Configuration menu - View commit details
-
Copy full SHA for 8722fdb - Browse repository at this point
Copy the full SHA 8722fdbView commit details -
Finnished the uint test for Confirm_user_backup()
* This unit test was mostly done we just needed to account for the user input and now it does that and it passes
Configuration menu - View commit details
-
Copy full SHA for ea55420 - Browse repository at this point
Copy the full SHA ea55420View commit details -
Removed leftover imports from breadcrumbs_test.py
* In the test for breadcrumbs_test.py there where imports for the test that were never used * fixed two tests that fail with the addition of this test because the function calls utils.ask_to_continue() more then the other tests expect
Configuration menu - View commit details
-
Copy full SHA for 2fde33a - Browse repository at this point
Copy the full SHA 2fde33aView commit details -
Draft used the mock for the users input
* This commit changes the way the unit test is set up so that it now uses the ask_to_continue_mock in main_test to mock user input * Removed the import for unit test as the old test used it * Reverted the other uint tests that used the ask_to_continue_mock fuctions and set their values back to what they were before modification in the first commit
Configuration menu - View commit details
-
Copy full SHA for ad0f23d - Browse repository at this point
Copy the full SHA ad0f23dView commit details -
Configuration menu - View commit details
-
Copy full SHA for b5cc721 - Browse repository at this point
Copy the full SHA b5cc721View commit details -
Added a mock to ask_to_continue() in
* in some of the other tests the ask_to_continue fuction is now being called from the test_confirm_user_backup() and this should be handled so for thoes functions a mock,monkeypatch, and a check was added. functions:test_main_rollback_analyze_exit_phase, test_main_rollback_analyze_exit_phase_without_subman, test_main_rollback_pre_ponr_changes_phase, test_main_traceback_before_action_completion, test_main_traceback_in_clear_versionlock,
Configuration menu - View commit details
-
Copy full SHA for 105c536 - Browse repository at this point
Copy the full SHA 105c536View commit details -
Draft for the fix for the sanity tests
* the intergration tests needed another yes input for the newly added prompt
Configuration menu - View commit details
-
Copy full SHA for 472b4de - Browse repository at this point
Copy the full SHA 472b4deView commit details -
Draft added in the rest of the fixes for the tests
* In the intergration tests there where two more tests that needed the extra input value "y" so that it would account for the new prompt
Configuration menu - View commit details
-
Copy full SHA for 886547b - Browse repository at this point
Copy the full SHA 886547bView commit details -
Configuration menu - View commit details
-
Copy full SHA for e18da44 - Browse repository at this point
Copy the full SHA e18da44View commit details -
Configuration menu - View commit details
-
Copy full SHA for f60d80f - Browse repository at this point
Copy the full SHA f60d80fView commit details -
Added in some more cases for the intergration test
* there are a few more intergration tests that needed another input for the new check that was added
Configuration menu - View commit details
-
Copy full SHA for e0abaa3 - Browse repository at this point
Copy the full SHA e0abaa3View commit details -
added in two yes prompts for the intergartion test
* added the extra propmts in test_basic_sanity_checks.py and test_assessment_report.py
Configuration menu - View commit details
-
Copy full SHA for 9c74bb7 - Browse repository at this point
Copy the full SHA 9c74bb7View commit details -
Configuration menu - View commit details
-
Copy full SHA for 3b08ad9 - Browse repository at this point
Copy the full SHA 3b08ad9View commit details -
Removed one of the c2r.expect lines from a test
* There was an extra c2r.expect line in the pre_registered_systems_conversions intergration test
Configuration menu - View commit details
-
Copy full SHA for 6607a58 - Browse repository at this point
Copy the full SHA 6607a58View commit details -
Added a prompt to another intergration test
* Added the input for the intergration test pre_registered_system_conversion
Configuration menu - View commit details
-
Copy full SHA for 011bdc0 - Browse repository at this point
Copy the full SHA 011bdc0View commit details -
Configuration menu - View commit details
-
Copy full SHA for 308feb0 - Browse repository at this point
Copy the full SHA 308feb0View commit details -
Configuration menu - View commit details
-
Copy full SHA for a37bb0c - Browse repository at this point
Copy the full SHA a37bb0cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 39138a4 - Browse repository at this point
Copy the full SHA 39138a4View commit details
Commits on Oct 21, 2024
-
Removed some old code from a rebase
Anderw Anglin committedOct 21, 2024 Configuration menu - View commit details
-
Copy full SHA for 4293d95 - Browse repository at this point
Copy the full SHA 4293d95View commit details -
fully removed the test_confirm_user_backup() from breadcrumbs_test
* in the last commit it just changed the test but it was supposed to remove the test all together it is now fully removed
Anderw Anglin committedOct 21, 2024 Configuration menu - View commit details
-
Copy full SHA for 6f74781 - Browse repository at this point
Copy the full SHA 6f74781View commit details -
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
Configuration menu - View commit details
-
Copy full SHA for b9e4db7 - Browse repository at this point
Copy the full SHA b9e4db7View commit details -
Draft trying to just pass the input for the test
Anderw Anglin committedOct 21, 2024 Configuration menu - View commit details
-
Copy full SHA for 42a484d - Browse repository at this point
Copy the full SHA 42a484dView commit details
Commits on Oct 22, 2024
-
draft Commited out the extra prompt
* This is just a test to try and see whats going on with the test Just to make sure that this test needs two prompts
Configuration menu - View commit details
-
Copy full SHA for ed1ccd2 - Browse repository at this point
Copy the full SHA ed1ccd2View commit details
Commits on Oct 23, 2024
-
Configuration menu - View commit details
-
Copy full SHA for b5860a6 - Browse repository at this point
Copy the full SHA b5860a6View commit details
Commits on Oct 24, 2024
-
Configuration menu - View commit details
-
Copy full SHA for fe6fc06 - Browse repository at this point
Copy the full SHA fe6fc06View commit details