mirror of
				https://github.com/open-metadata/OpenMetadata.git
				synced 2025-10-31 02:29:03 +00:00 
			
		
		
		
	Fixes 22418: Fixed the teams page breadcrumb url issue (#23824)
* Fix: Fixed the teams page breadcrum url issue * Playwright: Added test for verifing breadcrumb navigation for teams which contains dot in their name
This commit is contained in:
		
							parent
							
								
									598de16eea
								
							
						
					
					
						commit
						f97ed64316
					
				| @ -36,10 +36,14 @@ import { | |||||||
|   uuid, |   uuid, | ||||||
|   visitOwnProfilePage, |   visitOwnProfilePage, | ||||||
| } from '../../utils/common'; | } from '../../utils/common'; | ||||||
| import { addMultiOwner } from '../../utils/entity'; | import { | ||||||
|  |   addMultiOwner, | ||||||
|  |   waitForAllLoadersToDisappear, | ||||||
|  | } from '../../utils/entity'; | ||||||
| import { settingClick } from '../../utils/sidebar'; | import { settingClick } from '../../utils/sidebar'; | ||||||
| import { | import { | ||||||
|   addEmailTeam, |   addEmailTeam, | ||||||
|  |   addTeamHierarchy, | ||||||
|   addTeamOwnerToEntity, |   addTeamOwnerToEntity, | ||||||
|   addUserInTeam, |   addUserInTeam, | ||||||
|   addUserTeam, |   addUserTeam, | ||||||
| @ -47,6 +51,7 @@ import { | |||||||
|   createTeam, |   createTeam, | ||||||
|   executionOnOwnerGroupTeam, |   executionOnOwnerGroupTeam, | ||||||
|   executionOnOwnerTeam, |   executionOnOwnerTeam, | ||||||
|  |   getNewTeamDetails, | ||||||
|   hardDeleteTeam, |   hardDeleteTeam, | ||||||
|   searchTeam, |   searchTeam, | ||||||
|   softDeleteTeam, |   softDeleteTeam, | ||||||
| @ -676,6 +681,39 @@ test.describe('Teams Page', () => { | |||||||
| 
 | 
 | ||||||
|     await afterAction(); |     await afterAction(); | ||||||
|   }); |   }); | ||||||
|  | 
 | ||||||
|  |   test('Verify breadcrumb navigation for a team with a dot in its name', async ({ | ||||||
|  |     page, | ||||||
|  |   }) => { | ||||||
|  |     const team1Details = getNewTeamDetails(`test.department-${uuid()}`); | ||||||
|  |     const team2Details = getNewTeamDetails(`test.team-${uuid()}`); | ||||||
|  | 
 | ||||||
|  |     await settingClick(page, GlobalSettingOptions.TEAMS); | ||||||
|  |     await addTeamHierarchy(page, team1Details, 0); | ||||||
|  | 
 | ||||||
|  |     await page.getByRole('link', { name: team1Details.displayName }).click(); | ||||||
|  |     await waitForAllLoadersToDisappear(page); | ||||||
|  |     await addTeamHierarchy(page, team2Details, 1, true); | ||||||
|  | 
 | ||||||
|  |     await page.getByRole('link', { name: team2Details.displayName }).click(); | ||||||
|  |     await waitForAllLoadersToDisappear(page); | ||||||
|  | 
 | ||||||
|  |     await expect(page.getByTestId('team-heading')).toContainText( | ||||||
|  |       team2Details.displayName | ||||||
|  |     ); | ||||||
|  | 
 | ||||||
|  |     await page.getByRole('link', { name: team1Details.displayName }).click(); | ||||||
|  |     await waitForAllLoadersToDisappear(page); | ||||||
|  | 
 | ||||||
|  |     await expect(page.getByTestId('team-heading')).toContainText( | ||||||
|  |       team1Details.displayName | ||||||
|  |     ); | ||||||
|  |     await expect( | ||||||
|  |       page.getByTestId('team-hierarchy-table').getByRole('link') | ||||||
|  |     ).toContainText(team2Details.displayName); | ||||||
|  | 
 | ||||||
|  |     await hardDeleteTeam(page); | ||||||
|  |   }); | ||||||
| }); | }); | ||||||
| 
 | 
 | ||||||
| test.describe('Teams Page with EditUser Permission', () => { | test.describe('Teams Page with EditUser Permission', () => { | ||||||
|  | |||||||
| @ -447,7 +447,7 @@ const TeamDetailsV1 = ({ | |||||||
|           ? parentTeams.map((parent) => ({ |           ? parentTeams.map((parent) => ({ | ||||||
|               name: getEntityName(parent), |               name: getEntityName(parent), | ||||||
|               url: getTeamsWithFqnPath( |               url: getTeamsWithFqnPath( | ||||||
|                 parent.name ?? parent.fullyQualifiedName ?? '' |                 parent.fullyQualifiedName ?? parent.name ?? '' | ||||||
|               ), |               ), | ||||||
|             })) |             })) | ||||||
|           : []; |           : []; | ||||||
|  | |||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Rohit Jain
						Rohit Jain