diff --git a/moto/quicksight/urls.py b/moto/quicksight/urls.py index 5c1e8445bbda..ad2d5b45b862 100644 --- a/moto/quicksight/urls.py +++ b/moto/quicksight/urls.py @@ -12,10 +12,10 @@ url_paths = { r"{0}/accounts/(?P[\d]+)/data-sets$": response.dataset, r"{0}/accounts/(?P[\d]+)/data-sets/(?P[^/.]+)/ingestions/(?P[^/.]+)$": response.ingestion, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/groups$": response.groups, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/groups/(?P[^/.]+)$": response.group, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/groups/(?P[^/.]+)/members$": response.group_members, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/groups/(?P[^/.]+)/members/(?P[^/.]+)$": response.group_member, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/users$": response.users, - r"{0}/accounts/(?P[\d]+)/namespaces/(?P[^/.]+)/users/(?P[^/.]+)$": response.user, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/groups$": response.groups, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/groups/(?P[^/]+)$": response.group, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/groups/(?P[^/]+)/members$": response.group_members, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/groups/(?P[^/]+)/members/(?P[^/]+)$": response.group_member, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/users$": response.users, + r"{0}/accounts/(?P[\d]+)/namespaces/(?P[a-zA-Z0-9._-]+)/users/(?P[^/]+)$": response.user, } diff --git a/tests/test_quicksight/test_quicksight_users.py b/tests/test_quicksight/test_quicksight_users.py index c9c03a64cfd3..a9cee283b6d4 100644 --- a/tests/test_quicksight/test_quicksight_users.py +++ b/tests/test_quicksight/test_quicksight_users.py @@ -146,7 +146,7 @@ def test_create_group_membership(): Namespace="default", Email="fakeemail@example.com", IdentityType="QUICKSIGHT", - UserName="user1", + UserName="user.1", UserRole="READER", ) client.create_group( @@ -154,7 +154,7 @@ def test_create_group_membership(): ) resp = client.create_group_membership( - MemberName="user1", + MemberName="user.1", GroupName="group1", AwsAccountId=ACCOUNT_ID, Namespace="default", @@ -162,8 +162,8 @@ def test_create_group_membership(): resp.should.have.key("GroupMember").equals( { - "Arn": f"arn:aws:quicksight:us-east-2:{ACCOUNT_ID}:group/default/group1/user1", - "MemberName": "user1", + "Arn": f"arn:aws:quicksight:us-east-2:{ACCOUNT_ID}:group/default/group1/user.1", + "MemberName": "user.1", } ) resp.should.have.key("Status").equals(200)