tree 695c7ab131254c5324609ad6aa53cb543e17ac91
parent beff0ca8c398cea5dcbdf887b14e97cc24998c17
author Felipe Monteiro <felipe.monteiro@att.com> 1493939813 +0100
committer Felipe Monteiro <felipe.monteiro@att.com> 1495064404 +0100

Remove singleton from RbacUtils constructor

Currently, the RbacUtils class in rbac_utils is a singleton,
which means the constructor is only called once. The problem with
that is when we instantiate RbacUtils in each rbac_base class,
we have to then also call switch_role(toggle_rbac_role=False).
We could do that inside the constructor to simplify the code,
but only if RbacUtils constructor stops being a singleton -- or
else admin credentials are not guaranteed during set up across
all test classes.

In addition, setting "rbac_utils = RbacUtils" at the end of rbac_utils
is pointless and only makes the code harder to read. This patch
removes that line of code and refactors the imports for rbac_utils where
necessary.

Change-Id: I778ae19b4bd0b71ab77984ae57dd96fd829a1fc4
Closes-Bug: #1688079