From 6427526686dbe82728f69a729fe88ba972f12329 Mon Sep 17 00:00:00 2001 From: Akita Noek Date: Fri, 29 Apr 2016 15:34:54 -0400 Subject: [PATCH] Updated JT migration tests to be more correct This test was passing before because we were erroneously making all users organization auditors, which gave users read access to all JT's under the org. --- awx/main/tests/functional/test_rbac_job_templates.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/awx/main/tests/functional/test_rbac_job_templates.py b/awx/main/tests/functional/test_rbac_job_templates.py index 93538f67f0..e439533f07 100644 --- a/awx/main/tests/functional/test_rbac_job_templates.py +++ b/awx/main/tests/functional/test_rbac_job_templates.py @@ -87,7 +87,7 @@ def test_job_template_team_migration_check(deploy_jobtemplate, check_jobtemplate rbac.migrate_projects(apps, None) rbac.migrate_inventory(apps, None) - assert joe in check_jobtemplate.read_role + assert joe not in check_jobtemplate.read_role assert admin in check_jobtemplate.execute_role assert joe not in check_jobtemplate.execute_role @@ -120,12 +120,13 @@ def test_job_template_team_deploy_migration(deploy_jobtemplate, check_jobtemplat rbac.migrate_projects(apps, None) rbac.migrate_inventory(apps, None) - assert joe in deploy_jobtemplate.read_role + assert joe not in deploy_jobtemplate.read_role assert admin in deploy_jobtemplate.execute_role assert joe not in deploy_jobtemplate.execute_role rbac.migrate_job_templates(apps, None) + assert joe in deploy_jobtemplate.read_role assert admin in deploy_jobtemplate.execute_role assert joe in deploy_jobtemplate.execute_role