[YUNIKORN-2861] - Angular upgrade to v18

[dcoric]

What is this PR for?

This PR updates the Angular version to the v18. I have done it in 2 commits - to v17 and to v18. Small changes were required with the zone.js framework import.
The only condition is - a requirement for the minimum node version: v18.19.x

What type of PR is it?

• - Bug Fix
• - Improvement
• - Feature
• - Documentation
• - Hot Fix
• - Refactoring

Todos

• - Task

What is the Jira issue?

YUNIKORN-2861 Angular upgrade to v18

How should this be tested?

I have executed the test suite and application build. Everything appears to be functional and correct.

Screenshots (if appropriate)

Questions:

• - The licenses files need update.
• - There is breaking changes for older versions.
• - It needs documentation.

[chenyulin0719]

Built web image failed. Could you also change this PR title to the Jira name? So this PR could be linked to the corresponding Jira automatically.

[dcoric]

Additional note: If the module federation PR goes through before this one, @angular-architects/module-federation dependency needs to be updated to the 18.x version that matches the rest of the dependency structure.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 38.77%. Comparing base (048f46c) to head (96dbe43).

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #203   +/-   ##
=======================================
  Coverage   38.77%   38.77%           
=======================================
  Files           2        2           
  Lines          49       49           
=======================================
  Hits           19       19           
  Misses         27       27           
  Partials        3        3           

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

[chenyulin0719]

@dcoric The PR of module federation is merged. Rebased required, thanks!

[dcoric]

@dcoric The PR of module federation is merged. Rebased required, thanks!

Thanks. I have rebased and updated minor versions. The allocations drawer test reported that MatDrawer had to be removed from declarations in allocations-drawer.component.spec.ts. Doing that fixed the failing test.

I have tested the changes by running the application in localhost, make build and make test_js

[chenyulin0719]

LGTM, checked the web after running the commands below:

• make install
• make test
• make clean image

Also checked the web container running as expected.