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
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) 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

2 participants