dapr / js-sdk

Dapr SDK for Javascript
Apache License 2.0
194 stars 82 forks source link

Fix e2e test http actortest #554

Closed MregXN closed 9 months ago

MregXN commented 9 months ago

Description

it fails due to the following log

== APP ==   ● http/actors › configuration › actor configuration endpoint should contain the correct parameters
== APP == 
== APP ==     expect(received).toBe(expected) // Object.is equality
== APP == 
== APP ==     Expected: 9
== APP ==     Received: 11
== APP == 
== APP ==       114 |       const config = JSON.parse(await res.text());
== APP ==       115 |
== APP ==     > 116 |       expect(config.entities.length).toBe(9);
== APP ==           |                                      ^
== APP ==       117 |       expect(config.actorIdleTimeout).toBe("1h");
== APP ==       118 |       expect(config.actorScanInterval).toBe("30s");
== APP ==       119 |       expect(config.drainOngoingCallTimeout).toBe("1m");
== APP == 
== APP ==       at test/e2e/http/actors.test.ts:116:38
== APP ==       at fulfilled (test/e2e/http/actors.test.ts:40:58)

This is because two actors were newly registered in pr #536. For more details could refer to this link .

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d33c5a4) 100.00% compared to head (4b78d5a) 100.00%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #554 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 1 1 Lines 6 6 Branches 1 1 ========================================= Hits 6 6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

MregXN commented 9 months ago

refer to issue #7128 in dapr/dapr