-
Notifications
You must be signed in to change notification settings - Fork 14
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
fix: crawler not handling descriptions properly (#993)
- Loading branch information
1 parent
e2e41d6
commit af68e61
Showing
10 changed files
with
211 additions
and
29 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
146 changes: 146 additions & 0 deletions
146
...t/java/de/sovity/edc/ext/catalog/crawler/crawling/writing/ConnectorSuccessWriterTest.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,146 @@ | ||
/* | ||
* Copyright (c) 2023 sovity GmbH | ||
* | ||
* This program and the accompanying materials are made available under the | ||
* terms of the Apache License, Version 2.0 which is available at | ||
* https://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* SPDX-License-Identifier: Apache-2.0 | ||
* | ||
* Contributors: | ||
* sovity GmbH - initial implementation | ||
* | ||
*/ | ||
|
||
package de.sovity.edc.ext.catalog.crawler.crawling.writing; | ||
|
||
import de.sovity.edc.ext.catalog.crawler.CrawlerTestDb; | ||
import de.sovity.edc.ext.catalog.crawler.TestData; | ||
import de.sovity.edc.ext.catalog.crawler.crawling.fetching.model.FetchedCatalog; | ||
import de.sovity.edc.ext.catalog.crawler.crawling.fetching.model.FetchedContractOffer; | ||
import de.sovity.edc.ext.catalog.crawler.crawling.fetching.model.FetchedDataOffer; | ||
import de.sovity.edc.ext.catalog.crawler.db.jooq.Tables; | ||
import de.sovity.edc.ext.catalog.crawler.db.jooq.enums.ConnectorContractOffersExceeded; | ||
import de.sovity.edc.ext.catalog.crawler.db.jooq.enums.ConnectorDataOffersExceeded; | ||
import de.sovity.edc.ext.catalog.crawler.db.jooq.enums.ConnectorOnlineStatus; | ||
import de.sovity.edc.ext.wrapper.api.common.model.UiAsset; | ||
import org.assertj.core.data.TemporalUnitLessThanOffset; | ||
import org.jooq.impl.DSL; | ||
import org.junit.jupiter.api.BeforeEach; | ||
import org.junit.jupiter.api.Test; | ||
import org.junit.jupiter.api.extension.RegisterExtension; | ||
|
||
import java.time.OffsetDateTime; | ||
import java.time.temporal.ChronoUnit; | ||
import java.util.List; | ||
|
||
import static org.assertj.core.api.Assertions.assertThat; | ||
import static org.mockito.Mockito.when; | ||
|
||
class ConnectorSuccessWriterTest { | ||
@RegisterExtension | ||
private static final CrawlerTestDb TEST_DATABASE = new CrawlerTestDb(); | ||
|
||
ConnectorUpdateSuccessWriter connectorUpdateSuccessWriter; | ||
|
||
@BeforeEach | ||
void setup() { | ||
var container = new DataOfferWriterTestDydi(); | ||
connectorUpdateSuccessWriter = container.getConnectorUpdateSuccessWriter(); | ||
when(container.getCrawlerConfig().getMaxContractOffersPerDataOffer()).thenReturn(1); | ||
when(container.getCrawlerConfig().getMaxDataOffersPerConnector()).thenReturn(1); | ||
} | ||
|
||
@Test | ||
void testDataOfferWriter_fullSingleUpdate() { | ||
TEST_DATABASE.testTransaction(dsl -> { | ||
// arrange | ||
var connectorRef = TestData.connectorRef; | ||
TestData.insertConnector(dsl, connectorRef, unused -> { | ||
}); | ||
var uiAsset = UiAsset.builder() | ||
.assetId("assetId") | ||
.title("title") | ||
.description("# Description\n\n**with Markdown**") | ||
.descriptionShortText("descriptionShortText") | ||
.dataCategory("dataCategory") | ||
.dataSubcategory("dataSubCategory") | ||
.dataModel("dataModel") | ||
.transportMode("transportMode") | ||
.geoReferenceMethod("geoReferenceMethod") | ||
.keywords(List.of("a", "b")) | ||
.build(); | ||
var fetchedContractOffer = FetchedContractOffer.builder() | ||
.contractOfferId("contractOfferId") | ||
.uiPolicyJson("\"test-policy\"") | ||
.build(); | ||
var fetchedDataOffer = FetchedDataOffer.builder() | ||
.assetId("assetId") | ||
.uiAsset(uiAsset) | ||
.uiAssetJson("\"test\"") | ||
.contractOffers(List.of(fetchedContractOffer)) | ||
.build(); | ||
var fetchedCatalog = FetchedCatalog.builder() | ||
.connectorRef(connectorRef) | ||
.dataOffers(List.of(fetchedDataOffer)) | ||
.build(); | ||
|
||
// act | ||
connectorUpdateSuccessWriter.handleConnectorOnline( | ||
dsl, | ||
connectorRef, | ||
dsl.fetchOne( | ||
Tables.CONNECTOR, | ||
Tables.CONNECTOR.CONNECTOR_ID.eq(connectorRef.getConnectorId()) | ||
), | ||
fetchedCatalog | ||
); | ||
|
||
// assert | ||
var connector = dsl.fetchOne( | ||
Tables.CONNECTOR, | ||
Tables.CONNECTOR.CONNECTOR_ID.eq(connectorRef.getConnectorId()) | ||
); | ||
var dataOffer = dsl.fetchOne( | ||
Tables.DATA_OFFER, | ||
DSL.and( | ||
Tables.DATA_OFFER.CONNECTOR_ID.eq(connectorRef.getConnectorId()), | ||
Tables.DATA_OFFER.ASSET_ID.eq("assetId") | ||
) | ||
); | ||
var contractOffer = dsl.fetchOne( | ||
Tables.CONTRACT_OFFER, | ||
DSL.and( | ||
Tables.CONTRACT_OFFER.CONNECTOR_ID.eq(connectorRef.getConnectorId()), | ||
Tables.CONTRACT_OFFER.ASSET_ID.eq("assetId"), | ||
Tables.CONTRACT_OFFER.CONTRACT_OFFER_ID.eq("contractOfferId") | ||
) | ||
); | ||
|
||
var now = OffsetDateTime.now(); | ||
var minuteAccuracy = new TemporalUnitLessThanOffset(1, ChronoUnit.MINUTES); | ||
assertThat(connector).isNotNull(); | ||
assertThat(connector.getOnlineStatus()).isEqualTo(ConnectorOnlineStatus.ONLINE); | ||
assertThat(connector.getLastRefreshAttemptAt()).isCloseTo(now, minuteAccuracy); | ||
assertThat(connector.getLastSuccessfulRefreshAt()).isCloseTo(now, minuteAccuracy); | ||
assertThat(connector.getDataOffersExceeded()).isEqualTo(ConnectorDataOffersExceeded.OK); | ||
assertThat(connector.getContractOffersExceeded()).isEqualTo(ConnectorContractOffersExceeded.OK); | ||
|
||
assertThat(dataOffer).isNotNull(); | ||
assertThat(dataOffer.getAssetTitle()).isEqualTo("title"); | ||
assertThat(dataOffer.getDescriptionNoMarkdown()).isEqualTo("Description with Markdown"); | ||
assertThat(dataOffer.getShortDescriptionNoMarkdown()).isEqualTo("descriptionShortText"); | ||
assertThat(dataOffer.getDataCategory()).isEqualTo("dataCategory"); | ||
assertThat(dataOffer.getDataSubcategory()).isEqualTo("dataSubCategory"); | ||
assertThat(dataOffer.getDataModel()).isEqualTo("dataModel"); | ||
assertThat(dataOffer.getTransportMode()).isEqualTo("transportMode"); | ||
assertThat(dataOffer.getGeoReferenceMethod()).isEqualTo("geoReferenceMethod"); | ||
assertThat(dataOffer.getKeywords()).containsExactly("a", "b"); | ||
assertThat(dataOffer.getKeywordsCommaJoined()).isEqualTo("a, b"); | ||
assertThat(dataOffer.getUiAssetJson().data()).isEqualTo("\"test\""); | ||
|
||
assertThat(contractOffer).isNotNull(); | ||
assertThat(contractOffer.getUiPolicyJson().data()).isEqualTo("\"test-policy\""); | ||
}); | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters