Skip to content

Commit 7cc7921

Browse files
potiukCloud Composer Team
authored and
Cloud Composer Team
committed
Add roles to create_user test (#20773)
Flask App Builder 3.4.3 made role and conf_password obligatory when creating user: dpgaspar/Flask-AppBuilder#1758 Our test for user creation did not have the role set (though the UI used it and the cient also allows to set it). This change adds the `roles` in our tests to enable upgrade to FAB 3.4.3 for the CI (currently tests in main fail because of the test failure) GitOrigin-RevId: a1a32f7c7c2df41e6150f2594a3a41bfecb76fb0
1 parent d8d3f0c commit 7cc7921

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/www/views/test_views_base.py

+1
Original file line numberDiff line numberDiff line change
@@ -326,6 +326,7 @@ def test_create_user(app, admin_client, non_exist_username):
326326
'last_name': 'fake_last_name',
327327
'username': non_exist_username,
328328
'email': 'fake_email@email.com',
329+
'roles': [1],
329330
'password': 'test',
330331
'conf_password': 'test',
331332
},

0 commit comments

Comments
 (0)