Added CLI tests for lms.userget.

edulinq · Sep 5, 2024 · f7ab70b · f7ab70b
1 parent c29453f
commit f7ab70b
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 2 deletions.
diff --git a/autograder/cli/common.py b/autograder/cli/common.py
@@ -15,12 +15,18 @@ def list_users(users, table = False):
         _list_users(users)
 
 def _list_users(users, indent = ''):
-    for user in users:
+    users = list(users)
+
+    for i in range(len(users)):
+        user = users[i]
+
+        if (i != 0):
+            print()
+
         print(indent + "Email:", user['email'])
         print(indent + "Name:", user['name'])
         print(indent + "Role:", user['role'])
         print(indent + "LMS ID:", user['lms-id'])
-        print()
 
 def _list_users_table(users, header = True, keys = HEADERS):
     if (header):

diff --git a/autograder/cli/lms/get-user.py → autograder/cli/lms/user-get.py b/autograder/cli/lms/get-user.py → autograder/cli/lms/user-get.py
diff --git a/tests/cli/testdata/lms/lms_user_get_base.txt b/tests/cli/testdata/lms/lms_user_get_base.txt
@@ -0,0 +1,11 @@
+{
+    "cli": "autograder.cli.lms.user-get",
+    "arguments": [
+        "--target-email", "[email protected]"
+    ]
+}
+---
+Email: [email protected]
+Name: course-student
+Role: student
+LMS ID: [email protected]
diff --git a/tests/cli/testdata/lms/lms_user_get_missing.txt b/tests/cli/testdata/lms/lms_user_get_missing.txt
@@ -0,0 +1,8 @@
+{
+    "cli": "autograder.cli.lms.user-get",
+    "arguments": [
+        "--target-email", "[email protected]"
+    ]
+}
+---
+No matching autograder user found.