Description of problem: Version-Release number of selected component (if applicable): How reproducible: Steps to Reproduce: 1. 2. 3. Actual results: Expected results: Additional info:
https://github.com/ManageIQ/manageiq-v2v/issues/821
https://github.com/ManageIQ/manageiq-v2v/pull/819
https://github.com/ManageIQ/manageiq-v2v/pull/820
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/e85532dcbf62a7e114a072e3c10869f520fbf74d commit e85532dcbf62a7e114a072e3c10869f520fbf74d Author: Mike Turley <mturley> AuthorDate: Wed Dec 5 17:16:53 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Wed Dec 5 17:16:53 2018 -0500 Add tests for PlanWizardInstancePropertiesStep Actions and Reducer https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStepActions.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStepReducer.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/PlanWizardInstancePropertiesStepActions.test.js | 67 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/PlanWizardInstancePropertiesStepReducer.test.js | 236 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/__snapshots__/PlanWizardInstancePropertiesStepActions.test.js.snap | 17 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/__snapshots__/PlanWizardInstancePropertiesStepReducer.test.js.snap | 116 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/planWizardInstancePropertiesStep.fixtures.js | 295 + 7 files changed, 733 insertions(+), 2 deletions(-)
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/38a0d376ab7f148db15926994bf05fbb5150b963 commit 38a0d376ab7f148db15926994bf05fbb5150b963 Author: Mike Ro <mikerodev> AuthorDate: Wed Dec 5 19:52:55 2018 -0500 Commit: Mike Ro <mikerodev> CommitDate: Wed Dec 5 19:52:55 2018 -0500 Test fetching cloud tenants https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStep.js | 9 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepConstants.js | 8 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepReducer.js | 2 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStep.test.js | 50 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/index.test.js | 29 +- app/javascript/react/screens/App/common/testReduxHelpers.js | 8 + 6 files changed, 81 insertions(+), 25 deletions(-)
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/8a14c8087624880ad9d7da73297d832a0b56e48e commit 8a14c8087624880ad9d7da73297d832a0b56e48e Author: Mike Turley <mturley> AuthorDate: Fri Dec 7 13:20:59 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Fri Dec 7 13:20:59 2018 -0500 Add tests for resolving migration plan POST body from redux state https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/screens/PlanWizard/__tests__/__snapshots__/helpers.test.js.snap | 59 + app/javascript/react/screens/App/Overview/screens/PlanWizard/__tests__/helpers.test.js | 36 + app/javascript/react/screens/App/Overview/screens/PlanWizard/planWizard.fixtures.js | 105 + 3 files changed, 200 insertions(+)
https://github.com/ManageIQ/manageiq-v2v/pull/827
https://github.com/ManageIQ/manageiq-v2v/pull/828
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/ef7bdee35942c7fa761c289fffda0342c444091e commit ef7bdee35942c7fa761c289fffda0342c444091e Author: Mike Turley <mturley> AuthorDate: Fri Dec 7 15:39:27 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Fri Dec 7 15:39:27 2018 -0500 Add tests for OSP VM power warnings in the Plan wizard https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/__tests__/PlanWizardResultsStep.test.js | 35 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/__tests__/__snapshots__/PlanWizardResultsStep.test.js.snap | 64 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/PlanWizardScheduleStep.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/__tests__/PlanWizardScheduleStep.test.js | 19 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/__tests__/__snapshots__/PlanWizardScheduleStep.test.js.snap | 77 + 5 files changed, 196 insertions(+), 1 deletion(-)
https://github.com/ManageIQ/manageiq-v2v/pull/829
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/8133d78c629e05d743617deef8508473c6c619dc commit 8133d78c629e05d743617deef8508473c6c619dc Author: Mike Turley <mturley> AuthorDate: Fri Dec 7 16:29:44 2018 -0500 Commit: Mike Turley <mturley> CommitDate: Fri Dec 7 16:29:44 2018 -0500 Add tests for OSP conversion host warning in mapping wizard clusters step https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStep.test.js | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq-v2v/master: https://github.com/ManageIQ/manageiq-v2v/commit/39dc591e23b8890412ad6039e6c1c12f7ddd708e commit 39dc591e23b8890412ad6039e6c1c12f7ddd708e Author: Mike Ro <mikerodev> AuthorDate: Fri Dec 7 14:59:45 2018 -0500 Commit: Mike Ro <mikerodev> CommitDate: Fri Dec 7 14:59:45 2018 -0500 Tests for enhanced error reporting https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/__test__/OverviewActions.test.js | 2 +- app/javascript/react/screens/App/Overview/__test__/OverviewSelectors.test.js | 4 +- app/javascript/react/screens/App/Overview/components/Migrations/__test__/MigrationsInProgressCard.test.js | 83 + app/javascript/react/screens/App/Overview/components/Migrations/__test__/__snapshots__/MigrationsInProgressCard.test.js.snap | 68 + app/javascript/react/screens/App/Overview/overview.transformationPlans.fixtures.js | 54 + 5 files changed, 208 insertions(+), 3 deletions(-)
New commits detected on ManageIQ/manageiq-v2v/hammer: https://github.com/ManageIQ/manageiq-v2v/commit/8e04c0172af08cb22504d4d885171c7ca91da99e commit 8e04c0172af08cb22504d4d885171c7ca91da99e Author: Michael Ro <mikerodev> AuthorDate: Tue Dec 4 14:47:29 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Tue Dec 4 14:47:29 2018 -0500 Merge pull request #816 from mturley/simple-api-mock Replace broken axios API mock adapter, fix broken tests (cherry picked from commit ed5249d75f12aa0b2b6b830849ef005d30a0c5f6) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 .jest-setup.js | 11 +- app/javascript/common/mockRequests.js | 34 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClusterStepActions.test.js | 8 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/__snapshots__/MappingWizardClusterStepActions.test.js.snap | 26 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardGeneralStep/__tests__/MappingWizardGeneralStepActions.test.js | 2 +- app/javascript/react/screens/App/Overview/__test__/OverviewActions.test.js | 14 +- app/javascript/react/screens/App/Overview/__test__/__snapshots__/OverviewActions.test.js.snap | 27 +- app/javascript/react/screens/App/Plan/__test__/PlanActions.test.js | 2 +- app/javascript/react/screens/App/Plan/__test__/__snapshots__/PlanActions.test.js.snap | 13 + 9 files changed, 111 insertions(+), 26 deletions(-) https://github.com/ManageIQ/manageiq-v2v/commit/f3b16cd7f626fe674063e51b7134fd4989135d86 commit f3b16cd7f626fe674063e51b7134fd4989135d86 Author: Michael Ro <mikerodev> AuthorDate: Thu Dec 6 14:00:08 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Thu Dec 6 14:00:08 2018 -0500 Merge pull request #817 from mturley/settings-tests Add unit tests for Settings page components, actions and reducers (cherry picked from commit 430933a767febaf649edc4477c4640c05ec7477a) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 .eslintrc.js | 1 + .jest-setup.js | 1 + app/javascript/common/mockRequests.js | 2 +- app/javascript/components/dates/IsoDate.test.js | 5 +- app/javascript/components/dates/LongDateTime.test.js | 7 +- app/javascript/components/dates/RelativeDateTime.test.js | 5 +- app/javascript/components/dates/ShortDateTime.test.js | 7 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/__tests__/MappingWizard.test.js | 3 +- app/javascript/react/screens/App/Settings/Settings.js | 2 +- app/javascript/react/screens/App/Settings/__tests__/Settings.test.js | 36 + app/javascript/react/screens/App/Settings/__tests__/SettingsActions.test.js | 103 + app/javascript/react/screens/App/Settings/__tests__/SettingsReducer.test.js | 104 + app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 96 + app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsActions.test.js.snap | 113 + app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 149 + app/javascript/react/screens/App/Settings/settings.fixures.js | 48 + app/javascript/react/screens/App/common/NotificationList/__test__/NotificationList.test.js | 3 +- package.json | 5 +- 18 files changed, 669 insertions(+), 21 deletions(-) https://github.com/ManageIQ/manageiq-v2v/commit/9d0ab94a87a837d12c74364e7d13254b9c9fd9f5 commit 9d0ab94a87a837d12c74364e7d13254b9c9fd9f5 Author: Michael Ro <mikerodev> AuthorDate: Fri Dec 7 11:28:22 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Fri Dec 7 11:28:22 2018 -0500 Merge pull request #820 from mturley/instance-properties-tests Add tests for Plan Wizard Instance Properties step actions and reducer (cherry picked from commit db3882ad0b41199df926ab538e82471f7e74c9cd) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStepActions.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/PlanWizardInstancePropertiesStepReducer.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/PlanWizardInstancePropertiesStepActions.test.js | 67 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/PlanWizardInstancePropertiesStepReducer.test.js | 236 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/__snapshots__/PlanWizardInstancePropertiesStepActions.test.js.snap | 17 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/__tests__/__snapshots__/PlanWizardInstancePropertiesStepReducer.test.js.snap | 116 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardInstancePropertiesStep/planWizardInstancePropertiesStep.fixtures.js | 295 + 7 files changed, 733 insertions(+), 2 deletions(-) https://github.com/ManageIQ/manageiq-v2v/commit/082b195e9c72e01c96d54d1ccc49121d0b2a05f0 commit 082b195e9c72e01c96d54d1ccc49121d0b2a05f0 Author: Mike Turley <mike.turley.umass.edu> AuthorDate: Fri Dec 7 12:33:18 2018 -0500 Commit: Mike Turley <mike.turley.umass.edu> CommitDate: Fri Dec 7 12:33:18 2018 -0500 Merge pull request #819 from michaelkro/osp-mapping-wizard Add tests for mapping wizard (cherry picked from commit ee80fcfcfb4013068f5eb92eae1f85e2b033fc0a) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 .jest-setup.js | 5 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/__tests__/__snapshots__/helpers.test.js.snap | 23 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/__tests__/helpers.test.js | 57 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStep.js | 9 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepConstants.js | 8 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepReducer.js | 2 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClusterStepActions.test.js | 6 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStep.test.js | 50 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/index.test.js | 29 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/mappingWizardClustersStep.fixtures.js | 89 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardDatastoresStep/__tests__/MappingWizardDatastoresStep.test.js | 48 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardDatastoresStep/__tests__/MappingWizardDatastoresStepActions.test.js | 77 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardDatastoresStep/__tests__/__snapshots__/MappingWizardDatastoresStepActions.test.js.snap | 50 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/MappingWizardNetworksStepReducer.js | 2 +- app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardDatastoresStepReducer.test.js | 98 - app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardNetworksStep.test.js | 70 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardNetworksStepActions.test.js | 108 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardNetworksStepReducer.test.js | 128 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/__snapshots__/MappingWizardDatastoresStepReducer.test.js.snap | 134 - app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/__snapshots__/MappingWizardNetworksStepActions.test.js.snap | 82 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/__snapshots__/MappingWizardNetworksStepReducer.test.js.snap | 234 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/mappingWizardNetworksStep.fixtures.js | 27 + app/javascript/react/screens/App/Mappings/screens/MappingWizard/mappingWizardTestHelpers.js | 13 + app/javascript/react/screens/App/common/testReduxHelpers.js | 11 + 24 files changed, 1097 insertions(+), 263 deletions(-) https://github.com/ManageIQ/manageiq-v2v/commit/0da8173ddae0b8707a177852a01eee93cc48dff6 commit 0da8173ddae0b8707a177852a01eee93cc48dff6 Author: Michael Ro <mikerodev> AuthorDate: Fri Dec 7 16:07:52 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Fri Dec 7 16:07:52 2018 -0500 Merge pull request #828 from mturley/vm-power-warning-tests Add tests for OSP VM power warnings in the Plan wizard (cherry picked from commit 1a1889b4a9542f6f1226c0bcccbd7dadd4bced36) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/__tests__/PlanWizardResultsStep.test.js | 35 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/__tests__/__snapshots__/PlanWizardResultsStep.test.js.snap | 64 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/PlanWizardScheduleStep.js | 2 +- app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/__tests__/PlanWizardScheduleStep.test.js | 19 + app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardScheduleStep/__tests__/__snapshots__/PlanWizardScheduleStep.test.js.snap | 77 + 5 files changed, 196 insertions(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-v2v/commit/029c2cfc1b30ba99af43e0d16021720812cbbaa5 commit 029c2cfc1b30ba99af43e0d16021720812cbbaa5 Author: Michael Ro <mikerodev> AuthorDate: Fri Dec 7 16:48:52 2018 -0500 Commit: Michael Ro <mikerodev> CommitDate: Fri Dec 7 16:48:52 2018 -0500 Merge pull request #829 from mturley/osp-conversion-host-warning-test Add tests for OSP conversion host warning in mapping wizard clusters step (cherry picked from commit b5112bf8d56e32bd09d9559c1ba2d86ffa400a2a) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStep.test.js | 26 +- 1 file changed, 25 insertions(+), 1 deletion(-) https://github.com/ManageIQ/manageiq-v2v/commit/328339ef27c6d8dab7d6b6f69cd08357e3663857 commit 328339ef27c6d8dab7d6b6f69cd08357e3663857 Author: Mike Turley <mike.turley.umass.edu> AuthorDate: Fri Dec 7 16:59:52 2018 -0500 Commit: Mike Turley <mike.turley.umass.edu> CommitDate: Fri Dec 7 16:59:52 2018 -0500 Merge pull request #827 from michaelkro/enhanced-error-messaging-tests Tests for enhanced error reporting (cherry picked from commit ad9eabf6d32c233ac18219cf978a1bda62232711) https://bugzilla.redhat.com/show_bug.cgi?id=1657285 app/javascript/react/screens/App/Overview/__test__/OverviewActions.test.js | 2 +- app/javascript/react/screens/App/Overview/__test__/OverviewSelectors.test.js | 4 +- app/javascript/react/screens/App/Overview/components/Migrations/__test__/MigrationsInProgressCard.test.js | 83 + app/javascript/react/screens/App/Overview/components/Migrations/__test__/__snapshots__/MigrationsInProgressCard.test.js.snap | 68 + app/javascript/react/screens/App/Overview/overview.transformationPlans.fixtures.js | 54 + 5 files changed, 208 insertions(+), 3 deletions(-)
I am assuming this bugzilla is just for tracking sake. I remember discussion when once QE got lots of UI issues that devel decided to start writing unit tests for v2v related to stuff. This is helpful since before this bugs reach to QE now this will get caught by unit test itself. I was really confused initially on how to test this thing, later just saw code is present on appliance and hence verifying it. Checked on - 5.10.0.30.20181218191323_900a416