Passed
Pull Request — master (#156)
by Jan
09:48
created

oval_graph.command_line_client.client   A

Complexity

Total Complexity 31

Size/Duplication

Total Lines 201
Duplicated Lines 0 %

Test Coverage

Coverage 100%

Importance

Changes 0
Metric Value
eloc 142
dl 0
loc 201
ccs 91
cts 91
cp 1
rs 9.92
c 0
b 0
f 0
wmc 31

18 Methods

Rating   Name   Duplication   Size   Complexity  
A Client.parse_arguments() 0 3 1
A Client.search_rules_id() 0 5 1
A Client._get_rules() 0 4 2
A Client._get_choices() 0 4 2
A Client._get_message() 0 5 1
A Client.get_only_fail_rule() 0 6 1
A Client.prepare_data() 0 6 1
A Client.__init__() 0 15 1
A Client.get_questions() 0 13 1
A Client._get_list_of_lines() 0 15 4
A Client._get_list_of_matched_rules() 0 4 2
A Client.get_selection_rules() 0 2 1
A Client.prepare_args_when_user_can_list_in_rules() 0 11 1
B Client.prepare_parser() 0 39 1
A Client.run_gui_and_return_answers() 0 12 4
A Client._check_rules_id() 0 13 5
A Client._get_wanted_rules() 0 4 1
A Client._get_rows_of_unselected_rules() 0 5 1
1 1
import argparse
0 ignored issues
show
introduced by
Missing module docstring
Loading history...
2 1
import re
3 1
import sys
4 1
from datetime import datetime
5
6 1
from .. import __version__
7
8
9 1
class Client():
0 ignored issues
show
introduced by
Missing class docstring
Loading history...
best-practice introduced by
Too many instance attributes (12/7)
Loading history...
10 1
    def __init__(self, args):
11 1
        self.parser = None
12 1
        self.arg = self.parse_arguments(args)
13 1
        self.hide_passing_tests = self.arg.hide_passing_tests
14 1
        self.source_filename = self.arg.source_filename
15 1
        self.rule_name = self.arg.rule_id
16 1
        self.out = self.arg.output
17 1
        self.verbose = self.arg.verbose
18
19 1
        self.date = str(datetime.now().strftime("-%d_%m_%Y-%H_%M_%S"))
20 1
        self.isatty = sys.stdout.isatty()
21
22 1
        self.all_rules = self.arg.all
23 1
        self.show_failed_rules = False
24 1
        self.show_not_selected_rules = False
25
26 1
    @staticmethod
27
    def _get_message():
28 1
        return {
29
            'description': '',
30
            'source_filename': '',
31
        }
32
33
    #    Non-implemented functions can handle different types of the input file.
34
    #    Functions for input file processing and corresponding rule ID.
35
36 1
    def prepare_data(self, rules):
37
        """
38
        Function processes HTML graphs or JSON and
39
        return array of where is saved output file if exitsts.
40
        """
41 1
        raise NotImplementedError
42
43
    # Functions for selection of rules
44
45 1
    def search_rules_id(self):
46
        """
47
        Function retunes array of all matched IDs of rules in selected file.
48
        """
49 1
        raise NotImplementedError
50
51 1
    def get_only_fail_rule(self, rules):
52
        """
53
        Function processes array of matched IDs of rules in selected file.
54
        Function retunes array of failed matched IDs of rules in selected file.
55
        """
56 1
        raise NotImplementedError
57
58 1
    def _get_rows_of_unselected_rules(self):
59
        """
60
        Function retunes array of rows where is not selected IDs of rules in selected file.
61
        """
62 1
        raise NotImplementedError
63
64 1
    def run_gui_and_return_answers(self):
0 ignored issues
show
introduced by
Missing function or method docstring
Loading history...
65 1
        if self.isatty:
66 1
            if self.all_rules:
67 1
                return self._get_rules()
68
69 1
            try:
70 1
                import inquirer
0 ignored issues
show
introduced by
Import outside toplevel (inquirer)
Loading history...
71 1
                return inquirer.prompt(self.get_questions())
72 1
            except ImportError:
73 1
                print(self.get_selection_rules())
74 1
            return None
75 1
        return self._get_rules()
76
77 1
    def _get_rules(self):
78 1
        if self.show_failed_rules:
79 1
            return {'rules': self.get_only_fail_rule(self.search_rules_id())}
80 1
        return {'rules': self.search_rules_id()}
81
82 1
    def _get_list_of_matched_rules(self):
83 1
        if self.show_failed_rules:
84 1
            return self.get_only_fail_rule(self.search_rules_id())
85 1
        return self.search_rules_id()
86
87 1
    def _get_list_of_lines(self):
88 1
        lines = ['== The Rule ID regular expressions ==']
89 1
        for rule in self._get_list_of_matched_rules():
90 1
            lines.append("^" + rule + "$")
91 1
        if self.show_not_selected_rules:
92 1
            for line in self._get_rows_of_unselected_rules():
93 1
                lines.append(line)
94 1
        lines.append(
95
            "Interactive rule selection is not available,"
96
            " because inquirer is not installed."
97
            " Copy id of the rule you want to visualize and"
98
            " paste it into a command with regular"
99
            " expression characters(^$).\n"
100
            "Alternatively, use the --all or --all-in-one arguments.")
101 1
        return lines
102
103 1
    def get_selection_rules(self):
0 ignored issues
show
introduced by
Missing function or method docstring
Loading history...
104 1
        return "\n".join(self._get_list_of_lines())
105
106 1
    def _get_choices(self):
107 1
        if self.show_not_selected_rules:
108 1
            print("\n".join(self._get_rows_of_unselected_rules()))
109 1
        return self._get_list_of_matched_rules()
110
111 1
    def get_questions(self):
0 ignored issues
show
introduced by
Missing function or method docstring
Loading history...
112 1
        from inquirer.questions import Checkbox as checkbox
0 ignored issues
show
introduced by
Import outside toplevel (inquirer.questions.Checkbox)
Loading history...
113 1
        choices = self._get_choices()
114 1
        questions = [
115
            checkbox(
116
                'rules',
117
                message=(
118
                    "= The Rules IDs = (move - UP and DOWN arrows,"
119
                    " select - SPACE or LEFT and RIGHT arrows, submit - ENTER)"),
120
                choices=choices,
121
            ),
122
        ]
123 1
        return questions
124
125 1
    def _get_wanted_rules(self, rules):
126 1
        return [
127
            x for x in rules if re.search(
128
                self.rule_name, x)]
129
130 1
    def _check_rules_id(self, rules, notselected_rules):
131 1
        if notselected_rules and not rules:
132 1
            raise ValueError(
133
                ('Rule(s) "{}" was not selected, '
134
                 "so there are no results. The rule is"
135
                 ' "notselected" because it'
136
                 " wasn't a part of the executed profile"
137
                 " and therefore it wasn't evaluated "
138
                 "during the scan.")
139
                .format(notselected_rules))
140 1
        if not notselected_rules and not rules:
141 1
            raise ValueError('404 rule "{}" not found!'.format(self.rule_name))
142 1
        return rules
143
144
    # Function for setting arguments
145
146 1
    def parse_arguments(self, args):
0 ignored issues
show
introduced by
Missing function or method docstring
Loading history...
147 1
        self.prepare_parser()
148 1
        return self.parser.parse_args(args)
149
150 1
    def prepare_args_when_user_can_list_in_rules(self):
0 ignored issues
show
Coding Style Naming introduced by
Method name "prepare_args_when_user_can_list_in_rules" doesn't conform to '[a-z_][a-z0-9_]2,30$' pattern ('[a-z_][a-z0-9_]2,30$' pattern)

This check looks for invalid names for a range of different identifiers.

You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements.

If your project includes a Pylint configuration file, the settings contained in that file take precedence.

To find out more about Pylint, please refer to their site.

Loading history...
introduced by
Missing function or method docstring
Loading history...
151 1
        self.parser.add_argument(
152
            '--show-failed-rules',
153
            action="store_true",
154
            default=False,
155
            help="Show only FAILED rules")
156 1
        self.parser.add_argument(
157
            '--show-not-selected-rules',
158
            action="store_true",
159
            default=False,
160
            help="Show notselected rules. These rules will not be visualized.")
161
162 1
    def prepare_parser(self):
0 ignored issues
show
introduced by
Missing function or method docstring
Loading history...
163 1
        self.parser = argparse.ArgumentParser(
164
            prog='oval-graph',
165
            description=self._get_message().get('description'))
166 1
        self.parser.add_argument(
167
            '--version',
168
            action='version',
169
            version='%(prog)s ' + __version__)
170 1
        self.parser.add_argument(
171
            '-a',
172
            '--all',
173
            action="store_true",
174
            default=False,
175
            help="Process all matched rules.")
176 1
        self.parser.add_argument(
177
            '--hide-passing-tests',
178
            action="store_true",
179
            default=False,
180
            help=(
181
                "Do not display passing tests for better orientation in"
182
                " graphs that contain a large amount of nodes."))
183 1
        self.parser.add_argument(
184
            '-v',
185
            '--verbose',
186
            action="store_true",
187
            default=False,
188
            help="Displays details about the results of the running command.")
189 1
        self.parser.add_argument(
190
            '-o',
191
            '--output',
192
            action="store",
193
            default=None,
194
            help='The file where to save output.')
195 1
        self.parser.add_argument(
196
            "source_filename",
197
            help=self._get_message().get('source_filename'))
198 1
        self.parser.add_argument(
199
            "rule_id", help=(
200
                "Rule ID to be visualized. A part from the full rule ID"
201
                " a part of the ID or a regular expression can be used."
202
                " If brackets are used in the regular expression "
203
                "the regular expression must be quoted."))
204