chore: default list methods to descending order (#234)

faroceann · web-flow · commit e65ec1e82e3c · 2024-03-08T15:22:12.000-05:00
* chore: default list methods to descending order

* chore: format

* chore: format

* cleanup

* chore: bump version

* update image

* chore: drop tests on python v2

* chore: drop tests on python v2

* chore: drop tests on python v2
diff --git a/.semaphore/semaphore.yml b/.semaphore/semaphore.yml
@@ -3,31 +3,21 @@ name: Python
 agent:
   machine:
     type: e1-standard-2
-    os_image: ubuntu1804
+  containers:
+    - name: main
+      image: "registry.semaphoreci.com/python:3.7"
 blocks:
   - task:
       secrets:
         - name: codecov-workos-python
       prologue:
         commands:
           - checkout
-          - sem-version python 2.7
-          - python -m pip install --upgrade pip
-          - pip install -e .[dev]
-          - sem-version python 3.7
           - python -m pip install --upgrade pip
           - pip install -e .[dev]
       jobs:
-        - name: python
-          commands:
-            - sem-version python 2.7
-            - flake8 . --count --select=E9,F7,F82 --show-source --statistics
-            - flake8 . --count --exit-zero --max-complexity=10 --statistics
-            - pytest
-        # Coverage would be the same regardless of Python version
         - name: python3 + codecov
           commands:
-            - sem-version python 3.7
             - flake8 . --count --select=E9,F7,F82 --show-source --statistics
             - flake8 . --count --exit-zero --max-complexity=10 --statistics
             - pytest --cov-report xml --cov=workos
diff --git a/workos/__about__.py b/workos/__about__.py
@@ -12,7 +12,7 @@
 
 __package_url__ = "https://github.com/workos-inc/workos-python"
 
-__version__ = "2.2.0"
+__version__ = "3.0.0"
 
 __author__ = "WorkOS"
 
diff --git a/workos/directory_sync.py b/workos/directory_sync.py
@@ -69,7 +69,7 @@ def list_users(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if group is not None:
@@ -138,7 +138,7 @@ def list_users_v2(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if group is not None:
@@ -206,7 +206,12 @@ def list_groups(
             limit = RESPONSE_LIMIT
             default_limit = True
 
-        params = {"limit": limit, "before": before, "after": after, "order": order}
+        params = {
+            "limit": limit,
+            "before": before,
+            "after": after,
+            "order": order or "desc",
+        }
         if user is not None:
             params["user"] = user
         if directory is not None:
@@ -268,7 +273,12 @@ def list_groups_v2(
             limit = RESPONSE_LIMIT
             default_limit = True
 
-        params = {"limit": limit, "before": before, "after": after, "order": order}
+        params = {
+            "limit": limit,
+            "before": before,
+            "after": after,
+            "order": order or "desc",
+        }
         if user is not None:
             params["user"] = user
         if directory is not None:
@@ -395,7 +405,7 @@ def list_directories(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
@@ -463,7 +473,7 @@ def list_directories_v2(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
diff --git a/workos/organizations.py b/workos/organizations.py
@@ -60,7 +60,7 @@ def list_organizations(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
@@ -122,7 +122,7 @@ def list_organizations_v2(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
diff --git a/workos/resources/list.py b/workos/resources/list.py
@@ -75,7 +75,7 @@ def auto_paging_iter(self):
             params = {
                 "after": after,
                 "before": before,
-                "order": order,
+                "order": order or "desc",
             }
             params.update(resource_specific_params)
 
diff --git a/workos/sso.py b/workos/sso.py
@@ -234,7 +234,7 @@ def list_connections(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
@@ -317,7 +317,7 @@ def list_connections_v2(
             "limit": limit,
             "before": before,
             "after": after,
-            "order": order,
+            "order": order or "desc",
         }
 
         if order is not None:
diff --git a/workos/user_management.py b/workos/user_management.py
@@ -107,6 +107,7 @@ def list_users(
             "limit": limit,
             "before": before,
             "after": after,
+            "order": order or "desc",
         }
 
         if order is not None:
@@ -278,6 +279,7 @@ def list_organization_memberships(
             "limit": limit,
             "before": before,
             "after": after,
+            "order": order or "desc",
         }
 
         if order is not None:
@@ -944,6 +946,7 @@ def list_invitations(
             "limit": limit,
             "before": before,
             "after": after,
+            "order": order or "desc",
         }
 
         if order is not None:

Original file line number	Diff line number	Diff line change
`@@ -75,7 +75,7 @@ def auto_paging_iter(self):`
`75`	`75`	`params = {`
`76`	`76`	`"after": after,`
`77`	`77`	`"before": before,`
`78`		`- "order": order,`
	`78`	`+ "order": order or "desc",`
`79`	`79`	`}`
`80`	`80`	`params.update(resource_specific_params)`
`81`	`81`
Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,7 @@ def list_users(`
`107`	`107`	`"limit": limit,`
`108`	`108`	`"before": before,`
`109`	`109`	`"after": after,`
	`110`	`+ "order": order or "desc",`
`110`	`111`	`}`
`111`	`112`
`112`	`113`	`if order is not None:`
`@@ -278,6 +279,7 @@ def list_organization_memberships(`
`278`	`279`	`"limit": limit,`
`279`	`280`	`"before": before,`
`280`	`281`	`"after": after,`
	`282`	`+ "order": order or "desc",`
`281`	`283`	`}`
`282`	`284`
`283`	`285`	`if order is not None:`
`@@ -944,6 +946,7 @@ def list_invitations(`
`944`	`946`	`"limit": limit,`
`945`	`947`	`"before": before,`
`946`	`948`	`"after": after,`
	`949`	`+ "order": order or "desc",`
`947`	`950`	`}`
`948`	`951`
`949`	`952`	`if order is not None:`