[5737] Fix workbench sharing URL to include all the opened representations

Bug: #5737
Signed-off-by: Pierre-Charles David <[email protected]>

Author: pcdavid

CHANGELOG.adoc

@@ -121,6 +121,7 @@ Use `org.eclipse.emf.ecore.util.ECrossReferenceAdapter.getCrossReferenceAdapter(
 - https://github.com/eclipse-sirius/sirius-web/issues/5695[#5695] [diagram] Correctly update the selection when opening or closing the palette
 - https://github.com/eclipse-sirius/sirius-web/issues/5694[#5694] [trees] Ensure that expand all is available in read only workbenches
 - https://github.com/eclipse-sirius/sirius-web/issues/5245[#5245] [sirius-web] Ensure that representations cannot be forked in read only workbenches
+- https://github.com/eclipse-sirius/sirius-web/issues/5737[#5737] [sirius-web] Fix a regression where sharing a workbench URL did not include all the opened representations
 
 
 === New Features

integration-tests-cypress/cypress/e2e/project/workbench/workbench-configuration.cy.ts

@@ -172,6 +172,23 @@ describe('Workbench Configuration Resolution', () => {
           workbench = new Workbench();
         });
 
+        it('Then the share URL contains the workbench configuration', () => {
+          // Wait for the diagram to be actually loaded, otherwise if we start interacting
+          // with the project menu and *then* the diagram appears, the menu does not open.
+          cy.getByTestId('FreeForm - Motion_Engine').should('be.visible');
+          cy.getByTestId(`navbar-title`).should('be.visible');
+          cy.getByTestId('navigation-bar').findByTestId('more').should('exist').click();
+          cy.getByTestId('share-project').should('exist').click();
+          cy.getByTestId('share-path')
+            .should('exist')
+            .should(
+              'include.text',
+              encodeURIComponent(
+                `"representationEditors":[{"representationId":"${topography2Id}","isActive":true},{"representationId":"${topography1Id}","isActive":false}]`
+              )
+            );
+        });
+
         it('Then, in the URL, the "workbenchConfiguration" search param is removed', () => {
           cy.url().should('not.include', 'workbenchConfiguration=');
         });

integration-tests-cypress/cypress/pages/Project.ts

@@ -77,7 +77,7 @@ class ProjectNavigationBar {
   }
 
   public getShareButton(): Cypress.Chainable<JQuery<HTMLElement>> {
-    return cy.getByTestId('share');
+    return cy.getByTestId('share-project');
   }
 
   public getShareDialog(): ShareProjectDialog {

packages/core/frontend/sirius-components-core/src/workbench/Workbench.tsx

@@ -293,6 +293,7 @@ export const Workbench = forwardRef<WorkbenchHandle | null, WorkbenchProps>(
               representations={state.representationsMetadata}
               displayedRepresentation={displayedRepresentationMetadata}
               onRepresentationClick={onRepresentationClick}
+              ref={refRepresentationNavigationHandle}
               onClose={onClose}
             />
             <RepresentationComponent

packages/sirius-web/frontend/sirius-web-application/src/views/edit-project/navbar/context-menu/ShareProjectMenuItem.tsx

@@ -26,7 +26,7 @@ export const ShareProjectMenuItem = ({ workbenchHandle }: ShareProjectMenuItemPr
 
   return (
     <>
-      <MenuItem onClick={() => setState((prevState) => ({ ...prevState, isOpen: true }))} data-testid="share">
+      <MenuItem onClick={() => setState((prevState) => ({ ...prevState, isOpen: true }))} data-testid="share-project">
         <ListItemIcon>
           <ShareIcon />
         </ListItemIcon>