Solved comments, added validations for test and added unit test for image manager

Author: edsonarios

tests/CLI/modules/image_tests.py

@@ -57,11 +57,14 @@ def test_datacenter_remove_fails(self):
     def test_share(self):
         result = self.run_command(['image', 'share', '123456', '--account-id', '654321'])
         self.assert_no_fail(result)
+        self.assertIn("Image template 123456 was shared to account 654321.", result.output)
 
     def test_share_without_id(self):
         result = self.run_command(['image', 'share'])
         self.assertEqual(2, result.exit_code)
+        self.assertIn("Error: Missing argument 'IDENTIFIER'.", result.output)
 
     def test_share_without_id_account(self):
         result = self.run_command(['image', 'share', "123456"])
         self.assertEqual(2, result.exit_code)
+        self.assertIn("Error: Missing option '--account-id'.", result.output)

tests/managers/image_tests.py

@@ -223,3 +223,9 @@ def test_get_locations_id_fails(self):
             100,
             locations
         )
+
+    def test_share_image(self):
+        result = self.image.share_image(image_id=123456, account_id=654321)
+
+        self.assert_called_with(IMAGE_SERVICE, 'permitSharingAccess', identifier=123456, args=(654321,))
+        self.assertEqual(True, result)