# HG changeset patch
# User Sebastien Jodogne <s.jodogne@gmail.com>
# Date 1788798198 -7200
# Node ID d038ae0719348b0ac1190cda0ca1edf0a8c089fb
# Parent  9866889be864c44767c08f13570688567878cb5d
fix

diff -r 9866889be864 -r d038ae071934 Plugins/WSI/Annotations.py
--- a/Plugins/WSI/Annotations.py	Mon Sep 07 18:20:22 2026 +0200
+++ b/Plugins/WSI/Annotations.py	Mon Sep 07 18:23:18 2026 +0200
@@ -96,16 +96,6 @@
     return DoPost(ORTHANC, uri, body, headers = { 'Mail' : user })
 
 
-def UnpackSetOfStandardUsers(users):
-    s = []
-    for user in users:
-        assert(user['type'] == 1)
-        assert(not user['name'] in s)
-        s.append(user['name'])
-    return s
-
-
-
 class Orthanc(unittest.TestCase):
     def setUp(self):
         if (sys.version_info >= (3, 0)):
@@ -316,6 +306,14 @@
 
 
     def test_search_active_users(self):
+        def UnpackSetOfStandardUsers(users):
+            s = []
+            for user in users:
+                self.assertEqual(1, user['type'])  # This corresponds to int(OrthancWSI::UserId::Type_Standard) in C++ code
+                self.assertFalse(user['name'] in s)
+                s.append(user['name'])
+            return s
+
         a = Execute('/wsi/api/create-user-layer')  # admin@uclouvain.be
         b = Execute('/wsi/api/create-user-layer', user = 'instructor@uclouvain.be')
         c = Execute('/wsi/api/create-user-layer', user = 'learner@uclouvain.be')
@@ -349,7 +347,6 @@
         self.assertTrue('learner2@uclouvain.be' in v)
 
         info = Execute('/wsi/api/workspace-info')
-        pprint.pprint(info)
 
         v = UnpackSetOfStandardUsers(Execute('/wsi/api/search-active-users', { 'query' : '' }, user = 'learner@uclouvain.be'))
         if info['learner_to_learner_sharing']: