[vdusek]

test_cb_kwargs_spider failed on an E2E run with Missing product: Widget A. The run succeeded and the API reported 3 items in the dataset, but the items page it returned was empty. The test read the dataset right after the run finished, before the pushed items were readable.

verify_spider_results now polls list_items() with poll_until_condition until the expected items show up, for up to 30 s. The assertions are unchanged, and every scrapy spider test that uses the helper gets the fix.

✍️ Drafted by Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.81%. Comparing base (aab5cf8) to head (0087914).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1225      +/-   ##
==========================================
+ Coverage   93.46%   93.81%   +0.34%     
==========================================
  Files          54       54              
  Lines        3797     3799       +2     
==========================================
+ Hits         3549     3564      +15     
+ Misses        248      235      -13     

Flag	Coverage Δ
e2e	31.48% <ø> (-0.02%)	⬇️
integration	51.38% <ø> (-0.03%)	⬇️
unit	90.99% <ø> (+0.66%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[apify-factory]

Looks good, nothing blocking.

🏭 Apify Factory auto approved ✅ — the strict one and the suspicious one both nodded

Every gate passed, and neither the reviewer nor a juror found anything a human needs to look at.

_{Approval locked to 0087914f4 · workflow run}