fix: curation API differentiates unpublished collections for revisions

[ebezzi]

For an unpublished collection that is not a revision, Curation API should return the collection_url pointing to the permalink instead of the private version. This is a requirement for the curation workflow.

[codecov]

Codecov Report

Merging #3912 (0eadad1) into staging (b4e5fb7) will decrease coverage by 22.47%.
The diff coverage is 59.02%.

❗ Current head 0eadad1 differs from pull request most recent head a7262c1. Consider uploading reports for the commit a7262c1 to get more accurate results

@@             Coverage Diff              @@
##           staging    #3912       +/-   ##
============================================
- Coverage    87.36%   64.88%   -22.48%     
============================================
  Files          141       48       -93     
  Lines         9138     3417     -5721     
============================================
- Hits          7983     2217     -5766     
- Misses        1155     1200       +45     

Flag	Coverage Δ
backend	?
python	?
unitTest	?
unittests	64.88% <59.02%> (?)

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

Impacted Files	Coverage Δ
backend/common/utils/semvar_utils.py	87.50% <ø> (ø)
backend/common/utils/type_conversion_utils.py	46.83% <ø> (ø)
...ckend/portal/pipeline/processing/h5ad_data_file.py	89.36% <ø> (ø)
backend/common/utils/secret_config.py	51.28% <12.50%> (ø)
backend/layers/persistence/persistence.py	18.88% <18.88%> (ø)
backend/layers/thirdparty/crossref_provider.py	24.46% <24.46%> (ø)
backend/common/corpora_config.py	40.00% <25.00%> (ø)
backend/common/utils/aws.py	31.42% <33.33%> (ø)
backend/common/corpora_orm.py	67.84% <39.13%> (ø)
backend/layers/processing/downloader.py	44.92% <44.92%> (ø)
... and 152 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more