Add a missing test for the is_pkg_admin function
Add tests for the is_safe_url function
Add commit and approveacls ACLs to Josef on geany/master
Adjust unit-tests for the new ACLs of josef on geany
Add watchcommits to josef on geany/master
Adjust the unit-tests for josef's new ACL
Check if the full works properly when trying to give packages
Apparently coverage does not `cover` pass, so ignoring it
convert full to string before checking if it is in a list of strings
Add tests to check if the `full` argument works properly
Ignore some pieces of the code that we should never hit, and email if we do

The PkgdbException thrown by these parts of the code should never be seen,
so we actually want to know what happened if they are.
Add tests checking that update_acl is different for the user and the package admin

The user only sees his/her ACL, the package admin sees his/her and the
ones awaiting reviews
Users are allowed to remove their own ACLs
If nothing changed, say so
If the user is not already in the list, we cannot do anything about it

Either the user should request the ACLs or the admin should add the user
manually
Make sure that the branch exists before using it
Add test when the user is not a packager and tries to get ACLs on a package
Document a little the tests so that we know why they behave this way
Test the number of approved ACLs before changing them
Add test checking that a user is allowed to remove his/her own ACL