[Test] Fix up E2E tests according to starting duplicate workspaces (#23611)

* Update 'Factory' test
* Remove unnecessary variables
* Update 'getProjectNameFromGitUrl' method

Author: artaleks9

tests/e2e/constants/BASE_TEST_CONSTANTS.ts

@@ -16,8 +16,6 @@ export const BASE_TEST_CONSTANTS: {
 	OCP_INFRA: string;
 	DELETE_WORKSPACE_ON_FAILED_TEST: boolean;
 	DELETE_WORKSPACE_ON_SUCCESSFUL_TEST: boolean;
-	SELECT_OPENING_EXISTING_WORKSPACE_INSTEAD_OF_CREATION_NEW: boolean;
-	SELECT_CREATING_NEW_WORKSPACE: boolean;
 	IS_CLUSTER_DISCONNECTED: () => boolean;
 	IS_PRODUCT_DOCUMENTATION_RELEASED: any;
 	OCP_VERSION: string;
@@ -139,19 +137,6 @@ export const BASE_TEST_CONSTANTS: {
 	 */
 	DELETE_WORKSPACE_ON_SUCCESSFUL_TEST: process.env.DELETE_WORKSPACE_ON_SUCCESSFUL_TEST !== 'false',
 
-	/**
-	 * select opening an existing workspace instead of creating a new one, if a duplicate workspace is created from the factory or sample list.
-	 * this option is false by default.
-	 */
-	SELECT_OPENING_EXISTING_WORKSPACE_INSTEAD_OF_CREATION_NEW:
-		process.env.SELECT_OPENING_EXISTING_WORKSPACE_INSTEAD_OF_CREATION_NEW === 'true',
-
-	/**
-	 * select creation of a new workspace, if a duplicate workspace is created from the factory or sample list.
-	 * this option is false by default.
-	 */
-	SELECT_CREATING_NEW_WORKSPACE: process.env.SELECT_CREATING_NEW_WORKSPACE === 'true',
-
 	/**
 	 * constant, which prolong timeout constants for local debug.
 	 */

tests/e2e/constants/FACTORY_TEST_CONSTANTS.ts

@@ -1,5 +1,5 @@
 /** *******************************************************************
- * copyright (c) 2020-2023 Red Hat, Inc.
+ * copyright (c) 2020-2025 Red Hat, Inc.
  *
  * This program and the accompanying materials are made
  * available under the terms of the Eclipse Public License 2.0
@@ -74,8 +74,7 @@ export const FACTORY_TEST_CONSTANTS: {
 	 */
 	TS_SELENIUM_FACTORY_URL(): string {
 		return (
-			process.env.TS_SELENIUM_FACTORY_URL ||
-			BASE_TEST_CONSTANTS.TS_SELENIUM_BASE_URL + '/dashboard/#/' + this.TS_SELENIUM_FACTORY_GIT_REPO_URL
+			process.env.TS_SELENIUM_FACTORY_URL || BASE_TEST_CONSTANTS.TS_SELENIUM_BASE_URL + '/#/' + this.TS_SELENIUM_FACTORY_GIT_REPO_URL
 		);
 	}
 };

tests/e2e/specs/SmokeTest.spec.ts

@@ -1,5 +1,5 @@
 /** *******************************************************************
- * copyright (c) 2023 Red Hat, Inc.
+ * copyright (c) 2023-2025 Red Hat, Inc.
  *
  * This program and the accompanying materials are made
  * available under the terms of the Eclipse Public License 2.0

tests/e2e/specs/dashboard-samples/RecommendedExtensions.spec.ts

@@ -253,7 +253,7 @@ for (const sample of samples) {
 		let extensionSection: ExtensionsViewSection;
 		let extensionsView: SideBarView | undefined;
 		let publisherNames: string[];
-		let skipSuite: boolean = false;
+		const skipSuite: boolean = false;
 
 		const [pathToExtensionsListFileName, extensionsListFileName]: string[] = ['.vscode', 'extensions.json'];
 

tests/e2e/specs/factory/Factory.spec.ts

@@ -78,20 +78,6 @@ suite(
 			});
 		}
 
-		if (BASE_TEST_CONSTANTS.SELECT_OPENING_EXISTING_WORKSPACE_INSTEAD_OF_CREATION_NEW) {
-			test('Select opening an existing workspace instead of creating a new one', async function (): Promise<void> {
-				await dashboard.waitExistingWorkspaceFoundAlert();
-				await dashboard.clickOpenExistingWorkspaceButton();
-			});
-		}
-
-		if (BASE_TEST_CONSTANTS.SELECT_CREATING_NEW_WORKSPACE) {
-			test('Select of creating a new workspace', async function (): Promise<void> {
-				await dashboard.waitExistingWorkspaceFoundAlert();
-				await dashboard.clickOnCreateNewWorkspaceButton();
-			});
-		}
-
 		test('Obtain workspace name from workspace loader page', async function (): Promise<void> {
 			await workspaceHandlingTests.obtainWorkspaceNameFromStartingPage();
 		});

tests/e2e/utils/StringUtil.ts

@@ -1,5 +1,5 @@
 /** *******************************************************************
- * copyright (c) 2019 Red Hat, Inc.
+ * copyright (c) 2019-2025 Red Hat, Inc.
  *
  * This program and the accompanying materials are made
  * available under the terms of the Eclipse Public License 2.0
@@ -14,27 +14,40 @@ import { injectable } from 'inversify';
 @injectable()
 export class StringUtil {
 	/**
-	 * method extracts a test repo name from git clone https url;
-	 * it splits the url into string[] by "/" or ".", deletes empty elements and elements that contains just "git", "main" or "tree" word, than returns the last one;
-	 * please, avoid to call the test repo as just "git" or to use dots in the name, like: github.com/user/git.git, github.com/user/name.with.dots.
-	 * @param url git https url (which using for "git clone")
-	 * @return project name
+	 * extracts the project (repository) name from a Git HTTPS URL.
+	 *
+	 * Handles URLs from GitLab, GitHub, and Bitbucket by:
+	 * - Removing query parameters and hash fragments;
+	 * - Trimming branch or tree paths (e.g. `/tree/<branch>`, `/-/tree/<branch>`, `/src/<branch>`);
+	 * - Stripping the `.git` suffix.
+	 *
+	 * ⚠️ Avoid naming repositories as `"git"` or using dots in names (`name.with.dots`),
+	 * as they may be treated as separators.
+	 *
+	 * @param url Git HTTPS URL (as used for `git clone`)
+	 * @returns Extracted project name, or an empty string if parsing fails
 	 */
 	static getProjectNameFromGitUrl(url: string): string {
-		Logger.debug(`${url}`);
+		Logger.debug(`Original URL: ${url}`);
 
-		if (url.includes('?')) {
-			url = url.substring(0, url.indexOf('?'));
-		}
-		if (url.includes('/tree/')) {
-			url = url.split('/').slice(0, -2).join('/');
+		try {
+			// remove query and hash fragments
+			url = url.split('?')[0].split('#')[0];
+
+			// remove branch/tree path fragments for major providers
+			url = url
+				.replace(/\/-?\/tree\/[^/]+$/, '') // gitLab, GitHub
+				.replace(/\/src\/[^/]+.*$/, ''); // bitbucket
+
+			// take the last segment of the path and strip ".git"
+			const projectName: string = (url.split('/').filter(Boolean).pop() || '').replace(/\.git$/, '');
+
+			Logger.debug(`Extracted project name: ${projectName}`);
+			return projectName;
+		} catch (err) {
+			Logger.error(`Failed to extract project name from URL ${url}: ${err}`);
+			return '';
 		}
-		const projectName: string = url
-			.split(/[\/.]/)
-			.filter((e: string): boolean => !['git', ''].includes(e))
-			.reverse()[0];
-		Logger.debug(`${projectName}`);
-		return projectName;
 	}
 
 	static sanitizeTitle(arg: string): string {