diff options
| author | deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com> | 2025-02-17 13:04:43 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-17 13:04:43 +0000 |
| commit | e796c7bb6cc6de6368c2d195e233d0b11cf7e699 (patch) | |
| tree | a99037c07a2b2bce642a0ce4648ac858ac0e325c | |
| parent | 74843c48b7aa6f862b2965d590a711aa4cfc5f42 (diff) | |
| download | gdpr-obfuscator-e796c7bb6cc6de6368c2d195e233d0b11cf7e699.tar.gz gdpr-obfuscator-e796c7bb6cc6de6368c2d195e233d0b11cf7e699.zip | |
style: format code with Autopep8, Black and Ruff Formatter
This commit fixes the style issues introduced in 74843c4 according to the output
from Autopep8, Black and Ruff Formatter.
Details: https://github.com/ajschofield/gdpr-obfuscator/pull/2
| -rw-r--r-- | obfuscator/obfuscate.py | 8 | ||||
| -rw-r--r-- | test/test_obfuscator.py | 63 |
2 files changed, 56 insertions, 15 deletions
diff --git a/obfuscator/obfuscate.py b/obfuscator/obfuscate.py index c9c116c..ac0bd21 100644 --- a/obfuscator/obfuscate.py +++ b/obfuscator/obfuscate.py @@ -3,13 +3,15 @@ from obfuscator.logger import get_logger logger = get_logger("Obfuscator") -def obfuscate(data: List[Dict[str, str]], pii_fields: List[str]) -> List[Dict[str,str]]: + +def obfuscate( + data: List[Dict[str, str]], pii_fields: List[str] +) -> List[Dict[str, str]]: if not data: logger.info("No valid data was provided to obfuscate") return [] - + return [ {k: ("***" if k in pii_fields else v) for k, v in record.items()} for record in data ] -
\ No newline at end of file diff --git a/test/test_obfuscator.py b/test/test_obfuscator.py index cfc88e5..c77b6b4 100644 --- a/test/test_obfuscator.py +++ b/test/test_obfuscator.py @@ -1,47 +1,86 @@ from obfuscator.obfuscate import obfuscate + def test_obfuscate_data_with_valid_pii_fields(): data = [ - {"student_id": "1234", "name": "John Smith", "course": "Software", "email_address": "j.smith@email.com"}, - {"student_id": "5678", "name": "Jane Doe", "course": "Data Science", "email_address": "j.doe@email.com"} + { + "student_id": "1234", + "name": "John Smith", + "course": "Software", + "email_address": "j.smith@email.com", + }, + { + "student_id": "5678", + "name": "Jane Doe", + "course": "Data Science", + "email_address": "j.doe@email.com", + }, ] pii_fields = ["name", "email_address"] expected = [ - {"student_id": "1234", "name": "***", "course": "Software", "email_address": "***"}, - {"student_id": "5678", "name": "***", "course": "Data Science", "email_address": "***"} + { + "student_id": "1234", + "name": "***", + "course": "Software", + "email_address": "***", + }, + { + "student_id": "5678", + "name": "***", + "course": "Data Science", + "email_address": "***", + }, ] - + result = obfuscate(data, pii_fields) assert result == expected + def test_obfuscate_data_with_missing_pii_field(): data = [ {"student_id": "1234", "name": "John Smith", "course": "Software"}, - {"student_id": "5678", "name": "Jane Doe", "course": "Data Science", "email_address": "j.doe@email.com"} + { + "student_id": "5678", + "name": "Jane Doe", + "course": "Data Science", + "email_address": "j.doe@email.com", + }, ] pii_fields = ["name", "email_address"] expected = [ {"student_id": "1234", "name": "***", "course": "Software"}, - {"student_id": "5678", "name": "***", "course": "Data Science", "email_address": "***"} + { + "student_id": "5678", + "name": "***", + "course": "Data Science", + "email_address": "***", + }, ] - + result = obfuscate(data, pii_fields) assert result == expected + def test_obfuscate_data_with_no_data(): data = [] pii_fields = ["name", "email_address"] expected = [] - + result = obfuscate(data, pii_fields) assert result == expected + def test_obfuscate_data_with_empty_pii_fields(): data = [ - {"student_id": "1234", "name": "John Smith", "course": "Software", "email_address": "j.smith@email.com"} + { + "student_id": "1234", + "name": "John Smith", + "course": "Software", + "email_address": "j.smith@email.com", + } ] pii_fields = [] expected = data.copy() - + result = obfuscate(data, pii_fields) - assert result == expected
\ No newline at end of file + assert result == expected |
