Fixing details loading for datasets

[klpoland]

SFDS-308

In this PR:

• Added test coverage for publishing, more specific details population tests (for owner and shared users), more specific sharing action tests
• Remove redundant/conflicting dataset details loading flow
• Add logic for finding files when user is not the owner (previously ONLY used sanitize_path_rel_to_user which assumes path is owned by user)

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
gateway/sds_gateway/static/js/actions/__tests__/DownloadActionManager.test.js	10% smaller
gateway/sds_gateway/static/js/actions/__tests__/ShareActionManager.test.js	5% smaller
gateway/sds_gateway/static/js/actions/DetailsActionManager.js	3% smaller
gateway/sds_gateway/static/js/actions/__tests__/DetailsActionManager.test.js	2% smaller
gateway/sds_gateway/static/js/actions/PublishActionManager.js	0% smaller
gateway/sds_gateway/static/js/actions/__tests__/PublishActionManager.test.js	0% smaller
gateway/sds_gateway/static/js/share/__tests__/ShareGroupManager.test.js	0% smaller
gateway/sds_gateway/static/js/tests-config/jest.config.js	0% smaller
gateway/sds_gateway/static/js/tests-config/jest.setup.js	0% smaller
gateway/sds_gateway/users/views.py	0% smaller

[Copilot]

Pull request overview

This PR fixes dataset details loading (especially for shared datasets) by updating server-side file tree base directory selection and refactoring client-side dataset modal loading, alongside expanded Jest test coverage for publish/sharing/details/download flows.

Changes:

• Update dataset details endpoint to determine file-tree base directory differently for shared vs owned datasets.
• Refactor DetailsActionManager to rely on Bootstrap modal show events for dataset details loading and remove redundant dataset-button handling.
• Add/expand Jest setup and multiple new/updated tests (publish/share/details/download) and adjust coverage exclusions.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
gateway/sds_gateway/users/views.py	Adjusts dataset details file-tree base directory logic for shared datasets.
gateway/sds_gateway/static/js/tests-config/jest.setup.js	Makes window.location.href settable in tests without navigation.
gateway/sds_gateway/static/js/tests-config/jest.config.js	Excludes deprecated/legacy JS files from coverage collection.
gateway/sds_gateway/static/js/share/tests/ShareGroupManager.test.js	Adds tests for rendering shared-assets info (filter/sort/empty).
gateway/sds_gateway/static/js/actions/tests/ShareActionManager.test.js	Adds more granular tests for searching/selecting users and sharing payloads.
gateway/sds_gateway/static/js/actions/tests/PublishActionManager.test.js	Adds new tests for publish behavior and error handling.
gateway/sds_gateway/static/js/actions/tests/DownloadActionManager.test.js	Improves dataset/capture download tests and DOMUtils mocking.
gateway/sds_gateway/static/js/actions/tests/DetailsActionManager.test.js	Adds dataset modal loading tests for owner and shared viewers.
gateway/sds_gateway/static/js/actions/PublishActionManager.js	Exposes PublishActionManager on window and via module.exports for Jest.
gateway/sds_gateway/static/js/actions/DetailsActionManager.js	Removes dataset details button flow; loads dataset details on modal show; awaits async file-tree rendering.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.