-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate ProductReferenceResolver #937
Migrate ProductReferenceResolver #937
Conversation
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…va-sdk-v2-product-reference-resolver # Conflicts: # src/main/java/com/commercetools/sync/sdk2/products/utils/ProductReferenceResolutionUtils.java # src/test/java/com/commercetools/sync/sdk2/products/utils/ProductReferenceResolutionUtilsTest.java
Codecov Report
@@ Coverage Diff @@
## java-sdk-v2-product-sync-migration #937 +/- ##
========================================================================
- Coverage 96.48% 95.84% -0.65%
- Complexity 3804 3899 +95
========================================================================
Files 334 337 +3
Lines 10816 11374 +558
Branches 647 695 +48
========================================================================
+ Hits 10436 10901 +465
- Misses 280 364 +84
- Partials 100 109 +9
|
* Migrate WaitingToBeResolved class and tests * Update comment
…oduct-reference-resolver
…oduct-reference-resolver # Conflicts: # src/test/java/com/commercetools/sync/sdk2/products/ProductSyncMockUtils.java
# Conflicts: # build.gradle # gradle-scripts/spotless.gradle
# Conflicts: # build.gradle # gradle-scripts/spotless.gradle
…2-product-sync-and-tests
…dk2/ctpprojectsource/products/ProductSyncIT.java Co-authored-by: Sarah Lander <70885646+salander85@users.noreply.github.com>
…oduct-reference-resolver
…oduct-reference-resolver # Conflicts: # build.gradle
…solver' into java-sdk-v2-product-reference-resolver
getReferencesWithIds(ensureCategories(CTP_SOURCE_CLIENT, getCategoryDrafts(null, 2))); | ||
|
||
ensureProductType(PRODUCT_TYPE_RESOURCE_PATH, CTP_TARGET_CLIENT); | ||
// ensureProductType(PRODUCT_TYPE_NO_KEY_RESOURCE_PATH, CTP_TARGET_CLIENT); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.thenApply(ProductProjectionPagedQueryResponse::getResults) | ||
.join(); | ||
|
||
final List<ProductDraft> productDrafts = // mapToProductDrafts(products); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is commented multiple times. I think you can remove it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary
Migrate ProductReferenceResolver and related tests.