Skip to content
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

When amending an OrderQuote from Opportunities X to Opportunities Y, the capacity on X do NOT get restored #168

Open
lukehesluke opened this issue Oct 7, 2021 · 0 comments

Comments

@lukehesluke
Copy link
Contributor

lukehesluke commented Oct 7, 2021

Example test run:

amending-order-quote_amend-c1-and-c2_OpenBookingSimpleFlow_ScheduledSession.md

Since Opportunities X are no longer in an active OrderQuote, the lease should be released.

Assumption: Leases should release capacity immediately to OrderItems that no longer form part of the OrderQuote. And they should take capacity immediately from new OrderItems added to the OrderQuote

Acceptance Criteria

When working on this, be sure to use a coverage/* branch (more info about coverage branches) and start a branch in Test Suite with the same name.

In this branch on Test Suite, search for all instances of https://github.com/openactive/OpenActive.Server.NET/issues/168 in the code. Here's an example of one:

https://github.com/openactive/openactive-test-suite/blob/e92f7ab1f89ac4a7b9a818e6bd0763eb58876d43/packages/openactive-integration-tests/test/features/core/amending-order-quote/common.js#L52-L53

Update it so that the comment and the doSkip line (which includes RUN_TESTS_WHICH_FAIL_REFIMPL) are removed for each instance. The effect of this will be turning on the post-OrderQuote-amendment capacity checks in Test Suite.

The acceptance criteria is that this branch of Test Suite passes against the OpenActive.Server.Net branch


This was revealed while working on this Test Suite issue: openactive/openactive-test-suite#460

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

No branches or pull requests

1 participant