Skip to content

docs: add OpenCart option price adjustment fixture - #1033

Open
RujulaSharma wants to merge 1 commit into
Posnic:mainfrom
RujulaSharma:docs/opencart-option-price-fixture
Open

RujulaSharma wants to merge 1 commit into
Posnic:mainfrom
RujulaSharma:docs/opencart-option-price-fixture

Conversation

@RujulaSharma

Copy link
Copy Markdown
Contributor

Summary

  • Add synthetic OpenCart product and order fixtures for option price adjustments.
  • Cover positive, negative, and zero option price deltas.
  • Document expected POS line-item mappings for quantity, price, discount, tax, and totals.
  • Add unsupported option combination fixtures.
  • Reference official OpenCart documentation.
  • Add deterministic tests for the fixture behavior.

Testing

  • node --test tests/opencart-option-price-fixture.test.js
  • 4 tests passed

Scope

This PR is limited to synthetic fixture data, documentation, and tests.

It does not implement an OpenCart API client, live store integration,
credentials, scraping, or real merchant-data ingestion.

Closes #426

@sridharkalaibala sridharkalaibala left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the focused, synthetic OpenCart option-pricing fixture and boundary documentation. I verified the focused test passes (4/4). I am holding approval because the current merge result has a failing Packaging checks job. Please rebase or rerun once that repository-level check is green, then we can complete review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants