Skip to content

Conversation

@ChengShi-1
Copy link
Contributor

@ChengShi-1 ChengShi-1 commented Dec 10, 2025

What this PR does / why we need it:

Create Get Dataset Storage Driver use case

Which issue(s) this PR closes:

Related Dataverse PRs:

Special notes for your reviewer:

Suggestions on how to test this:

Is there a release notes or changelog update needed for this change?:

yes, changelog updated

Additional documentation:

@ChengShi-1 ChengShi-1 linked an issue Dec 11, 2025 that may be closed by this pull request
@ChengShi-1 ChengShi-1 added Size: 3 A percentage of a sprint. 2.1 hours. Original size: 3 FY25 Sprint 12 FY25 Sprint 12 (2024-12-04 - 2024-12-18) FY26 Sprint 12 FY26 Sprint 12 (2025-12-03 - 2025-12-17) GREI Re-arch GREI re-architecture-related and removed FY25 Sprint 12 FY25 Sprint 12 (2024-12-04 - 2024-12-18) labels Dec 11, 2025
@ChengShi-1 ChengShi-1 moved this to In Progress 💻 in IQSS Dataverse Project Dec 11, 2025
@ChengShi-1 ChengShi-1 marked this pull request as ready for review December 11, 2025 18:16
Copilot AI review requested due to automatic review settings December 11, 2025 18:16
@ChengShi-1 ChengShi-1 moved this from In Progress 💻 to Ready for Review ⏩ in IQSS Dataverse Project Dec 11, 2025
@ChengShi-1 ChengShi-1 added the SPA.Q4.2025.9 Storage Driver label Dec 11, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a new use case to retrieve storage driver configuration information for datasets, supporting the Dataverse API endpoint GET /datasets/{identifier}/storageDriver.

  • Adds GetDatasetStorageDriver use case and StorageDriver model interface
  • Implements repository method in DatasetsRepository and interface in IDatasetsRepository
  • Includes comprehensive unit and integration tests for the new functionality

Reviewed changes

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

Show a summary per file
File Description
src/datasets/domain/models/StorageDriver.ts Defines the StorageDriver interface with properties for storage configuration
src/datasets/domain/useCases/GetDatasetStorageDriver.ts Implements the use case to retrieve storage driver information for a dataset
src/datasets/domain/repositories/IDatasetsRepository.ts Adds method signature for getDatasetStorageDriver to the repository interface
src/datasets/infra/repositories/DatasetsRepository.ts Implements the getDatasetStorageDriver method to call the API endpoint
src/datasets/index.ts Exports the new getDatasetStorageDriver use case
test/unit/datasets/GetDatasetStorageDriver.test.ts Adds unit tests for the use case with success and error scenarios
test/integration/datasets/DatasetsRepository.test.ts Adds integration test to verify the repository method works with the API
docs/useCases.md Documents the new use case with usage examples and parameter descriptions
CHANGELOG.md Records the addition of the new feature

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

@ChengShi-1

This comment was marked as resolved.

This comment was marked as resolved.

@ChengShi-1 ChengShi-1 moved this from Ready for Review ⏩ to In Progress 💻 in IQSS Dataverse Project Dec 12, 2025
@ChengShi-1 ChengShi-1 moved this from In Progress 💻 to Ready for Review ⏩ in IQSS Dataverse Project Dec 13, 2025
@cmbz cmbz added the FY26 Sprint 13 FY26 Sprint 13 (2025-12-17 - 2025-12-31) label Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

FY26 Sprint 12 FY26 Sprint 12 (2025-12-03 - 2025-12-17) FY26 Sprint 13 FY26 Sprint 13 (2025-12-17 - 2025-12-31) GREI Re-arch GREI re-architecture-related Original size: 3 Size: 3 A percentage of a sprint. 2.1 hours. SPA.Q4.2025.9 Storage Driver

Projects

Status: Ready for Review ⏩

Development

Successfully merging this pull request may close these issues.

Get Dataset Storage Driver use case

3 participants