Updated copyright for 2025.
10116
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
1
|
# -*- coding: utf-8 -*- |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
2
|
|
11090
|
3
|
# Copyright (c) 2023 - 2025 Detlev Offenbach <detlev@die-offenbachs.de> |
10116
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
4
|
# |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
5
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
6
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
7
|
Module implementing a checker for "async" related issues. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
8
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
9
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
10
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11
|
import copy |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14
|
class AsyncChecker: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
15
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
16
|
Class implementing a checker for "async" related issues. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
17
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
19
|
Codes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
20
|
"ASY100", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
21
|
"ASY101", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22
|
"ASY102", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
23
|
"ASY103", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
24
|
"ASY104", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
25
|
"ASY105", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
26
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28
|
def __init__(self, source, filename, tree, select, ignore, expected, repeat, args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
29
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30
|
Constructor |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
31
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32
|
@param source source code to be checked |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
34
|
@param filename name of the source file |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
35
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36
|
@param tree AST tree of the source code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37
|
@type ast.Module |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
38
|
@param select list of selected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40
|
@param ignore list of codes to be ignored |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42
|
@param expected list of expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44
|
@param repeat flag indicating to report each occurrence of a code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45
|
@type bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46
|
@param args dictionary of arguments for the various checks |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47
|
@type dict |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49
|
self.__select = tuple(select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50
|
self.__ignore = ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51
|
("",) if select else tuple(x for x in ignore if x.startswith("ASY")) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53
|
self.__expected = expected[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54
|
self.__repeat = repeat |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55
|
self.__filename = filename |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56
|
self.__source = source[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57
|
self.__tree = copy.deepcopy(tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
58
|
self.__args = args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60
|
# statistics counters |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61
|
self.counters = {} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63
|
# collection of detected errors |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64
|
self.errors = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66
|
checkersWithCodes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
67
|
( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
68
|
self.__checkSyncUses, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69
|
("ASY100", "ASY101", "ASY102", "ASY103", "ASY104", "ASY105"), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70
|
), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73
|
self.__checkers = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74
|
for checker, codes in checkersWithCodes: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75
|
if any(not (code and self.__ignoreCode(code)) for code in codes): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76
|
self.__checkers.append(checker) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78
|
def __ignoreCode(self, code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80
|
Private method to check if the message code should be ignored. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82
|
@param code message code to check for |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84
|
@return flag indicating to ignore the given code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85
|
@rtype bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87
|
return code.startswith(self.__ignore) and not code.startswith(self.__select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
88
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
89
|
def __error(self, lineNumber, offset, code, *args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91
|
Private method to record an issue. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93
|
@param lineNumber line number of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95
|
@param offset position within line of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
96
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
97
|
@param code message code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
99
|
@param args arguments for the message |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
100
|
@type list |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
102
|
if self.__ignoreCode(code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
103
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
104
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
105
|
if code in self.counters: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
106
|
self.counters[code] += 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
107
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
108
|
self.counters[code] = 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
109
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
110
|
# Don't care about expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
111
|
if code in self.__expected: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
112
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
113
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
114
|
if code and (self.counters[code] == 1 or self.__repeat): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
115
|
# record the issue with one based line number |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
116
|
self.errors.append( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
117
|
{ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
118
|
"file": self.__filename, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
119
|
"line": lineNumber + 1, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
120
|
"offset": offset, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
121
|
"code": code, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
122
|
"args": args, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
123
|
} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
124
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
125
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
126
|
def run(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
127
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
128
|
Public method to check the given source against miscellaneous |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
129
|
conditions. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
130
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
131
|
if not self.__filename: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
132
|
# don't do anything, if essential data is missing |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
133
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
134
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
135
|
if not self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
136
|
# don't do anything, if no codes were selected |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
137
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
138
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
139
|
for check in self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
140
|
check() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
141
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
142
|
def __checkSyncUses(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
143
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
144
|
Private method to check for use of synchroneous functions in async methods. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
145
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
146
|
from .AsyncVisitor import AsyncVisitor |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
147
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
148
|
visitor = AsyncVisitor(self.__args, self) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
149
|
visitor.visit(copy.deepcopy(self.__tree)) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
150
|
for violation in visitor.violations: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
151
|
if not self.__ignoreCode(violation[1]): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
152
|
node = violation[0] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
153
|
reason = violation[1] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
154
|
self.__error(node.lineno - 1, node.col_offset, reason) |