-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Labels
back endPythonPythongood first issueGood for newcomersGood for newcomersstructureThis will make something more clear and less prone to programming errorsThis will make something more clear and less prone to programming errors
Description
Some tests should be created for the update settings endpoint. This endpoint should also be changed to give a response indicating whether it was successful.
This will require setting the Account.directory variable for each test (See documentation in file_server/web/account.py)
Tests:
- Verify
settingsproperty returnsAccount.DEFAULT_SETTINGSon a new account, and verify_settingsproperty on account is an emptydict. - Set properties using endpoint and make sure all expected properties are there (and no extras). Verify
settingsand_settingsproperties are correct. Reload account usingAccount.load_accountand verify those properties again - Try to set properties not in
Account.DEFAULT_PROPERTIESand verify they aren't added
Metadata
Metadata
Assignees
Labels
back endPythonPythongood first issueGood for newcomersGood for newcomersstructureThis will make something more clear and less prone to programming errorsThis will make something more clear and less prone to programming errors