Fix test failure, attempt 2

Author: timvaillancourt

go/logic/inspect.go

@@ -51,18 +51,15 @@ func (this *Inspector) InitDBConnections() (err error) {
 	if this.db, _, err = mysql.GetDB(this.migrationContext.Uuid, inspectorUri); err != nil {
 		return err
 	}
-	if err := this.validateConnection(); err != nil {
-		return err
-	}
-	if this.migrationContext.InspectorServerInfo, err = mysql.GetServerInfo(this.db); err != nil {
-		return err
-	}
 
 	informationSchemaUri := this.connectionConfig.GetDBUri("information_schema")
 	if this.informationSchemaDb, _, err = mysql.GetDB(this.migrationContext.Uuid, informationSchemaUri); err != nil {
 		return err
 	}
 
+	if err := this.validateConnection(); err != nil {
+		return err
+	}
 	if !this.migrationContext.AliyunRDS && !this.migrationContext.GoogleCloudPlatform && !this.migrationContext.AzureMySQL {
 		if impliedKey, err := mysql.GetInstanceKey(this.db); err != nil {
 			return err
@@ -217,11 +214,15 @@ func (this *Inspector) inspectOriginalAndGhostTables() (err error) {
 }
 
 // validateConnection issues a simple can-connect to MySQL
-func (this *Inspector) validateConnection() error {
+func (this *Inspector) validateConnection() (err error) {
 	if len(this.connectionConfig.Password) > mysql.MaxReplicationPasswordLength {
 		return fmt.Errorf("MySQL replication length limited to 32 characters. See https://dev.mysql.com/doc/refman/5.7/en/assigning-passwords.html")
 	}
 
+	if this.migrationContext.InspectorServerInfo, err = mysql.GetServerInfo(this.db); err != nil {
+		return err
+	}
+
 	return base.ValidateConnection(this.ServerInfo(), this.connectionConfig, this.migrationContext, this.name)
 }