Completed
Push — master ( f419ff...edacad )
by Manas
02:43
created

GetVMUUID.run()   C

Complexity

Conditions 11

Size

Total Lines 17

Duplication

Lines 0
Ratio 0 %
Metric Value
cc 11
dl 0
loc 17
rs 5.5714

How to fix   Complexity   

Complexity

Complex classes like GetVMUUID.run() often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

1
# Licensed to the StackStorm, Inc ('StackStorm') under one or more
2
# contributor license agreements.  See the NOTICE file distributed with
3
# this work for additional information regarding copyright ownership.
4
# The ASF licenses this file to You under the Apache License, Version 2.0
5
# (the "License"); you may not use this file except in compliance with
6
# the License.  You may obtain a copy of the License at
7
#
8
#     http://www.apache.org/licenses/LICENSE-2.0
9
#
10
# Unless required by applicable law or agreed to in writing, software
11
# distributed under the License is distributed on an "AS IS" BASIS,
12
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13
# See the License for the specific language governing permissions and
14
# limitations under the License.
15
16
from vmwarelib import inventory
17
from vmwarelib.actions import BaseAction
18
19
20
class GetVMUUID(BaseAction):
21
22
    def run(self, vm_ids, vm_names):
23
        results = []
24
        if not vm_ids and not vm_names:
25
            raise Exception("No ID nor Names provided")
26
        if vm_ids:
27
            for vm in vm_ids:
28
                vm = inventory.get_virtualmachine(self.si_content, moid=vm)
29
                if vm:
30
                    if vm.summary.config.uuid not in results:
31
                        results.append(vm.summary.config.uuid)
32
        if vm_names:
33
            for vm in vm_names:
34
                vm = inventory.get_virtualmachine(self.si_content, name=vm)
35
                if vm:
36
                    if vm.summary.config.uuid not in results:
37
                        results.append(vm.summary.config.uuid)
38
        return results
39