[ME]: Fix misc issues with dashboard view

apps/metadata-editor-e2e/src/e2e/dashboard.cy.ts

@@ -187,7 +187,7 @@ describe('dashboard', () => {
       cy.get('@draft')
         .children('div')
         .eq(5)
-        .should('contain', ' Not published ')
+        .should('include.text', 'Not published')
       cy.get('@draft').children('div').eq(6).should('contain', ' - ')
       cy.clearRecordDrafts()
     })
@@ -204,23 +204,44 @@ describe('dashboard', () => {
         .find('span')
         .invoke('text')
         .should('eq', 'admin admin')
-      cy.get('@record').children('div').eq(5).should('contain', ' Published ')
+      cy.get('@record')
+        .children('div')
+        .eq(5)
+        .find('span')
+        .should('include.text', 'Published')
       cy.get('@record').children('div').eq(6).should('not.contain', ' - ')
       cy.clearRecordDrafts()
     })
   })
   describe('navigation', () => {
+    function checkDashboardFiltered() {
+      cy.get('gn-ui-autocomplete').type('Mat')
+      cy.get('mat-option').first().click()
+      cy.get('gn-ui-interactive-table')
+        .find('[data-cy="table-row"]')
+        .should('have.length', '1')
+    }
     beforeEach(() => {
       cy.login('admin', 'admin', false)
       cy.visit('/catalog/search')
     })
     describe('search input', () => {
       it('should filter the dashboard based on the search input', () => {
+        checkDashboardFiltered()
+      })
+      it('should navigate to list of all records and filter the dashboard based on the search input when on different page', () => {
+        cy.visit('/my-space/my-records')
+        checkDashboardFiltered()
+      })
+      it('should clear the search input when navigating to my records', () => {
         cy.get('gn-ui-autocomplete').type('Mat')
-        cy.get('mat-option').first().click()
-        cy.get('gn-ui-interactive-table')
-          .find('[data-cy="table-row"]')
-          .should('have.length', '1')
+        cy.get('md-editor-dashboard-menu').find('a').eq(5).click()
+        cy.get('gn-ui-autocomplete').should('have.value', '')
+      })
+      it('should clear the search input when navigating to my drafts', () => {
+        cy.get('gn-ui-autocomplete').type('Mat')
+        cy.get('md-editor-dashboard-menu').find('a').eq(6).click()
+        cy.get('gn-ui-autocomplete').should('have.value', '')
       })
     })
     describe('my records', () => {
@@ -232,6 +253,31 @@ describe('dashboard', () => {
           .next()
           .should('contain', 'admin admin')
       })
+      it('should display the correct amount of records', () => {
+        cy.get('md-editor-dashboard-menu').find('a').eq(5).click()
+        cy.get('gn-ui-results-table')
+          .find('[data-cy="table-row"]')
+          .should('have.length', '10')
+      })
+      it('should sort the records by title', () => {
+        cy.get('md-editor-dashboard-menu').find('a').eq(5).click()
+        cy.get('gn-ui-results-table')
+          .find('[data-cy="table-row"]')
+          .first()
+          .invoke('text')
+          .then((firstRecord) => {
+            console.log(firstRecord)
+            cy.get('gn-ui-results-table')
+              .find('.table-header-cell')
+              .eq(1)
+              .click()
+            cy.get('gn-ui-results-table')
+              .find('[data-cy="table-row"]')
+              .first()
+              .invoke('text')
+              .should('not.eq', firstRecord)
+          })
+      })
     })
   })
 })

apps/metadata-editor-e2e/src/e2e/edit.cy.ts

@@ -1,9 +1,10 @@
 /* eslint-disable cypress/no-unnecessary-waiting */
 
 describe('editor form', () => {
-  let recordUuid
+  let recordUuid: any
   before(() => {
     cy.login('admin', 'admin', false)
+    cy.viewport(1920, 2400)
 
     cy.clearRecordDrafts()
 
@@ -53,6 +54,7 @@ describe('editor form', () => {
   })
   beforeEach(() => {
     cy.login('admin', 'admin', false)
+    cy.visit('/catalog/search')
     cy.wrap(recordUuid).as('recordUuid')
 
     cy.get('@recordUuid').then((recordUuid) => {

apps/metadata-editor-e2e/src/e2e/record-actions.cy.ts

@@ -4,6 +4,7 @@ describe('record-actions', () => {
     cy.visit('/catalog/search')
   })
   describe('delete', () => {
+    const recordId = `TEST_RECORD_${Date.now()}`
     describe('record with draft', () => {
       it('should delete the record, delete its associated draft and refresh the interface', () => {
         // First create a record and its draft
@@ -13,13 +14,15 @@ describe('record-actions', () => {
           .as('abstractField')
           .focus()
         cy.get('@abstractField').type('record abstract')
+        cy.get('[data-test="recordTitleInput"]').click()
+        cy.get('[data-test="recordTitleInput"]').type('{selectAll}{backspace}')
+        cy.get('[data-test="recordTitleInput"]').type(recordId)
         cy.intercept({
           method: 'PUT',
           pathname: '**/records',
         }).as('insertRecord')
         cy.get('md-editor-publish-button').click()
         cy.wait('@insertRecord')
-        cy.get('@abstractField').focus()
         cy.get('@abstractField').type('draft abstract')
         // Assert that the draft exists in the local storage
         cy.editor_readFormUniqueIdentifier().then((uniqueIdentifier) =>
@@ -31,18 +34,17 @@ describe('record-actions', () => {
         )
         cy.visit('/my-space/my-records')
         cy.get('[data-cy="table-row"]')
-          .contains('My new record')
+          .contains(recordId)
           .should('have.length', 1)
         cy.get('[data-cy="dashboard-drafts-count"]').should('contain', '1')
         // Delete the record
         cy.get('[data-test="record-menu-button"]').last().click()
         cy.get('[data-test="record-menu-delete-button"]').click()
         cy.get('[data-cy="confirm-button"]').click()
         cy.get('[data-cy="table-row"]')
-          .contains('My new record')
+          .contains(recordId)
           .should('have.length', 0)
         cy.get('gn-ui-notification').should('contain', 'Delete success')
-        cy.get('[data-cy="dashboard-drafts-count"]').should('contain', '0')
       })
     })
 
@@ -51,6 +53,9 @@ describe('record-actions', () => {
         // First create a draft
         cy.get('[data-cy="create-record"]').click()
         cy.url().should('include', '/create')
+        cy.get('[data-test="recordTitleInput"]').click()
+        cy.get('[data-test="recordTitleInput"]').type('{selectAll}{backspace}')
+        cy.get('[data-test="recordTitleInput"]').type(recordId)
         cy.get('gn-ui-form-field[ng-reflect-model=abstract] textarea')
           .as('abstractField')
           .focus()
@@ -63,17 +68,14 @@ describe('record-actions', () => {
         })
         cy.visit('/my-space/my-draft')
         cy.get('[data-cy="table-row"]')
-          .contains('My new record')
+          .contains(recordId)
           .should('have.length', 1)
         cy.get('[data-cy="dashboard-drafts-count"]').should('contain', '1')
         // Delete the draft
         cy.get('[data-test="record-menu-button"]').click()
         cy.get('[data-test="record-menu-delete-button"]').click()
         cy.get('[data-cy="confirm-button"]').click()
-        cy.get('[data-cy="table-row"]')
-          .contains('New record')
-          .should('have.length', 0)
-        cy.get('[data-cy="dashboard-drafts-count"]').should('contain', '0')
+        cy.get('[data-cy="table-row"]').should('not.exist')
       })
     })
   })

apps/metadata-editor/src/app/app.routes.ts

@@ -3,14 +3,13 @@ import { DashboardPageComponent } from './dashboard/dashboard-page.component'
 import { SignInPageComponent } from './sign-in/sign-in-page.component'
 import { EditPageComponent } from './edit/edit-page.component'
 import { EditRecordResolver } from './edit-record.resolver'
-import { MyRecordsComponent } from './records/my-records/my-records.component'
 import { MyDraftComponent } from './records/my-draft/my-draft.component'
-import { MyLibraryComponent } from './records/my-library/my-library.component'
-import { SearchRecordsComponent } from './records/search-records/search-records-list.component'
+import { TemplatesComponent } from './records/templates/templates.component'
 import { MyOrgUsersComponent } from './my-org-users/my-org-users.component'
-import { MyOrgRecordsComponent } from './records/my-org-records/my-org-records.component'
 import { NewRecordResolver } from './new-record.resolver'
 import { DuplicateRecordResolver } from './duplicate-record.resolver'
+import { AllRecordsComponent } from './records/all-records/all-records.component'
+import { MyRecordsStateWrapperComponent } from './records/my-records/my-records-state-wrapper.component'
 
 export const appRoutes: Route[] = [
   { path: '', redirectTo: 'catalog/search', pathMatch: 'prefix' },
@@ -26,35 +25,30 @@ export const appRoutes: Route[] = [
       },
       {
         path: 'discussion',
-        component: SearchRecordsComponent,
+        component: AllRecordsComponent,
         pathMatch: 'prefix',
       },
       {
         path: 'calendar',
-        component: SearchRecordsComponent,
+        component: AllRecordsComponent,
         pathMatch: 'prefix',
       },
       {
         path: 'contacts',
-        component: SearchRecordsComponent,
+        component: AllRecordsComponent,
         pathMatch: 'prefix',
       },
       {
         path: 'thesaurus',
-        component: SearchRecordsComponent,
+        component: AllRecordsComponent,
         pathMatch: 'prefix',
       },
       {
         path: 'search',
         title: 'Search Records',
-        component: SearchRecordsComponent,
+        component: AllRecordsComponent,
         pathMatch: 'prefix',
       },
-      {
-        path: 'my-org',
-        title: 'My Organisation',
-        component: MyOrgRecordsComponent,
-      },
     ],
   },
   {
@@ -66,7 +60,7 @@ export const appRoutes: Route[] = [
       {
         path: 'my-records',
         title: 'My Records',
-        component: MyRecordsComponent,
+        component: MyRecordsStateWrapperComponent,
         pathMatch: 'prefix',
       },
       {
@@ -78,7 +72,7 @@ export const appRoutes: Route[] = [
       {
         path: 'templates',
         title: 'Templates',
-        component: MyLibraryComponent,
+        component: TemplatesComponent,
         pathMatch: 'prefix',
       },
     ],

apps/metadata-editor/src/app/dashboard/dashboard-menu/dashboard-menu.component.html

@@ -3,6 +3,7 @@
     <div class="menu-title" translate="">dashboard.labels.catalog</div>
     <a
       class="menu-item"
+      (click)="resetMainSearch()"
       routerLink="/catalog/search"
       routerLinkActive="btn-active"
       #rlaAll="routerLinkActive"
@@ -71,11 +72,11 @@
     >
       <mat-icon class="material-symbols-outlined">edit_note</mat-icon>
       <span translate="">dashboard.records.myDraft</span>
-      <gn-ui-badge
-        [style.--gn-ui-badge-rounded]="'100px'"
-        data-cy="dashboard-drafts-count"
-        >{{ draftsCount$ | async }}</gn-ui-badge
-      >
+      <ng-container *ngIf="draftsCount$ | async as draftsCount">
+        <gn-ui-badge *ngIf="draftsCount > 0" data-cy="dashboard-drafts-count">{{
+          draftsCount
+        }}</gn-ui-badge>
+      </ng-container>
     </a>
     <a
       class="menu-item btn-inactive"

apps/metadata-editor/src/app/dashboard/dashboard-menu/dashboard-menu.component.spec.ts

@@ -5,11 +5,13 @@ import { TranslateModule } from '@ngx-translate/core'
 import { cold, hot } from 'jasmine-marbles'
 import { MockBuilder, MockProviders } from 'ng-mocks'
 import { DashboardMenuComponent } from './dashboard-menu.component'
+import { SearchFacade } from '@geonetwork-ui/feature/search'
 
 describe('DashboardMenuComponent', () => {
   let component: DashboardMenuComponent
   let fixture: ComponentFixture<DashboardMenuComponent>
   let recordsRepository: RecordsRepositoryInterface
+  let searchFacade: SearchFacade
 
   beforeEach(() => {
     return MockBuilder(DashboardMenuComponent)
@@ -18,9 +20,12 @@ describe('DashboardMenuComponent', () => {
   beforeEach(async () => {
     await TestBed.configureTestingModule({
       imports: [DashboardMenuComponent, TranslateModule.forRoot()],
-      providers: [MockProviders(ActivatedRoute, RecordsRepositoryInterface)],
+      providers: [
+        MockProviders(ActivatedRoute, RecordsRepositoryInterface, SearchFacade),
+      ],
     }).compileComponents()
     recordsRepository = TestBed.inject(RecordsRepositoryInterface)
+    searchFacade = TestBed.inject(SearchFacade)
     fixture = TestBed.createComponent(DashboardMenuComponent)
     component = fixture.componentInstance
     fixture.detectChanges()
@@ -45,4 +50,10 @@ describe('DashboardMenuComponent', () => {
     // Assert that draftsCount$ behaves as expected
     expect(component.draftsCount$).toBeObservable(expected)
   })
+
+  it('should reset filters in main search', () => {
+    searchFacade.setFilters = jest.fn()
+    component.resetMainSearch()
+    expect(searchFacade.setFilters).toHaveBeenCalledWith({})
+  })
 })

apps/metadata-editor/src/app/dashboard/dashboard-menu/dashboard-menu.component.ts

@@ -6,6 +6,7 @@ import { TranslateModule } from '@ngx-translate/core'
 import { RecordsRepositoryInterface } from '@geonetwork-ui/common/domain/repository/records-repository.interface'
 import { map, startWith, switchMap } from 'rxjs/operators'
 import { BadgeComponent } from '@geonetwork-ui/ui/inputs'
+import { SearchFacade } from '@geonetwork-ui/feature/search'
 
 @Component({
   selector: 'md-editor-dashboard-menu',
@@ -29,5 +30,12 @@ export class DashboardMenuComponent {
   )
   activeLink = false
 
-  constructor(private recordsRepository: RecordsRepositoryInterface) {}
+  constructor(
+    private recordsRepository: RecordsRepositoryInterface,
+    private searchFacade: SearchFacade
+  ) {}
+
+  resetMainSearch() {
+    this.searchFacade.setFilters({})
+  }
 }