Testing out a fix for sql-server integration tests

go/libraries/doltcore/servercfg/yaml_config.go

@@ -316,7 +316,13 @@ func (cfg YAMLConfig) WriteTimeout() uint64 {
 // User returns the username that connecting clients must use.
 func (cfg YAMLConfig) User() string {
 	if cfg.UserConfig.Name == nil {
-		return DefaultUser
+		// TODO: Having this return "root" by default prevents us from being able to
+		//       tell is a user was explicitly specified or not. The server initialization
+		//       code then thinks an ephemeral user was requested, and doesn't create the
+		//       implicit, persisted root superuser.
+		// TODO: But... why would this have not allowed a root user to log in?
+		//return DefaultUser
+		return ""
 	}
 
 	return *cfg.UserConfig.Name

go/libraries/doltcore/servercfg/yaml_config_test.go

@@ -324,7 +324,12 @@ func TestYAMLConfigDefaults(t *testing.T) {
 
 	assert.Equal(t, DefaultHost, cfg.Host())
 	assert.Equal(t, DefaultPort, cfg.Port())
-	assert.Equal(t, DefaultUser, cfg.User())
+	// TODO: Testing out not defaulting the user to "root", so that we can
+	//       distinguish between when a user is explicitly specified or not.
+	//       We want this, so we can tell if the user was explicitly specified
+	//       and should be an ephemeral superuser, or if none was specified
+	//       and we should initialize the implicit root superuser.
+	assert.Equal(t, "", cfg.User())
 	assert.Equal(t, DefaultPass, cfg.Password())
 	assert.Equal(t, uint64(DefaultTimeout), cfg.WriteTimeout())
 	assert.Equal(t, uint64(DefaultTimeout), cfg.ReadTimeout())