CMR-10121 As a user I want to see entry-id (EntryId) in the UMM and/or JSON response.

[daniel-zamora]

Overview

What is the feature/fix?

Adds entry-id to atom and json collection search response.

What is the Solution?

Adds entry-id to atom and json collection search response. Updates test utils.

What areas of the application does this impact?

List impacted areas.

Checklist

• I have updated/added unit and int tests that prove my fix is effective or that my feature works
• New and existing unit and int tests pass locally and remotely
• clj-kondo has been run locally and all errors corrected
• I have removed unnecessary/dead code and imports in files I have changed
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have cleaned up integration tests by doing one or more of the following:
  • migrated any are2 tests to are3 in files I have changed
  • de-duped, consolidated, removed dead int tests
  • transformed applicable int tests into unit tests
  • refactored to reduce number of system state resets by updating fixtures (use-fixtures :each (ingest/reset-fixture {})) to be :once instead of :each

[daniel-zamora]

converted to draft so I can address all the checkboxes

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 50.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 58.17%. Comparing base (53f7115) to head (b9ebc8e).

Files with missing lines	Patch %	Lines
...r/search/results_handlers/atom_results_handler.clj	0.00%	2 Missing ⚠️
...rch/results_handlers/atom_json_results_handler.clj	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #2178      +/-   ##
==========================================
- Coverage   58.18%   58.17%   -0.01%     
==========================================
  Files        1047     1047              
  Lines       70480    70486       +6     
  Branches     2004     1999       -5     
==========================================
  Hits        41008    41008              
- Misses      27608    27613       +5     
- Partials     1864     1865       +1     

Flag	Coverage Δ
	58.18% <50.00%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[eudoroolivares2016]

Looks good to me