Separate admin test from RoutersNegativeTest

Most tests of RoutersNegativeTest don't require admin credential,
test_router_set_gateway_used_ip_returns_409 only requires it.
So this patch makes RoutersNegativeTest inherit from BaseNetworkTest
which is based on non-admin credential. In addition, this patch makes
a test stop using _add_router_interface_with_subnet_id() on the test
class for simple code and removing the dependency to BaseRouterTest.

NOTE: test_router_set_gateway_used_ip_returns_409 is not used on
      defcore side. So this path change doesn't affect on defcore.

Change-Id: I8bf2d523e52353989be8f83b06c0ce6718931e8c
2 files changed