chore: Remove test data generation for system and admin (groups, project, users) (DEV-3523)

[seakayone]

Pull Request Checklist

Task Description/Number

Removes following folders from the client-test-data.zip:

/system/health
/admin/groups
/admin/projects
/admin/users

Issue Number: DEV-

PR Type

• build/chore: maintenance tasks (no production code change)
• docs: documentation changes (no production code change)
• feat: represents new features
• fix: represents bug fixes
• perf: performance improvements
• refactor: represents production code refactoring
• test: adding or refactoring tests (no production code change)
• deprecated: Deprecation warning (ideally referencing a migration guide)

Basic requirements for bug fixes and features

• Tests for the changes have been added
• Docs have been added / updated

Does this PR introduce a breaking change?

• Yes

Does this PR change client-test-data?

• Yes

[linear]

DEV-3523 Remove system and admin (project, user, group) client test data generation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.93%. Comparing base (a622e4f) to head (79b505b).
Report is 28 commits behind head on main.

❗ Current head 79b505b differs from pull request most recent head 630a51d. Consider uploading reports for the commit 630a51d to get more accurate results

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #3203       +/-   ##
===========================================
+ Coverage   13.22%   86.93%   +73.70%     
===========================================
  Files         270      272        +2     
  Lines       22244    22220       -24     
===========================================
+ Hits         2941    19316    +16375     
+ Misses      19303     2904    -16399     

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

[siers]

Why? Is it because client test data is too big? Are the tests also not relevant on top of that?

[BalduinLandolt]

I'm all for removing these (and generally the client test data), but I'm curious on the selection: Why this particular part of the test data? Was there a concrete motivation for it? Or is it just intended as the first step of removing all the client test data?

[seakayone]

Why ?
Why this particular set?

See the meeting minutes in https://linear.app/dasch/issue/DEV-3520/dsp-api-remove-generation-of-test-data-for-js-lib

We have identified this set of test data as not being used anymore. So there is not need to collect them anymore.

[BalduinLandolt]

Why ?
Why this particular set?

See the meeting minutes in https://linear.app/dasch/issue/DEV-3520/dsp-api-remove-generation-of-test-data-for-js-lib

We have identified this set of test data as not being used anymore. So there is not need to collect them anymore.

That makes a lot of sense. Thanks for the clarification!