-
Notifications
You must be signed in to change notification settings - Fork 58
Closed
Labels
t-toolingIssues with this label are in the ownership of the tooling team.Issues with this label are in the ownership of the tooling team.
Description
apify-sdk-js/test/e2e/sdk/actorCharge/test.mjs
Lines 61 to 66 in 6e48b85
| test('default charge limit 0', async () => { | |
| const run = await runActor({}, {}); | |
| assert.strictEqual(run.status, 'SUCCEEDED'); | |
| assert.deepEqual(run.chargedEventCounts, { foobar: 0 }); | |
| }); |
This test is failing.
+{ foobar: 4 }
-{ foobar: 0 } Looking at the run, it seems that passing no limit to the client now means "Unlimited" spending for the PPE Actor.
I'm not sure if this is a regression, a designed change, or even anything we can solve in SDK (might be an API update we didn't change our E2Es for yet).
cc @janbuchar
Metadata
Metadata
Assignees
Labels
t-toolingIssues with this label are in the ownership of the tooling team.Issues with this label are in the ownership of the tooling team.