I have a question, in case you add another sub to this module that has nothing todo with connecting to a database, like returning a formatted date per say, it will mean that this module will be connecting to this database forever right, because this line
is not part of the module, but part of the test.pl script. Shouldn't be a passing value to this module to indicate that a connection to the database has been required? Otherwise keep the call on the module "disconnected"?