Refactor/access control

[alcueca]

Now that OpenZeppelin's AccessControl.sol does everything we did with RBAC.sol and later Roles.sol, I've refactored our contracts to use OpenZeppelin's.

[vibern0]

Smart move. Well done 👏
Looks good.
I see that the CI fails with test/access/Renounceable.test.ts(2,10): error TS2724: Module '"../../types/truffle-contracts"' has no exported member 'RenounceableMockInstance'. Did you mean 'RenounceableQueueInstance'?
Possibly, it did not fail locally because the files from the last compile still exist, and maybe, with luck, it deploys in the same address, so it did not crash on the tests. Just remove your build folder and you should see the same error 👍

[vibern0]

Just fix the import on tests. Thanks, man 🙏

[alcueca]

Just forgot to remove the test files that were not needed anymore. Thanks!

[vibern0]

Cool.
I see the CI got crazy and failed in one step. Let's see if it works if run again.

Also, I've realized one interesting thing that I haven't seen before.
It is not urgent because this error happens in coverage, but it's something to look at. I'll open an issue with it.

[alcueca]

Failed again :/

I saw that warning, thanks for opening the issue.