Move hook_schema out of ban.install and create the table lazy

Let's see what CI picks up.

I drew inspiration from this issue patch: #2338747: Move {router} out of system.install and create the table lazy →

The ban_ip table is being created when the ban module is installed but it is not being deleted when it is being uninstalled. I can't find a module which uses this approach to know how to hook this up correctly.

Update from @catch on slack:

You can drop the tables on uninstall. In ModuleInstaller, it calls removeBin when a module providing a cache bin is uninstalled, and the database backend drops the table in ::removeBin()

I rebased this, I also added the uninstall hook to remove the db too.

Can the issue summary be complete with full template please.

Seems almost like an API change and since the tests broke and had to be updated possible cold break contrib tests now too.

There is a question on the parent issue that I think needs resolving first actually.

For future self when updating IS.
I don't think this is an api change, the test failed cause previously it called install schema, which we no longer need to do because this creates the table on the fly.

The issue is the test called db connection directly to check for existence, but since install schema no longer is called the table doesn't exist.

All I did was add a call that would trigger the db table creation which is performs the same function as the schema call used to serve.