Add a recognized setting for test_settingsqr_ignores_unrecognized_setting
What the developer wrote
Authored by biel
Add a recognized setting for test_settingsqr_ignores_unrecognized_setting
What changed, and why it matters
This commit only changes a test file. It adds one more unrecognized setting ('passphrase=E') to an existing test that checks the SettingsQR parser ignores unknown settings. There is no change to production code, no security fix, and no vulnerability introduced.
No security action required. This is a routine test-only commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff modifies tests/test_settings.py in the SeedSigner repository. The test test_settingsqr_ignores_unrecognized_setting now includes ‘passphrase=E’ in the settingsqr_data string and asserts that ‘passphrase’ is not present in the parsed settings_update_dict. This is a test-only enhancement verifying that the SettingsQR parser correctly filters out unrecognized keys. No application logic was changed.
Changed components
tests/test_settings.pyInspect captured patch +2 / −1
diff --git a/tests/test_settings.py b/tests/test_settings.py
index 0aecfa1..24aef76 100644
--- a/tests/test_settings.py
+++ b/tests/test_settings.py
@@ -77,10 +77,11 @@ class TestSettings(BaseTest):
def test_settingsqr_ignores_unrecognized_setting(self):
""" SettingsQR parser should ignore unrecognized settings """
- settingsqr_data = "settings::v1 name=Foo favorite_food=bacon"
+ settingsqr_data = "settings::v1 name=Foo favorite_food=bacon passphrase=E"
config_name, settings_update_dict = Settings.parse_settingsqr(settingsqr_data)
assert "favorite_food" not in settings_update_dict
+ assert "passphrase" not in settings_update_dict
# Accepts update with no Exceptions
self.settings.update(new_settings=settings_update_dict)
Why this scored 12/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.