Fixing missing faces, also adding E2E tests to test them #704, fixes #713

src/frontend/app/ui/faces/face/face.component.html

@@ -7,10 +7,10 @@
        [style.height.px]="size">
 
     <div class="photo"
-         *ngIf="thumbnail && thumbnail.Available && false"
+         *ngIf="thumbnail && thumbnail.Available"
          [style.background-image]="getSanitizedThUrl()"></div>
 
-    <ng-icon *ngIf="!thumbnail || !thumbnail.Available || true"
+    <ng-icon *ngIf="!thumbnail || !thumbnail.Available"
              class="no-image" name="ionPersonOutline"></ng-icon>
   </div>
 

test/cypress/e2e/faces.cy.ts

@@ -0,0 +1,28 @@
+describe('Faces', () => {
+  beforeEach(() => {
+    cy.visit('/');
+    cy.get('.card-body');
+    cy.get('.col-sm-12').contains('Login');
+    /* ==== Generated with Cypress Studio ==== */
+    cy.get('#username').type('admin');
+    cy.get('#password').clear();
+    cy.get('#password').type('admin');
+    cy.intercept({
+      method: 'Get',
+      url: '/pgapi/person',
+    }).as('getPerson');
+    cy.get('.col-sm-12 > .btn').click();
+    cy.get('nav .nav-item .nav-link').contains('Faces').click();
+  });
+  it('Show faces', () => {
+    cy.wait('@getPerson', {timeout: 10000});
+    // contains a folder
+    cy.get('app-face  a > .info').contains('Alvin the Squirrel').should('exist');
+  });
+  it('Faces should have photos', () => {
+    cy.wait('@getPerson', {timeout: 10000});
+    // should have a photo
+    cy.get('app-face .photo-container .photo', {timeout: 10000}).should('exist');
+  });
+
+});