Changeset 1d92bf2 in OpenWorkouts-current for ow/tests/test_catalog.py

Timestamp:

Dec 16, 2018, 1:07:04 AM (5 years ago)

Author:

borja <borja@…>

Branches:

current, feature/docs, master

Children:

6560b8f

Parents:

929097a

Message:

(#37) Allow login using email address instead of username:

• Use user uids as keys in the root folder for referencing user objects (instead of username)

• Use uids for referencing users all over the place (auth, permissions, traversal urls, etc)

• Replaced the username concept with nickname. This nickname will be used as a shortcut to access "public profile" pages for users

• Reworked lots of basic methods in the OpenWorkouts root object (s/username/nickname, marked as properties some methods like users, emails, etc)

• Added new add_user() and delete_user() helpers to the OpenWorkouts root object

• Modified views to match the new OpenWorkouts root helpers

• Fixed bug in the dashboard redirect view, causing an endless loop if an authenticated user does not exist anymore when loading a page.

• Lots of tests fixes, adaptations and catch up.

File:

• ow/tests/test_catalog.py (modified) (3 diffs)

ow/tests/test_catalog.py

@@ -56 +56 @@
         changes = update_indexes(catalog, indexes)
         assert changes['added'] == ['newindex']
-        assert changes['removed'] == ['sport']
+        assert changes['removed'] == ['email', 'sport']
 
     def test_update_indexes_empty(self, root):
@@ -63 +63 @@
         changes = update_indexes(catalog, indexes)
         assert changes['added'] == []
-        assert changes['removed'] == ['sport']
+        assert changes['removed'] == ['email', 'sport']
 
     def test_install_catalog(self):
         root = OpenWorkouts()
+        assert isinstance(getattr(root, 'catalog', None), Catalog)
+        del root.catalog
         assert getattr(root, 'catalog', None) is None
         install_catalog(root)
@@ -78 +80 @@
     def test_get_catalog_not_existing_catalog(self):
         root = OpenWorkouts()
+        assert isinstance(getattr(root, 'catalog', None), Catalog)
+        del root.catalog
         assert getattr(root, 'catalog', None) is None
         catalog = get_catalog(root)