fix unit tests
This commit is contained in:
		
							parent
							
								
									fcd9d27f18
								
							
						
					
					
						commit
						27c6ac2565
					
				|  | @ -1170,7 +1170,6 @@ func (c *Cluster) initTeamMembers(teamID string, isPostgresSuperuserTeam bool) e | ||||||
| 		newRole := spec.PgUser{ | 		newRole := spec.PgUser{ | ||||||
| 			Origin:     spec.RoleOriginTeamsAPI, | 			Origin:     spec.RoleOriginTeamsAPI, | ||||||
| 			Name:       username, | 			Name:       username, | ||||||
| 			Namespace:  c.Namespace, |  | ||||||
| 			Flags:      flags, | 			Flags:      flags, | ||||||
| 			MemberOf:   memberOf, | 			MemberOf:   memberOf, | ||||||
| 			Parameters: c.OpConfig.TeamAPIRoleConfiguration, | 			Parameters: c.OpConfig.TeamAPIRoleConfiguration, | ||||||
|  |  | ||||||
|  | @ -221,7 +221,7 @@ func (c *Cluster) readPgUsersFromDatabase(userNames []string) (users spec.PgUser | ||||||
| 			roldeleted = true | 			roldeleted = true | ||||||
| 		} | 		} | ||||||
| 
 | 
 | ||||||
| 		users[rolname] = spec.PgUser{Name: rolname, Namespace: c.Namespace, Password: rolpassword, Flags: flags, MemberOf: memberof, Parameters: parameters, Deleted: roldeleted} | 		users[rolname] = spec.PgUser{Name: rolname, Password: rolpassword, Flags: flags, MemberOf: memberof, Parameters: parameters, Deleted: roldeleted} | ||||||
| 	} | 	} | ||||||
| 
 | 
 | ||||||
| 	return users, nil | 	return users, nil | ||||||
|  |  | ||||||
|  | @ -557,7 +557,7 @@ func (c *Cluster) syncRoles() (err error) { | ||||||
| 	// create list of database roles to query
 | 	// create list of database roles to query
 | ||||||
| 	for _, u := range c.pgUsers { | 	for _, u := range c.pgUsers { | ||||||
| 		pgRole := u.Name | 		pgRole := u.Name | ||||||
| 		if u.Namespace != c.Namespace { | 		if u.Namespace != c.Namespace && u.Namespace != "" { | ||||||
| 			// to avoid the conflict of having multiple users of same name
 | 			// to avoid the conflict of having multiple users of same name
 | ||||||
| 			// but each in different namespace.
 | 			// but each in different namespace.
 | ||||||
| 			pgRole = fmt.Sprintf("%s.%s", u.Name, u.Namespace) | 			pgRole = fmt.Sprintf("%s.%s", u.Name, u.Namespace) | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue