Hi @deanearlbartolabac 

Please reconsider the use of setting the admin role for testing.

Currently, it looks like the function test_plugin_menu_added is the only test that uses an admin to evaluate but this function is also using another line that sets the admin $this->set_admin_user(). Commenting this would still result to passed since the admin has been set on the init.

This could cause an issue for future tests asserting admin as the role may be defaulted to admin since it has been set twice.

I would suggest removing setting the admin on setup and just apply it on tests that would require admin roles (retain line no. 83 as it also has a switch to go back to normal user for further assertion).

If you would like to retain setting it on setUp(), then you can omit the one within the function but I would suggest adding a user role reset on tearDown.