Making this table-driven like [TestParseOpts](https://github.com/lib/pq/blob/master/conn_test.go#L668) might be a little cleaner (and you could probably fold in the failure case above like that as well).