Run Cypress under GitHub Actions & install Node dependencies in Docker image

[osma]

Reasons for creating this PR

We want to have Cypress tests for the new frontend, and those tests should be automated as much as possible. PR #1509 changed the Cypress test to run against a standardized, dockerized Skosmos and Fuseki environment. This PR takes it a step further by also running the Cypress tests under GitHub Actions as a separate job.

In addition, while making this PR, I noticed that the Node dependencies of the frontend (e.g. Bootstrap and Vue.js) were not installed by the Dockerfile (they would only be included if they had been installed in the surrounding Skosmos codebase, and in that case often the development dependencies would be included as well). So I adjusted the Dockerfile to include a separate npm-installer stage that only installs those Node dependencies, then they are copied over to the next stage which is the Skosmos runtime image.

Link to relevant issue(s), if any

• Closes Add e2e tests #1189
• Supersedes PR Run cypress #1479 by @kinow (this PR builds on the previously merged ones which didn't yet exist when 1479 was opened; I used that PR for inspiration, but wasn't able to copy the implementation)

Description of the changes in this PR

• add GitHub Actions job that executes Cypress tests
• adjust Dockerfile.ubuntu (and .dockerignore) to make sure that the Node.js dependencies for production (Bootstrap and Vue) are included in the image, but not the development dependencies

Known problems or uncertainties in this PR

n/a

Checklist

• phpUnit tests pass locally with my changes
• I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
• The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
• The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (4af39d3) 70.51% compared to head (23fdb64) 70.51%.
Report is 3 commits behind head on skosmos-3.

Additional details and impacted files

@@             Coverage Diff              @@
##             skosmos-3    #1518   +/-   ##
============================================
  Coverage        70.51%   70.51%           
  Complexity        1642     1642           
============================================
  Files               32       32           
  Lines             4314     4314           
============================================
  Hits              3042     3042           
  Misses            1272     1272           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[miguelvaara]

I do not see any harm in adding --ignore-scripts. Let's proceed with it according to your view. Otherwise, the tests are working smoothly, everything is okay and looks good! 👍

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[osma]

CI tests are still passing, so I will merge this now.