Yasuke/moto
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix some 'DeprecationWarning: invalid escape sequence' warnings and use str.format for string interpolation.

Browse Source 
Similar to https://github.com/spulec/moto/pull/2811
This commit is contained in:
Asher Foa 2020-03-25 11:07:59 -07:00
parent ef704852dd
commit 2e20ad14df
4 changed files with 18 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
moto/elbv2/models.py
View File

@ -582,11 +582,13 @@ class ELBv2Backend(BaseBackend):
report='Missing required parameter in Actions[%s].FixedResponseConfig: "StatusCode"' report='Missing required parameter in Actions[%s].FixedResponseConfig: "StatusCode"'
% i % i
) )
if not re.match(r"^(2|4|5)\d\d$", status_code): expression = r"^(2|4|5)\d\d$"
if not re.match(expression, status_code):
raise InvalidStatusCodeActionTypeError( raise InvalidStatusCodeActionTypeError(
"1 validation error detected: Value '%s' at 'actions.%s.member.fixedResponseConfig.statusCode' failed to satisfy constraint: \ "1 validation error detected: Value '{}' at 'actions.{}.member.fixedResponseConfig.statusCode' failed to satisfy constraint: \
Member must satisfy regular expression pattern: ^(2|4|5)\d\d$" Member must satisfy regular expression pattern: {}".format(
% (status_code, index) status_code, index, expression
)
) )
content_type = action.data["fixed_response_config._content_type"] content_type = action.data["fixed_response_config._content_type"]
if content_type and content_type not in [ if content_type and content_type not in [
@ -603,16 +605,19 @@ Member must satisfy regular expression pattern: ^(2|4|5)\d\d$"
def create_target_group(self, name, **kwargs): def create_target_group(self, name, **kwargs):
if len(name) > 32: if len(name) > 32:
raise InvalidTargetGroupNameError( raise InvalidTargetGroupNameError(
"Target group name '%s' cannot be longer than '32' characters" % name "Target group name '{}' cannot be longer than '32' characters".format(
name
)
) )
if not re.match("^[a-zA-Z0-9\-]+$", name): if not re.match(r"^[a-zA-Z0-9\-]+$", name):
raise InvalidTargetGroupNameError( raise InvalidTargetGroupNameError(
"Target group name '%s' can only contain characters that are alphanumeric characters or hyphens(-)" "Target group name '{}' can only contain characters that are alphanumeric characters or hyphens(-)".format(
% name name
)
) )
# undocumented validation # undocumented validation
if not re.match("(?!.*--)(?!^-)(?!.*-$)^[A-Za-z0-9-]+$", name): if not re.match(r"(?!.*--)(?!^-)(?!.*-$)^[A-Za-z0-9-]+$", name):
raise InvalidTargetGroupNameError( raise InvalidTargetGroupNameError(
"1 validation error detected: Value '%s' at 'targetGroup.targetGroupArn.targetGroupName' failed to satisfy constraint: Member must satisfy regular expression pattern: (?!.*--)(?!^-)(?!.*-$)^[A-Za-z0-9-]+$" "1 validation error detected: Value '%s' at 'targetGroup.targetGroupArn.targetGroupName' failed to satisfy constraint: Member must satisfy regular expression pattern: (?!.*--)(?!^-)(?!.*-$)^[A-Za-z0-9-]+$"
% name % name

4
moto/events/models.py
View File

@ -305,12 +305,12 @@ class EventsBackend(BaseBackend):
if principal is None or self.ACCOUNT_ID.match(principal) is None: if principal is None or self.ACCOUNT_ID.match(principal) is None:
raise JsonRESTError( raise JsonRESTError(
"InvalidParameterValue", "Principal must match ^(\d{1,12}|\*)$" "InvalidParameterValue", r"Principal must match ^(\d{1,12}|\*)$"
) )
if statement_id is None or self.STATEMENT_ID.match(statement_id) is None: if statement_id is None or self.STATEMENT_ID.match(statement_id) is None:
raise JsonRESTError( raise JsonRESTError(
"InvalidParameterValue", "StatementId must match ^[a-zA-Z0-9-_]{1,64}$" "InvalidParameterValue", r"StatementId must match ^[a-zA-Z0-9-_]{1,64}$"
) )
event_bus._permissions[statement_id] = { event_bus._permissions[statement_id] = {

2
moto/packages/httpretty/http.py
View File

@ -134,7 +134,7 @@ def parse_requestline(s):
ValueError: Not a Request-Line ValueError: Not a Request-Line
""" """
methods = "|".join(HttpBaseClass.METHODS) methods = "|".join(HttpBaseClass.METHODS)
m = re.match(r"(" + methods + ")\s+(.*)\s+HTTP/(1.[0|1])", s, re.I) m = re.match(r"({})\s+(.*)\s+HTTP/(1.[0|1])".format(methods), s, re.I)
if m: if m:
return m.group(1).upper(), m.group(2), m.group(3) return m.group(1).upper(), m.group(2), m.group(3)
else: else:

2
moto/secretsmanager/utils.py
View File

@ -89,7 +89,7 @@ def _exclude_characters(password, exclude_characters):
for c in exclude_characters: for c in exclude_characters:
if c in string.punctuation: if c in string.punctuation:
# Escape punctuation regex usage # Escape punctuation regex usage
c = "\{0}".format(c) c = r"\{0}".format(c)
password = re.sub(c, "", str(password)) password = re.sub(c, "", str(password))
return password return password