Github user sanju1010 commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1377#discussion_r51681219
  
    --- Diff: test/integration/component/test_escalations_snapshots.py ---
    @@ -639,3 +671,47 @@ def test_04_list_vm_snapshots_byid(self):
                              "Listed VM Snapshot details are not as expected"
                              )
             return
    +
    +    @attr(tags=["advanced", "basic"], required_hardware="true")
    +    def test_05_check_vm_snapshot_creation_after_Instance_creation(self):
    +        """
    +        @summary: Test  if SS creation is successful in the first
    +        10 minutes of VM deployment
    +
    +        Step1: Create a VM with any Service offering
    +        Step2: Create a VM snapshot
    +        Step3: Verify if SS creation is successful within first
    +               10 minutes of VM deployment
    +        """
    +        if self.hypervisor.lower() not in ['vmware']:
    +            raise unittest.SkipTest("This test case is only for vmware. 
Hence, skipping the test")
    --- End diff --
    
    Can you replace with skipTest  to avoid raising an exception?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to