added message for empty migrations

Author: allmightyspiff

SoftLayer/CLI/virt/migrate.py

@@ -60,6 +60,8 @@ def cli(env, guest, migrate_all, host):
     # Migrate all guests with pendingMigrationFlag=True
     elif migrate_all:
         require_migration = vsi.list_instances(filter=pending_filter, mask="mask[id]")
+        if not require_migration:
+            click.secho("No guests require migration at this time", fg='green')
         for vsi_object in require_migration:
             migrate(vsi, vsi_object['id'])
     # Just migrate based on the options

tests/CLI/modules/vs/vs_tests.py

@@ -836,6 +836,16 @@ def test_vs_migrate_all(self):
         self.assert_called_with('SoftLayer_Virtual_Guest', 'migrate', identifier=104)
         self.assert_not_called_with('SoftLayer_Virtual_Guest', 'migrateDedicatedHost')
 
+    def test_vs_migrate_all_empty(self):
+        mock = self.set_mock('SoftLayer_Account', 'getVirtualGuests')
+        mock.return_value = []
+        result = self.run_command(['vs', 'migrate', '-a'])
+        self.assert_no_fail(result)
+        self.assertIn('No guests require migration at this time', result.output)
+        self.assert_called_with('SoftLayer_Virtual_Guest', 'migrate', identifier=100)
+        self.assert_called_with('SoftLayer_Virtual_Guest', 'migrate', identifier=104)
+        self.assert_not_called_with('SoftLayer_Virtual_Guest', 'migrateDedicatedHost')
+
     def test_vs_migrate_dedicated(self):
         result = self.run_command(['vs', 'migrate', '-g', '100', '-h', '999'])
         self.assert_no_fail(result)