Skip to content

Commit c984fe5

Browse files
committed
resolve sonar issues
1 parent 61bc62e commit c984fe5

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

lambdas/shared/src/common/validator/expression_checker.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ def _validate_datetime(self, rule, field_name, field_value, row) -> ErrorReport:
129129
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
130130

131131
# UUID validate
132-
def _validate_uuid(self, expressionRule, field_name, field_value, row) -> ErrorReport:
132+
def _validate_uuid(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
133133
try:
134134
uuid.UUID(str(field_value))
135135
except RecordError as e:
@@ -185,7 +185,7 @@ def _validate_integer(self, expression_rule, field_name, field_value, row) -> Er
185185
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
186186

187187
# Float Validate
188-
def _validate_float(self, expression_rule, field_name, field_value, row) -> ErrorReport:
188+
def _validate_float(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
189189
try:
190190
float(field_value)
191191
except RecordError as e:
@@ -384,7 +384,7 @@ def _validate_in_array(self, expression_rule, field_name, field_value, row) -> E
384384
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
385385

386386
# Upper Validate
387-
def _validate_upper(self, expression_rule, field_name, field_value, row) -> ErrorReport:
387+
def _validate_upper(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
388388
try:
389389
result = field_value.isupper()
390390

@@ -410,7 +410,7 @@ def _validate_upper(self, expression_rule, field_name, field_value, row) -> Erro
410410
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
411411

412412
# Lower Validate
413-
def _validate_lower(self, expression_rule, field_name, field_value, row) -> ErrorReport:
413+
def _validate_lower(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
414414
try:
415415
result = field_value.islower()
416416

@@ -486,7 +486,7 @@ def _validate_ends_with(self, expression_rule, field_name, field_value, row) ->
486486
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
487487

488488
# Empty Validate
489-
def _validate_empty(self, expression_rule, field_name, field_value, row) -> ErrorReport:
489+
def _validate_empty(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
490490
try:
491491
if field_value:
492492
raise RecordError(
@@ -510,7 +510,7 @@ def _validate_empty(self, expression_rule, field_name, field_value, row) -> Erro
510510
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
511511

512512
# Not Empty Validate
513-
def _validate_not_empty(self, expression_rule, field_name, field_value, row) -> ErrorReport:
513+
def _validate_not_empty(self, _expression_rule, field_name, field_value, row) -> ErrorReport:
514514
try:
515515
if not field_value:
516516
raise RecordError(
@@ -532,7 +532,7 @@ def _validate_not_empty(self, expression_rule, field_name, field_value, row) ->
532532
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, field_name, "", self.summarise)
533533

534534
# Positive Validate
535-
def _validate_positive(self, expressionRule, fieldName, fieldValue, row) -> ErrorReport:
535+
def _validate_positive(self, _expression_rule, fieldName, fieldValue, row) -> ErrorReport:
536536
try:
537537
value = float(fieldValue)
538538
if value < 0:
@@ -557,7 +557,7 @@ def _validate_positive(self, expressionRule, fieldName, fieldValue, row) -> Erro
557557
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, fieldName, "", self.summarise)
558558

559559
# NHSNumber Validate
560-
def _validate_nhs_number(self, expressionRule, fieldName, fieldValue, row) -> ErrorReport:
560+
def _validate_nhs_number(self, _expression_rule, fieldName, fieldValue, row) -> ErrorReport:
561561
try:
562562
regexRule = "^6[0-9]{10}$"
563563
result = re.search(regexRule, fieldValue)
@@ -583,7 +583,7 @@ def _validate_nhs_number(self, expressionRule, fieldName, fieldValue, row) -> Er
583583
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, fieldName, "", self.summarise)
584584

585585
# Gender Validate
586-
def _validate_gender(self, expressionRule, fieldName, fieldValue, row) -> ErrorReport:
586+
def _validate_gender(self, _expression_rule, fieldName, fieldValue, row) -> ErrorReport:
587587
try:
588588
ruleList = ["0", "1", "2", "9"]
589589

@@ -609,7 +609,7 @@ def _validate_gender(self, expressionRule, fieldName, fieldValue, row) -> ErrorR
609609
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, fieldName, "", self.summarise)
610610

611611
# PostCode Validate
612-
def _validate_post_code(self, expressionRule, fieldName, fieldValue, row) -> ErrorReport:
612+
def _validate_post_code(self, _expression_rule, fieldName, fieldValue, row) -> ErrorReport:
613613
try:
614614
# UK postcode regex (allows optional space)
615615
regexRule = r"^(GIR\s?0AA|(?:(?:[A-PR-UWYZ][0-9]{1,2})|(?:[A-PR-UWYZ][A-HK-Y][0-9]{1,2})|(?:[A-PR-UWYZ][0-9][A-HJKS-UW])|(?:[A-PR-UWYZ][A-HK-Y][0-9][ABEHMNPRV-Y]))\s?[0-9][ABD-HJLNP-UW-Z]{2})$"
@@ -688,7 +688,7 @@ def _validate_only_if(self, expressionRule, fieldName, fieldValue, row) -> Error
688688
return ErrorReport(ExceptionMessages.UNEXPECTED_EXCEPTION, message, row, fieldName, "", self.summarise)
689689

690690
# Check with Lookup
691-
def _validate_against_lookup(self, expressionRule, fieldName, fieldValue, row) -> ErrorReport:
691+
def _validate_against_lookup(self, _expression_rule, fieldName, fieldValue, row) -> ErrorReport:
692692
try:
693693
result = self.data_look_up.find_lookup(fieldValue)
694694
if not result:

0 commit comments

Comments
 (0)