sphereio / sphere-node-product-csv-sync

80%

master: 78%

LAST BUILD ON BRANCH 223-set-product-state-when-not-specified

branch: 223-set-product-state-when-not-specified

CHANGE BRANCH

x

Reset

• 223-set-product-state-when-not-specified
• 124-export-mapping-ltext
• 149-escape-double-quotes
• 173-add-info-about-localized-string-import
• 174-add-support-for-key
• 178_fix_template_header_spaces
• 187-stringify-xlsx-rich-text
• 189-master-variant-not-1
• 189-masterVariant-update
• 191-auto-split-update-actions-if-more-than-500
• 197-update-integration-tests
• 200-js-sdk
• 201-decaf
• 206-sameForAll
• 210-Changing-misleading-error-message
• 211-exporting-product-state
• 213-fix-typo-in-action
• 217-exporting-current-product-version
• 219-null-values-not-shown-in-csv-export
• 228-add-param-for-exporting-only-master-variants
• 230-conflicting-attribute-names
• 234-import-set-of-references
• 236-localisation-conflicting-attributes
• 236-specify-all-prefixed-attribute-names
• 242-use-sphere-product-csv-sync-to-sync-tier-prices
• 249-logging-entires-missing-on-product-status-during-unpublish-of-products
• 267-drop-support-for-node-v-<-8
• Add-node-v8-to-CI
• SUPPORT-5219
• TOOM-2141-full-data-export
• TOOM-2632-csv-delimiters
• TOOM-2632-delimiter-params
• add_search_keyword_delta
• batch-import-test
• bugfix/fix-broken-mapping-tests
• bugfix/match-by-custom-attributes
• categories-by-externalId
• category-by
• categoryBy_bug
• chunk-export
• cli-defaults
• configure_protocol
• daern91-patch-1
• discounted-prices
• docs(readme)-update-export-categories-description
• export-file-encoding
• export-xlsx
• export_lenum_labels
• feature/add-category-order-hint
• feature/add-variant-and-price-filters
• feature/export-external-id-category-order-hint
• feature/filter-matching-variants
• fix-duplicate-header-entries
• fix-empty-search-values
• fix-empty-values
• fix-money-formating
• fix-regex
• fix-show-error-messages-in-xlsx-product-import
• fix/reduce
• full-product-export-fix-querystring
• import-multiple-files
• import-xlsx
• io-layer-debugging
• issue-107-update-product-by-sku
• issue-117-externalid
• issue-139-remove-json-output-format
• issue-148-publish-after-import
• issue-150-price-validity
• issue-154-category-order-hints-by-external-id
• issue-158-concurrent-modification
• issue-158-documentation-change
• issue-159-number-as-boolean
• issue-165-fix-publish-after-update
• issue-204-check-for-existence-of-body-in-error
• issue-208-fix-publishing-after-update
• issue-44
• latest
• ltext-set-att-fix
• master
• match-on-import
• multiple-product-type-import-fix
• optimization
• output-encoding
• patch-access_token
• perf-update
• product-import-missing-dependency
• refactoring
• renovate/all-minor-patch
• renovate/commercetools-api-request-builder-5.x
• renovate/commercetools-sdk-client-2.x
• renovate/commercetools-sdk-middleware-auth-5.x
• renovate/commercetools-sdk-middleware-user-agent-2.x
• renovate/configure
• renovate/lock-file-maintenance
• repeat-product-info-in-variants
• revert-145-issue-117-externalid
• sku-match
• stdio
• sub-commands
• testBranch
• tests
• update-deps
• use-client
• v0.0.38
• v0.0.40
• v0.0.42
• v0.0.44
• v0.0.46
• v0.1.2
• v0.1.4
• v0.1.6
• v0.12.0
• v0.12.1
• v0.13.0
• v0.13.1
• v0.16.0
• v0.16.1
• v0.17.0
• v0.18.0
• v0.18.1
• v0.18.2
• v0.19.0
• v0.19.1
• v0.19.2
• v0.20.0
• v0.20.1
• v0.20.2
• v0.20.3
• v0.20.4
• v0.4.14
• v0.4.15
• v0.4.16
• v0.4.2
• v0.4.5
• v0.5.0
• v0.5.15
• v0.5.16
• v0.5.17
• v0.5.18
• v0.5.19
• v0.5.2
• v0.5.3
• v0.5.4
• v0.5.5
• v0.5.6
• v0.5.7
• v0.6.10
• variant-update
• wizzy25-patch-1

Build # 1350

Build Type

Pull #225

travis-ci

Committed by web-flow

Commit Message

(fix) test and implement a fix for failure to maintain state properly when processing an update without one.

This concerns the situation when a default state is provided, i.e. all products must have
states, and an update is received for a product the existing version of which has a state,
but the version from the CSV file does not.  This situation arises when both the original
and update CSV files do not specify state, and the state is assigned during the initial
import from the default state.

Before this fix, this resulted in a transitionState action being created without specifying
what state to transition to, which led to the update being rejected with "Request body does
not contain valid JSON."

Pull Request Pull Request #225: (fix) test and implement a fix for failure to maintain state properly when processing an update without one

Run Details

566 of 850 branches covered (66.59%)

1640 of 2049 relevant lines covered (80.04%)

1395.97 hits per line