ME: delete a record and its draft, delete a draft without record

[LHBruneton-C2C]

Description

This PR introduces deleting of a record and its draft, or a draft without, but not rollback of draft with a record.

Architectural changes

The testing tool ng-mocks was added to the devDeps, and used in the files already concerned by this PR.

Screenshots

Quality Assurance Checklist

• Commit history is devoid of any merge commits and readable to facilitate reviews
• If new logic ⚙️ is introduced: unit tests were added
• If new user stories 🤏 are introduced: E2E tests were added
• If new UI components 🕹️ are introduced: corresponding stories in Storybook were created
• If breaking changes 🪚 are introduced: add the breaking change label
• If bugs 🐞 are fixed: add the backport <release branch> label
• The documentation website 📚 has received the love it deserves

[github-actions]

Affected libs: api-metadata-converter, feature-search, feature-router, feature-map, feature-dataviz, feature-record, api-repository, feature-catalog, feature-editor, feature-auth, ui-search, common-domain, common-fixtures, util-shared, ui-elements, feature-notifications, ui-catalog, ui-widgets, ui-inputs, ui-layout, ui-map, ui-dataviz, data-access-datafeeder, util-data-fetcher, data-access-gn4, util-app-config, util-i18n,
Affected apps: metadata-editor, metadata-converter, datahub, demo, webcomponents, map-viewer, search, datafeeder, data-platform,

• 🚀 Build and deploy storybook and demo on GitHub Pages
• 📦 Build and push affected docker images

[github-actions]

📷 Screenshots are here!

[LHBruneton-C2C]

@Angi-Kinas If you want, you can do a first review of this PR. I'll try to setup ng-mocks in a dedicated commit.

[Angi-Kinas]

Thanks @LHBruneton-C2C for the clean code and the ng-mocks (which looks like a game changer for the unit tests).

I left some comments, but haven't tested the functionality yet. But code wise it looks good 👍

[coveralls]

coverage: 83.548% (+0.6%) from 82.966%
when pulling eeb79bd on ME/delete-record
into 618ec56 on main.

[jahow]

@LHBruneton-C2C thank you for starting with ng-mocks, don't forget to include this in the best practices (to rename "code guide") in the docs!

[jahow]

Thank you, very good work!! Also nice job on the E2E tests 🙂

I made a few comments on top of what we discussed. I'll give a last review once you make the modifications on the records repository. Please let me know if anything is unclear. Thanks!

[jahow]

oh also feel free to rebase to have the e2e tests actually working from time to time :)

[LHBruneton-C2C]

TODO: documentation!!!