[jochen-testingbot]

Part of https://testingbot-team.atlassian.net/browse/TB-1008 (docs audit: run every SDK README snippet).

README walkthrough (Packagist 2.0.0, composer:2 / PHP 8.5)

28 of 30 calls work against the live API: browsers (incl. BrowserType::Webdriver), tests list/get/update/delete, NotFoundException on a bad id, builds, storage uploadRemoteUrl/get/delete/list, tunnels, user, team, devices, configuration, screenshots, getAuthenticationHash, and the 1.x TestingBotAPI facade.

• tests()->list(0, 10, ['build' => 'ci-42']) 404s only because that build doesn't exist in the account. That is expected.
• tests()->create(['name' => 'external result', 'success' => true]) fails:

  TestingBot\Exception\ApiException: test[test_environment_id] is missing
  

  With 'test_environment_id' => $browsers[0]['browser_id'] it succeeds (created, updated and deleted in the run).

Change

README only: the create example now passes test_environment_id, with a comment saying where the id comes from. No code change and no release needed. Packagist renders the README from the default branch.

[Copilot]

🟡 Changes recommended

The README example uses an undefined $browserId, resulting in an invalid request without a valid test_environment_id until the snippet is corrected.

1 open issue

• README example uses an undefined browser ID · New

What changed in this PR

Updates the README’s external test creation example to include the required test_environment_id parameter.

Changes:

• Documents that the environment ID comes from browsers()->list().
• Includes test_environment_id in the tests()->create() payload.
• Makes no source-code or release changes.

File	Summary	Review
README.md	Documents the required test environment ID for creating external results.	Nit: $browserId is referenced without being defined; derive it from the listed browser environments before calling create.

_{Review effort: Lite}

[Copilot]

🟢 Approved

The README example now supplies the required browser environment ID when creating a test, addressing the documented API requirement without changing library code.

No open issues 🎉

1 resolved since last review

• README example uses an undefined browser ID

_{Review effort: Lite}