From 71d52d45ba057dd547c3d5d10ebbfcded5e8ef15 Mon Sep 17 00:00:00 2001 From: Joseph Schorr Date: Tue, 7 Mar 2017 14:36:56 -0500 Subject: [PATCH] Add a test for same user returned twice in team sync --- data/users/test/test_teamsync.py | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/data/users/test/test_teamsync.py b/data/users/test/test_teamsync.py index 5cb4fcda8..34bb42fcd 100644 --- a/data/users/test/test_teamsync.py +++ b/data/users/test/test_teamsync.py @@ -129,6 +129,16 @@ class FakeUsers(FederatedUsers): UserInformation('someuser', 'someuser', 'someuser@devtable.com'), ], ['someuser', 'buynlarge+anotherbot']), + + # Team which returns the same member twice, as pagination in some engines (like LDAP) is not + # stable. + ([], + [ + UserInformation('someuser', 'someuser', 'someuser@devtable.com'), + UserInformation('anotheruser', 'anotheruser', 'anotheruser@devtable.com'), + UserInformation('someuser', 'someuser', 'someuser@devtable.com'), + ], + ['anotheruser', 'someuser']), ]) def test_syncing(starting_membership, group_membership, expected_membership, app): org = model.organization.get_organization('buynlarge')