confluentinc / confluentinc/vscode

Test flake: `authn/ccloudProvider.ts ConfluentCloudAuthProvider handleSessionRemoved()`

Open
#2,970 0 comments 0 reactions 0 assignees View on GitHub
testing
Dominant language
TypeScript
Stars
34
Forks
17
Avg merge
1d 22h
Merged PRs (30d)
8

Description

Seems like we have some test contamination (seen in https://semaphore.ci.confluent.io/workflows/afa800bb-d92f-4ffa-9f33-e9c2f63bb55d?pipeline_id=1cb1e990-a9a9-4bdb-90be-44b2fb28cc38):
```
AssertError: expected clearCurrentCCloudResources to be called once but was called twice
clearCurrentCCloudResources() => {} at _ConfluentCloudAuthProvider.handleSessionRemoved (out/src/authn/ccloudProvider.js:97:290)
clearCurrentCCloudResources() => {} at _ConfluentCloudAuthProvider.handleSessionRemoved (out/src/authn/ccloudProvider.js:97:290)
at Object.fail (out/node_modules/sinon/pkg/sinon-esm.js:131:27)
at failAssertion (out/node_modules/sinon/pkg/sinon-esm.js:272:20)
at assert. [as calledOnce] (out/node_modules/sinon/pkg/sinon-esm.js:301:17)
at Context. (out/src/authn/ccloudProvider.test.js:319:25)
```

```
AssertError: expected clearCurrentCCloudResources to be called once but was called thrice
clearCurrentCCloudResources() => {} at _ConfluentCloudAuthProvider.handleSessionRemoved (out/src/authn/ccloudProvider.js:97:290)
clearCurrentCCloudResources() => {} at _ConfluentCloudAuthProvider.handleSessionRemoved (out/src/authn/ccloudProvider.js:97:290)
clearCurrentCCloudResources() => {} at _ConfluentCloudAuthProvider.handleSessionRemoved (out/src/authn/ccloudProvider.js:97:290)
at Object.fail (out/node_modules/sinon/pkg/sinon-esm.js:131:27)
at failAssertion (out/node_modules/sinon/pkg/sinon-esm.js:272:20)
at assert. [as calledOnce] (out/node_modules/sinon/pkg/sinon-esm.js:301:17)
at Context. (out/src/authn/ccloudProvider.test.js:319:25)
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.