Update idp-configuration-id in tests to use UUID

Author: sf-stikone

test/assets/user_add.xml

@@ -1,4 +1,4 @@
 <?xml version='1.0' encoding='UTF-8'?>
 <tsResponse xmlns="http://tableau.com/api" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://tableau.com/api http://tableau.com/api/ts-api-2.3.xsd">
-    <user id="4cc4c17f-898a-4de4-abed-a1681c673ced" name="Cassie" siteRole="Viewer" authSetting="ServerDefault" externalAuthUserId="" idpConfigurationId="test-idp-id" />
+    <user id="4cc4c17f-898a-4de4-abed-a1681c673ced" name="Cassie" siteRole="Viewer" authSetting="ServerDefault" externalAuthUserId="" idpConfigurationId="b47f41b1-2c47-41a3-8b17-a38ebe8b340c" />
 </tsResponse>

test/test_user.py

@@ -1,5 +1,6 @@
 import os
 import unittest
+import uuid
 
 import requests_mock
 
@@ -132,16 +133,18 @@ def test_add(self) -> None:
             response_xml = f.read().decode("utf-8")
         with requests_mock.mock() as m:
             m.post(self.baseurl + "", text=response_xml)
+            # Use a fixed UUID for testing so it matches the XML response
+            test_idp_id = "b47f41b1-2c47-41a3-8b17-a38ebe8b340c"
             new_user = TSC.UserItem(
-                name="Cassie", site_role="Viewer", auth_setting="ServerDefault", idp_configuration_id="test-idp-id"
+                name="Cassie", site_role="Viewer", auth_setting="ServerDefault", idp_configuration_id=test_idp_id
             )
             new_user = self.server.users.add(new_user)
 
         self.assertEqual("4cc4c17f-898a-4de4-abed-a1681c673ced", new_user.id)
         self.assertEqual("Cassie", new_user.name)
         self.assertEqual("Viewer", new_user.site_role)
         self.assertEqual("ServerDefault", new_user.auth_setting)
-        self.assertEqual("test-idp-id", new_user.idp_configuration_id)
+        self.assertEqual(test_idp_id, new_user.idp_configuration_id)
 
     def test_populate_workbooks(self) -> None:
         with open(POPULATE_WORKBOOKS_XML, "rb") as f:

test/test_user_model.py

@@ -2,6 +2,7 @@
 import unittest
 from unittest.mock import *
 import io
+import uuid
 
 import pytest
 
@@ -20,7 +21,7 @@ def test_invalid_site_role(self):
             user.site_role = "Hello"
 
     def test_idp_configuration_id(self):
-        idp_id = "idp123"
+        idp_id = str(uuid.uuid4())
         user = TSC.UserItem("test", "Viewer", auth_setting="SAML", idp_configuration_id=idp_id)
         self.assertEqual(idp_id, user.idp_configuration_id)