@@ -28,10 +28,7 @@ def test_roledefinition_create_with_permissions_reverse_sync(admin_api_client, e
2828 # Mock successful response from resource server
2929 mock_response = mock .Mock ()
3030 mock_response .status_code = 201
31- mock_response .json .return_value = {
32- 'ansible_id' : str (uuid .uuid4 ()),
33- 'service_id' : str (uuid .uuid4 ())
34- }
31+ mock_response .json .return_value = {'ansible_id' : str (uuid .uuid4 ()), 'service_id' : str (uuid .uuid4 ())}
3532 mock_make_request .return_value = mock_response
3633
3734 # Create RoleDefinition via API (this should trigger reverse-sync)
@@ -40,7 +37,7 @@ def test_roledefinition_create_with_permissions_reverse_sync(admin_api_client, e
4037 'name' : 'test-reverse-sync-role' ,
4138 'description' : 'Test role for reverse sync' ,
4239 'permissions' : ['shared.view_organization' , 'shared.change_organization' ],
43- 'content_type' : 'shared.organization'
40+ 'content_type' : 'shared.organization' ,
4441 }
4542
4643 response = admin_api_client .post (url , data = data , format = "json" )
@@ -50,12 +47,10 @@ def test_roledefinition_create_with_permissions_reverse_sync(admin_api_client, e
5047 assert mock_make_request .called , "Resource server sync should have been called"
5148
5249 # Find the POST request that creates the resource
53- create_calls = [call for call in mock_make_request .call_args_list
54- if call [0 ][0 ].upper () == 'POST' ] # First arg is HTTP method
50+ create_calls = [call for call in mock_make_request .call_args_list if call [0 ][0 ].upper () == 'POST' ] # First arg is HTTP method
5551
5652 assert len (create_calls ) >= 1 , (
57- f"Should have at least one POST call to create resource. "
58- f"Found calls: { [call [0 ] for call in mock_make_request .call_args_list ]} "
53+ f"Should have at least one POST call to create resource. " f"Found calls: { [call [0 ] for call in mock_make_request .call_args_list ]} "
5954 )
6055
6156 # Get the request data from the create call
@@ -74,4 +69,4 @@ def test_roledefinition_create_with_permissions_reverse_sync(admin_api_client, e
7469 assert actual_permissions == expected_permissions , (
7570 f"Expected permissions { expected_permissions } in reverse-sync data, "
7671 f"but got { actual_permissions } . The delayed sync fix should include permissions!"
77- )
72+ )
0 commit comments