test: add functional and integration

Author: g-saracca

test/functional/datasets/LinkDataset.test.ts

@@ -0,0 +1,59 @@
+import { ApiConfig, createDataset, linkDataset, WriteError } from '../../../src'
+import { DataverseApiAuthMechanism } from '../../../src/core/infra/repositories/ApiConfig'
+import {
+  createCollectionViaApi,
+  deleteCollectionViaApi
+} from '../../testHelpers/collections/collectionHelper'
+import { deleteUnpublishedDatasetViaApi } from '../../testHelpers/datasets/datasetHelper'
+import { TestConstants } from '../../testHelpers/TestConstants'
+
+describe('execute', () => {
+  const testCollectionAlias = 'linkDatasetFunctionalTestCollection'
+  beforeEach(async () => {
+    ApiConfig.init(
+      TestConstants.TEST_API_URL,
+      DataverseApiAuthMechanism.API_KEY,
+      process.env.TEST_API_KEY
+    )
+  })
+
+  it('should link a dataset to another collection', async () => {
+    const createdDatasetIdentifiers = await createDataset.execute(
+      TestConstants.TEST_NEW_DATASET_DTO
+    )
+    await createCollectionViaApi(testCollectionAlias)
+
+    const result = await linkDataset.execute(
+      createdDatasetIdentifiers.numericId,
+      testCollectionAlias
+    )
+
+    expect(result).toBeUndefined()
+
+    await deleteUnpublishedDatasetViaApi(createdDatasetIdentifiers.numericId)
+    await deleteCollectionViaApi(testCollectionAlias)
+  })
+
+  it('should throw an error when trying to link a dataset to a non-existent collection', async () => {
+    const createdDatasetIdentifiers = await createDataset.execute(
+      TestConstants.TEST_NEW_DATASET_DTO
+    )
+    const nonExistentCollectionAlias = 'nonExistentCollection'
+
+    await expect(
+      linkDataset.execute(createdDatasetIdentifiers.numericId, nonExistentCollectionAlias)
+    ).rejects.toBeInstanceOf(WriteError)
+
+    await deleteUnpublishedDatasetViaApi(createdDatasetIdentifiers.numericId)
+  })
+
+  it('should throw an error when trying to link a dataset that does not exist', async () => {
+    await createCollectionViaApi(testCollectionAlias)
+    const nonExistentDatasetId = 'nonExistentDatasetId'
+    await expect(
+      linkDataset.execute(nonExistentDatasetId, testCollectionAlias)
+    ).rejects.toBeInstanceOf(WriteError)
+
+    await deleteCollectionViaApi(testCollectionAlias)
+  })
+})

test/functional/datasets/UnlinkDataset.test.ts

@@ -0,0 +1,51 @@
+import { ApiConfig, createDataset, linkDataset, unlinkDataset, WriteError } from '../../../src'
+import { DataverseApiAuthMechanism } from '../../../src/core/infra/repositories/ApiConfig'
+import {
+  createCollectionViaApi,
+  deleteCollectionViaApi
+} from '../../testHelpers/collections/collectionHelper'
+import { deleteUnpublishedDatasetViaApi } from '../../testHelpers/datasets/datasetHelper'
+import { TestConstants } from '../../testHelpers/TestConstants'
+
+describe('execute', () => {
+  const testCollectionAlias = 'unlinkDatasetFunctionalTestCollection'
+  beforeEach(async () => {
+    ApiConfig.init(
+      TestConstants.TEST_API_URL,
+      DataverseApiAuthMechanism.API_KEY,
+      process.env.TEST_API_KEY
+    )
+  })
+
+  it('should unlink a dataset from a collection', async () => {
+    const createdDatasetIdentifiers = await createDataset.execute(
+      TestConstants.TEST_NEW_DATASET_DTO
+    )
+    await createCollectionViaApi(testCollectionAlias)
+
+    await linkDataset.execute(createdDatasetIdentifiers.numericId, testCollectionAlias)
+
+    const result = await unlinkDataset.execute(
+      createdDatasetIdentifiers.numericId,
+      testCollectionAlias
+    )
+
+    expect(result).toBeUndefined()
+
+    await deleteUnpublishedDatasetViaApi(createdDatasetIdentifiers.numericId)
+    await deleteCollectionViaApi(testCollectionAlias)
+  })
+
+  it('should throw error when dataset is not linked to the collection', async () => {
+    const createdDatasetIdentifiers = await createDataset.execute(
+      TestConstants.TEST_NEW_DATASET_DTO
+    )
+    await createCollectionViaApi(testCollectionAlias)
+
+    await expect(
+      unlinkDataset.execute(createdDatasetIdentifiers.numericId, testCollectionAlias)
+    ).rejects.toBeInstanceOf(WriteError)
+
+    await deleteCollectionViaApi(testCollectionAlias)
+  })
+})

test/integration/datasets/DatasetsRepository.test.ts

@@ -1388,4 +1388,77 @@ describe('DatasetsRepository', () => {
       await expect(sut.deleteDatasetDraft(nonExistentTestDatasetId)).rejects.toThrow(expectedError)
     })
   })
+
+  describe('linkDataset', () => {
+    let testDatasetIds: CreatedDatasetIdentifiers
+    const testCollectionAlias = 'testLinkDatasetCollection'
+
+    beforeAll(async () => {
+      testDatasetIds = await createDataset.execute(TestConstants.TEST_NEW_DATASET_DTO)
+      await createCollectionViaApi(testCollectionAlias)
+    })
+
+    afterAll(async () => {
+      await deletePublishedDatasetViaApi(testDatasetIds.persistentId)
+      await deleteCollectionViaApi(testCollectionAlias)
+    })
+
+    test('should link a dataset to another collection', async () => {
+      const actual = await sut.linkDataset(testDatasetIds.numericId, testCollectionAlias)
+
+      expect(actual).toBeUndefined()
+
+      // TODO:ME - Once we get linked dataset collections use case assert that the collection exists
+    })
+
+    test('should return error when dataset does not exist', async () => {
+      await expect(sut.linkDataset(nonExistentTestDatasetId, testCollectionAlias)).rejects.toThrow()
+    })
+
+    test('should return error when collection does not exist', async () => {
+      await expect(
+        sut.linkDataset(testDatasetIds.numericId, 'nonExistentCollectionAlias')
+      ).rejects.toThrow()
+    })
+  })
+
+  describe('unlinkDataset', () => {
+    let testDatasetIds: CreatedDatasetIdentifiers
+    const testCollectionAlias = 'testUnlinkDatasetCollection'
+
+    beforeAll(async () => {
+      testDatasetIds = await createDataset.execute(TestConstants.TEST_NEW_DATASET_DTO)
+      await createCollectionViaApi(testCollectionAlias)
+    })
+
+    afterAll(async () => {
+      await deletePublishedDatasetViaApi(testDatasetIds.persistentId)
+      await deleteCollectionViaApi(testCollectionAlias)
+    })
+
+    test('should unlink a dataset from a collection', async () => {
+      await sut.linkDataset(testDatasetIds.numericId, testCollectionAlias)
+      const actual = await sut.unlinkDataset(testDatasetIds.numericId, testCollectionAlias)
+
+      expect(actual).toBeUndefined()
+
+      // TODO:ME - Once we get linked dataset collections use case assert that the collection exists
+    })
+
+    test('should return error when dataset does not exist', async () => {
+      await expect(sut.linkDataset(nonExistentTestDatasetId, testCollectionAlias)).rejects.toThrow()
+    })
+
+    test('should return error when collection does not exist', async () => {
+      await expect(
+        sut.linkDataset(testDatasetIds.numericId, 'nonExistentCollectionAlias')
+      ).rejects.toThrow()
+    })
+
+    test('should return error when dataset is not linked to the collection', async () => {
+      await expect(
+        sut.unlinkDataset(testDatasetIds.numericId, testCollectionAlias)
+      ).rejects.toThrow()
+    })
+  })
 })