Code Quality - included B101 in tox.ini to ignore assert warning
This commit is contained in:
parent
b43c9c4a75
commit
c2e407ef5a
|
@ -45,7 +45,7 @@ def sqlQuery(sql_statement, *args):
|
||||||
:param list args: SQL query parameters
|
:param list args: SQL query parameters
|
||||||
:rtype: list
|
:rtype: list
|
||||||
"""
|
"""
|
||||||
assert sql_available # nosec:B101
|
assert sql_available
|
||||||
sql_lock.acquire()
|
sql_lock.acquire()
|
||||||
sqlSubmitQueue.put(sql_statement)
|
sqlSubmitQueue.put(sql_statement)
|
||||||
|
|
||||||
|
@ -65,7 +65,7 @@ def sqlExecuteChunked(sql_statement, idCount, *args):
|
||||||
"""Execute chunked SQL statement to avoid argument limit"""
|
"""Execute chunked SQL statement to avoid argument limit"""
|
||||||
# SQLITE_MAX_VARIABLE_NUMBER,
|
# SQLITE_MAX_VARIABLE_NUMBER,
|
||||||
# unfortunately getting/setting isn't exposed to python
|
# unfortunately getting/setting isn't exposed to python
|
||||||
assert sql_available # nosec:B101
|
assert sql_available
|
||||||
sqlExecuteChunked.chunkSize = 999
|
sqlExecuteChunked.chunkSize = 999
|
||||||
|
|
||||||
if idCount == 0 or idCount > len(args):
|
if idCount == 0 or idCount > len(args):
|
||||||
|
@ -95,7 +95,7 @@ def sqlExecuteChunked(sql_statement, idCount, *args):
|
||||||
|
|
||||||
def sqlExecute(sql_statement, *args):
|
def sqlExecute(sql_statement, *args):
|
||||||
"""Execute SQL statement (optionally with arguments)"""
|
"""Execute SQL statement (optionally with arguments)"""
|
||||||
assert sql_available # nosec:B101
|
assert sql_available
|
||||||
sql_lock.acquire()
|
sql_lock.acquire()
|
||||||
sqlSubmitQueue.put(sql_statement)
|
sqlSubmitQueue.put(sql_statement)
|
||||||
|
|
||||||
|
@ -120,7 +120,7 @@ def sqlExecuteScript(sql_statement):
|
||||||
|
|
||||||
def sqlStoredProcedure(procName):
|
def sqlStoredProcedure(procName):
|
||||||
"""Schedule procName to be run"""
|
"""Schedule procName to be run"""
|
||||||
assert sql_available # nosec:B101
|
assert sql_available
|
||||||
sql_lock.acquire()
|
sql_lock.acquire()
|
||||||
sqlSubmitQueue.put(procName)
|
sqlSubmitQueue.put(procName)
|
||||||
if procName == "exit":
|
if procName == "exit":
|
||||||
|
@ -143,7 +143,7 @@ class SqlBulkExecute(object):
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def execute(sql_statement, *args):
|
def execute(sql_statement, *args):
|
||||||
"""Used for statements that do not return results."""
|
"""Used for statements that do not return results."""
|
||||||
assert sql_available # nosec:B101
|
assert sql_available
|
||||||
sqlSubmitQueue.put(sql_statement)
|
sqlSubmitQueue.put(sql_statement)
|
||||||
|
|
||||||
if args == ():
|
if args == ():
|
||||||
|
|
2
tox.ini
2
tox.ini
|
@ -21,7 +21,7 @@ deps =
|
||||||
bandit
|
bandit
|
||||||
flake8
|
flake8
|
||||||
commands =
|
commands =
|
||||||
bandit -r --exit-zero -s B105,B301,B411,B413,B608 \
|
bandit -r --exit-zero -s B105,B301,B411,B413,B608,B101 \
|
||||||
-x checkdeps.*,bitmessagecurses,bitmessageqt,tests pybitmessage
|
-x checkdeps.*,bitmessagecurses,bitmessageqt,tests pybitmessage
|
||||||
flake8 pybitmessage --count --select=E9,F63,F7,F82 \
|
flake8 pybitmessage --count --select=E9,F63,F7,F82 \
|
||||||
--show-source --statistics
|
--show-source --statistics
|
||||||
|
|
Reference in New Issue
Block a user