Fix flaky OrchestrationTutorialTest: Update orchestration tutorial to use fruitshop

[christiangoerdes]

Summary by CodeRabbit

• Documentation

  • Rewrote orchestration tutorial from a library/books example to a product catalog workflow that creates a copied product via /products/latest/copy; the flow now builds a new product with name suffixed "Copy" and a fixed price of 5, returning HTTP 201.

• Tests

  • Updated tests to remove the old books checks and verify the new copy endpoint returns 201, non-null id/name, exact price 5, and that name contains "Copy".

[christiangoerdes]

/ok-to-test

[membrane-ci-server]

This pull request needs "/ok-to-test" from an authorized committer.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8d2b9169-89b8-4ee6-bd6f-dd476ac5d930

📥 Commits

Reviewing files that changed from the base of the PR and between a368894 and fb6e1bb.

📒 Files selected for processing (2)

• distribution/src/test/java/com/predic8/membrane/tutorials/orchestration/OrchestrationTutorialTest.java
• distribution/tutorials/orchestration/30-Orchestration.yaml

✅ Files skipped from review due to trivial changes (1)

• distribution/tutorials/orchestration/30-Orchestration.yaml

---

📝 Walkthrough

Walkthrough

The PR rewrites the orchestration tutorial from a books-by-ID example to a product copy workflow: the YAML now GETs the Fruitshop listing for the newest product, builds a copy payload (name + "Copy", price 5), POSTs to create it, and returns 201. The Java test was updated to POST /products/latest/copy and assert 201 with created product fields.

Changes

Orchestration Tutorial Example Modernization

Layer / File(s)	Summary
Tutorial Specification distribution/tutorials/orchestration/30-Orchestration.yaml	Endpoint changed to /products/latest/copy. Orchestration flow replaced: single Fruitshop GET for newest product, JSONPath-based payload template (copy name + "Copy", fixed price: 5), POST to create product, response status set to 201. Curl example and tutorial text updated.
Test Validation distribution/src/test/java/com/predic8/membrane/tutorials/orchestration/OrchestrationTutorialTest.java	Hamcrest imports expanded (containsString, greaterThan, notNullValue). Removed books_areReturnedById test. Added latestProductCopy_isCreatedWithFixedPrice test that POSTs /products/latest/copy and asserts HTTP 201, non-null id/name, price == 5, and name contains "Copy".

🎯 3 (Moderate) | ⏱️ ~20 minutes

Suggested reviewers

• predic8

Poem

🐰 A tiny rabbit hops in cheer,
Found the latest fruit so dear.
Copied the name, set price to five,
A brand-new product comes alive.
Hop, deploy, the test says "OK"—hurray!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Fix flaky OrchestrationTutorialTest: Update orchestration tutorial to use fruitshop' accurately reflects the main objectives: fixing the flaky test and updating the tutorial to use fruitshop instead of OpenLibrary.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch unflaky-tutorial-test

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[predic8]

I'd like to keep the book example as it is something different than the fruitstore.

[predic8]

I'd like to keep the book example

[predic8]

How flaky is it?

[christiangoerdes]

It fails sometimes due to a wrong status code