foreman_snapshot_management 2.0.1 → 2.0.3

data/test/controllers/api/v2/snapshots_test.rb

@@ -12,13 +12,17 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
   let(:uuid) { '5032c8a5-9c5e-ba7a-3804-832a03e16381' }
   let(:host) { FactoryBot.create(:host, :managed, :compute_resource => compute_resource, :uuid => uuid) }
   let(:snapshot_id) { 'snapshot-0101' }
-  let(:proxmox_compute_resource) do
-    cr = FactoryBot.create(:proxmox_cr)
-    ComputeResource.find_by(id: cr.id)
+
+  if defined?(TEST_PROXMOX)
+    let(:vmid) { '100' }
+    let(:proxmox_compute_resource) do
+      cr = FactoryBot.create(:proxmox_cr)
+      ComputeResource.find_by(id: cr.id)
+    end
+    let(:proxmox_host) { FactoryBot.create(:host, :managed, :compute_resource => proxmox_compute_resource, :uuid => "1_#{vmid}") }
+    let(:proxmox_snapshot) { 'snapshot1' }
   end
-  let(:vmid) { '100' }
-  let(:proxmox_host) { FactoryBot.create(:host, :managed, :compute_resource => proxmox_compute_resource, :uuid => "1_#{vmid}") }
-  let(:proxmox_snapshot) { 'snapshot1' }
+
   let(:manager_user) do
     roles = [Role.find_by(name: 'Snapshot Manager')]
     FactoryBot.create(:user, :organizations => [tax_organization], :locations => [tax_location], :roles => roles)
@@ -34,7 +38,7 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
     setup do
       # Make sure that test-hosts are created here (by the admin-user)
       host
-      proxmox_host
+      proxmox_host if defined?(TEST_PROXMOX)
       @orig_user = User.current
       User.current = view_user
     end
@@ -51,14 +55,16 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_not_empty body['results']
     end
 
-    test 'should get index of Proxmox Snapshots' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :index, params: { :host_id => proxmox_host.to_param }
-      assert_response :success
-      assert_not_nil assigns(:snapshots)
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
-      assert_not_empty body['results']
+    if defined?(TEST_PROXMOX)
+      test 'should get index of Proxmox Snapshots' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :index, params: { :host_id => proxmox_host.to_param }
+        assert_response :success
+        assert_not_nil assigns(:snapshots)
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+        assert_not_empty body['results']
+      end
     end
 
     test 'should search VMware snapshot' do
@@ -71,15 +77,17 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_equal body['results'].count, 1
     end
 
-    test 'should search Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :index, params: { :host_id => proxmox_host.to_param, :search => 'name= snapshot1' }
-      assert_response :success
-      assert_not_nil assigns(:snapshots)
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
-      assert_not_empty body['results']
-      assert_equal body['results'].count, 1
+    if defined?(TEST_PROXMOX)
+      test 'should search Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :index, params: { :host_id => proxmox_host.to_param, :search => 'name= snapshot1' }
+        assert_response :success
+        assert_not_nil assigns(:snapshots)
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+        assert_not_empty body['results']
+        assert_equal body['results'].count, 1
+      end
     end
 
     test 'should refute search Vmware snapshot' do
@@ -95,13 +103,15 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_not_empty body
     end
 
-    test 'should show Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :show, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }
-      assert_not_nil assigns(:snapshot)
-      assert_response :success
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
+    if defined?(TEST_PROXMOX)
+      test 'should show Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :show, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }
+        assert_not_nil assigns(:snapshot)
+        assert_response :success
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+      end
     end
 
     test 'should 404 for unknown Vmware snapshot' do
@@ -125,10 +135,12 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_response :forbidden
     end
 
-    test 'should refute revert Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
-      assert_response :forbidden
+    if defined?(TEST_PROXMOX)
+      test 'should refute revert Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
+        assert_response :forbidden
+      end
     end
   end
 
@@ -136,7 +148,7 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
     setup do
       # Make sure that test-hosts are created here (by the admin-user)
       host
-      proxmox_host
+      proxmox_host if defined?(TEST_PROXMOX)
       @orig_user = User.current
       User.current = manager_user
     end
@@ -153,14 +165,16 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_not_empty body['results']
     end
 
-    test 'should get index of Proxmox Snapshots' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :index, params: { :host_id => proxmox_host.to_param }
-      assert_response :success
-      assert_not_nil assigns(:snapshots)
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
-      assert_not_empty body['results']
+    if defined?(TEST_PROXMOX)
+      test 'should get index of Proxmox Snapshots' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :index, params: { :host_id => proxmox_host.to_param }
+        assert_response :success
+        assert_not_nil assigns(:snapshots)
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+        assert_not_empty body['results']
+      end
     end
 
     test 'should search VMware snapshot' do
@@ -173,15 +187,17 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_equal body['results'].count, 1
     end
 
-    test 'should search Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :index, params: { :host_id => proxmox_host.to_param, :search => 'name= snapshot1' }
-      assert_response :success
-      assert_not_nil assigns(:snapshots)
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
-      assert_not_empty body['results']
-      assert_equal body['results'].count, 1
+    if defined?(TEST_PROXMOX)
+      test 'should search Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :index, params: { :host_id => proxmox_host.to_param, :search => 'name= snapshot1' }
+        assert_response :success
+        assert_not_nil assigns(:snapshots)
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+        assert_not_empty body['results']
+        assert_equal body['results'].count, 1
+      end
     end
 
     test 'should refute search Vmware snapshot' do
@@ -197,13 +213,15 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_not_empty body
     end
 
-    test 'should show Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      get :show, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }
-      assert_not_nil assigns(:snapshot)
-      assert_response :success
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_empty body
+    if defined?(TEST_PROXMOX)
+      test 'should show Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        get :show, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }
+        assert_not_nil assigns(:snapshot)
+        assert_response :success
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_empty body
+      end
     end
 
     test 'should 404 for unknown Vmware snapshot' do
@@ -217,11 +235,13 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_not_nil assigns(:snapshot)
     end
 
-    test 'should create Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      post :create, params: { :host_id => proxmox_host.to_param, :name => 'test' }
-      assert_response :created
-      assert_not_nil assigns(:snapshot)
+    if defined?(TEST_PROXMOX)
+      test 'should create Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        post :create, params: { :host_id => proxmox_host.to_param, :name => 'test' }
+        assert_response :created
+        assert_not_nil assigns(:snapshot)
+      end
     end
 
     test 'should update Vmware snapshot' do
@@ -230,18 +250,20 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_response :success
     end
 
-    test 'should update Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      description = 'snapshot1 updated'
-      put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param, :description => description.to_param }
-      assert_response :success
-    end
+    if defined?(TEST_PROXMOX)
+      test 'should update Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        description = 'snapshot1 updated'
+        put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param, :description => description.to_param }
+        assert_response :success
+      end
 
-    test 'should refute update Proxmox snapshot name' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      name = 'test'
-      put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param, :name => name.to_param }
-      assert_response :unprocessable_entity
+      test 'should refute update Proxmox snapshot name' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        name = 'test'
+        put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param, :name => name.to_param }
+        assert_response :unprocessable_entity
+      end
     end
 
     test 'should destroy Vmware snapshot' do
@@ -249,24 +271,26 @@ class Api::V2::SnapshotsControllerTest < ActionController::TestCase
       assert_response :success
     end
 
-    test 'should destroy Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      delete :destroy, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
-      assert_response :success
-      body = ActiveSupport::JSON.decode(@response.body)
-      assert_not_nil body['name']
-      assert_nil body['id']
-    end
-
     test 'should revert Vmware snapshot' do
       put :revert, params: { :host_id => host.to_param, :id => snapshot_id.to_param }
       assert_response :success
     end
 
-    test 'should revert Proxmox snapshot' do
-      Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-      put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
-      assert_response :success
+    if defined?(TEST_PROXMOX)
+      test 'should destroy Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        delete :destroy, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
+        assert_response :success
+        body = ActiveSupport::JSON.decode(@response.body)
+        assert_not_nil body['name']
+        assert_nil body['id']
+      end
+
+      test 'should revert Proxmox snapshot' do
+        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+        put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot.to_param }
+        assert_response :success
+      end
     end
   end
 end

data/test/controllers/foreman_snapshot_management/snapshots_controller_test.rb

@@ -11,17 +11,21 @@ module ForemanSnapshotManagement
       ComputeResource.find_by(id: cr.id)
     end
     let(:uuid) { '5032c8a5-9c5e-ba7a-3804-832a03e16381' }
-    let(:uuid2) { 'a7169e20-74d3-4367-afc2-d355716e7555' }
+    let(:uuid2) { '502916a3-b42e-17c7-43ce-b3206e9524dc' }
     let(:host) { FactoryBot.create(:host, :managed, :compute_resource => compute_resource, :uuid => uuid) }
     let(:host2) { FactoryBot.create(:host, :managed, :compute_resource => compute_resource, :uuid => uuid2) }
     let(:snapshot_id) { 'snapshot-0101' }
-    let(:proxmox_compute_resource) do
-      FactoryBot.create(:proxmox_cr)
-      ComputeResource.find_by(type: 'ForemanFogProxmox::Proxmox')
+
+    if defined?(TEST_PROXMOX)
+      let(:vmid) { '1_100' }
+      let(:proxmox_compute_resource) do
+        FactoryBot.create(:proxmox_cr)
+        ComputeResource.find_by(type: 'ForemanFogProxmox::Proxmox')
+      end
+      let(:proxmox_host) { FactoryBot.create(:host, :managed, :compute_resource => proxmox_compute_resource, :uuid => vmid) }
+      let(:proxmox_snapshot) { 'snapshot1' }
     end
-    let(:vmid) { '1_100' }
-    let(:proxmox_host) { FactoryBot.create(:host, :managed, :compute_resource => proxmox_compute_resource, :uuid => vmid) }
-    let(:proxmox_snapshot) { 'snapshot1' }
+
     setup { ::Fog.mock! }
     teardown { ::Fog.unmock! }
 
@@ -33,12 +37,14 @@ module ForemanSnapshotManagement
         assert_template 'foreman_snapshot_management/snapshots/_index'
       end
 
-      test 'should get Proxmox index' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        get :index, params: { :host_id => proxmox_host.to_param }, session: set_session_user
-        assert_response :success
-        assert_not_nil assigns(:snapshots)
-        assert_template 'foreman_snapshot_management/snapshots/_index'
+      if defined?(TEST_PROXMOX)
+        test 'should get Proxmox index' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          get :index, params: { :host_id => proxmox_host.to_param }, session: set_session_user
+          assert_response :success
+          assert_not_nil assigns(:snapshots)
+          assert_template 'foreman_snapshot_management/snapshots/_index'
+        end
       end
     end
 
@@ -49,15 +55,32 @@ module ForemanSnapshotManagement
         assert_includes flash[:notice] || flash[:success], 'Successfully created Snapshot.'
       end
 
-      test 'create valid proxmox snapshot' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        post :create, params: { :host_id => proxmox_host.to_param, :snapshot => { :name => 'test' } }, session: set_session_user
-        assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
-        assert_includes flash[:notice] || flash[:success], 'Successfully created Snapshot.'
+      if defined?(TEST_PROXMOX)
+        test 'create valid proxmox snapshot' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          post :create, params: { :host_id => proxmox_host.to_param, :snapshot => { :name => 'test' } }, session: set_session_user
+          assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
+          assert_includes flash[:notice] || flash[:success], 'Successfully created Snapshot.'
+        end
+      end
+
+      test 'create valid multiple with memory' do
+        post :create_multiple_host, params: { :host_ids => [host.id, host2.id], :snapshot => { :name => 'test', :snapshot_mode => 'memory' } }, session: set_session_user
+        puts @response
+        assert_redirected_to hosts_url
+        assert_includes flash[:notice] || flash[:success], 'Created Snapshots for 2 hosts'
       end
 
-      test 'create valid multiple' do
+      test 'create valid multiple with quiesce' do
+        post :create_multiple_host, params: { :host_ids => [host.id], :snapshot => { :name => 'test', :snapshot_mode => 'quiesce' } }, session: set_session_user
+        puts @response
+        assert_redirected_to hosts_url
+        assert_includes flash[:notice] || flash[:success], 'Created Snapshot for 1 host'
+      end
+
+      test 'create valid multiple without specific mode' do
         post :create_multiple_host, params: { :host_ids => [host.id, host2.id], :snapshot => { :name => 'test' } }, session: set_session_user
+        puts @response
         assert_redirected_to hosts_url
         assert_includes flash[:notice] || flash[:success], 'Created Snapshots for 2 hosts'
       end
@@ -84,11 +107,13 @@ module ForemanSnapshotManagement
         assert_includes flash[:notice] || flash[:success], 'Successfully deleted Snapshot.'
       end
 
-      test 'destroy successful' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        delete :destroy, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }, session: set_session_user
-        assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
-        assert_includes flash[:notice] || flash[:success], 'Successfully deleted Snapshot.'
+      if defined?(TEST_PROXMOX)
+        test 'destroy successful' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          delete :destroy, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }, session: set_session_user
+          assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
+          assert_includes flash[:notice] || flash[:success], 'Successfully deleted Snapshot.'
+        end
       end
 
       test 'destroy with error' do
@@ -106,11 +131,13 @@ module ForemanSnapshotManagement
         assert_includes flash[:notice] || flash[:success], 'VM successfully rolled back.'
       end
 
-      test 'revert successful proxmox snapshot' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }, session: set_session_user
-        assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
-        assert_includes flash[:notice] || flash[:success], 'VM successfully rolled back.'
+      if defined?(TEST_PROXMOX)
+        test 'revert successful proxmox snapshot' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          put :revert, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot }, session: set_session_user
+          assert_redirected_to host_url(proxmox_host, :anchor => 'snapshots')
+          assert_includes flash[:notice] || flash[:success], 'VM successfully rolled back.'
+        end
       end
 
       test 'revert with error' do
@@ -130,13 +157,15 @@ module ForemanSnapshotManagement
         assert_equal(data, body)
       end
 
-      test 'update successful proxmox' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        data = { 'name' => 'snapshot1', 'description' => 'updated snapshot1' }
-        put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot, :snapshot => data }, session: set_session_user
-        assert_response :success
-        body = ActiveSupport::JSON.decode(@response.body)
-        assert_equal(data, body)
+      if defined?(TEST_PROXMOX)
+        test 'update successful proxmox' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          data = { 'name' => 'snapshot1', 'description' => 'updated snapshot1' }
+          put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot, :snapshot => data }, session: set_session_user
+          assert_response :success
+          body = ActiveSupport::JSON.decode(@response.body)
+          assert_equal(data, body)
+        end
       end
 
       test 'update with error VMware snapshot' do
@@ -145,11 +174,13 @@ module ForemanSnapshotManagement
         assert_response :unprocessable_entity
       end
 
-      test 'update with error proxmox snapshot' do
-        Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
-        ForemanSnapshotManagement::Snapshot.any_instance.stubs(:save).returns(false)
-        put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot, :snapshot => { :name => 'snapshot1', :description => 'fail' } }, session: set_session_user
-        assert_response :unprocessable_entity
+      if defined?(TEST_PROXMOX)
+        test 'update with error proxmox snapshot' do
+          Host::Managed.any_instance.stubs(:vm_exists?).returns(false)
+          ForemanSnapshotManagement::Snapshot.any_instance.stubs(:save).returns(false)
+          put :update, params: { :host_id => proxmox_host.to_param, :id => proxmox_snapshot, :snapshot => { :name => 'snapshot1', :description => 'fail' } }, session: set_session_user
+          assert_response :unprocessable_entity
+        end
       end
     end
   end

data/webpack/components/SnapshotManagement/SnapshotManagement.js

@@ -67,6 +67,7 @@ SnapshotManagement.propTypes = {
   capabilities: PropTypes.shape({
     editSnapshotName: PropTypes.bool,
     limitSnapshotNameFormat: PropTypes.bool,
+    quiesceOption: PropTypes.bool,
   }),
 };
 
@@ -78,6 +79,7 @@ SnapshotManagement.defaultProps = {
   capabilities: {
     editSnapshotName: true,
     limitSnapshotNameFormat: false,
+    quiesceOption: false,
   },
 };
 

data/webpack/components/SnapshotManagement/SnapshotManagementActions.js

@@ -1,6 +1,6 @@
 import { API, actionTypeGenerator } from 'foremanReact/redux/API';
 import { sprintf, translate as __ } from 'foremanReact/common/I18n';
-import { addToast } from 'foremanReact/redux/actions/toasts';
+import { addToast } from 'foremanReact/components/ToastsList';
 
 import {
   SNAPSHOT_LIST,

data/webpack/components/SnapshotManagement/__tests__/__snapshots__/SnapshotManagementActions.test.js.snap

@@ -62,13 +62,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_DELETE_SUCCESS",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_DELETE_SUCCESS",
           "message": "Successfully removed Snapshot \\"Savegame\\" from host deep.thought",
           "type": "success",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [
@@ -108,13 +108,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_DELETE_FAILURE",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_DELETE_FAILURE",
           "message": "Error occurred while removing Snapshot: Error: some-error",
           "type": "error",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [
@@ -151,13 +151,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_ROLLBACK_FAILURE",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_ROLLBACK_FAILURE",
           "message": "Error occurred while rolling back VM: Error: some-error",
           "type": "error",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [
@@ -194,13 +194,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_ROLLBACK_SUCCESS",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_ROLLBACK_SUCCESS",
           "message": "Successfully rolled back Snapshot \\"Savegame\\" on host deep.thought",
           "type": "success",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [
@@ -241,13 +241,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_UPDATE_FAILURE",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_UPDATE_FAILURE",
           "message": "Error occurred while updating Snapshot: Error: some-error",
           "type": "error",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [
@@ -288,13 +288,13 @@ Array [
     Object {
       "payload": Object {
         "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_UPDATE_SUCCESS",
-        "message": Object {
+        "toast": Object {
           "key": "FOREMAN_SNAPSHOT_MANAGEMENT_SNAPSHOT_UPDATE_SUCCESS",
           "message": "Successfully updated Snapshot \\"Savegame\\"",
           "type": "success",
         },
       },
-      "type": "TOASTS_ADD",
+      "type": "toasts/addToast",
     },
   ],
   Array [