impl override does not make sense here (overrding load_dialect_impl)
impl is required, even if it's not used
removing unused imports
adding some missing test coverage