Skip to content

Add browser test - #220

Merged
Zhiyuan Liang (zhiyuanliang-ms) merged 2 commits into
mainfrom
zhiyuanliang/browser-vitest
Sep 8, 2025
Merged

Add browser test#220
Zhiyuan Liang (zhiyuanliang-ms) merged 2 commits into
mainfrom
zhiyuanliang/browser-vitest

Conversation

@zhiyuanliang-ms

Copy link
Copy Markdown
Member

No description provided.

@juniwang

Copy link
Copy Markdown
Member

The are errors in the build logs https://github.com/Azure/AppConfiguration-JavaScriptProvider/actions/runs/17439118545/job/49517111057?pr=220. Are they expected?

Failed to load. Error message: Test Error. Retrying in 5000 ms.
Refresh failed: Error: All fallback clients failed to get configuration settings.
    at #executeWithFailoverPolicy (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:701:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
    at async #getConfigurationSetting (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:633:24)
    at async #refreshKeyValues (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:544:30)
    at async Promise.allSettled (index 0)
    at async #refreshTasks (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:370:25)
    at async AzureAppConfigurationImpl.refresh (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:274:13)
    at async Context.<anonymous> (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/test/requestTracing.test.js:177:13)
Failed to load. Error message: All fallback clients failed to get configuration settings.. Retrying in 5000 ms.
Failed to load. Error message: All fallback clients failed to get configuration settings.. Retrying in 5000 ms.

@zhiyuanliang-ms

Copy link
Copy Markdown
Member Author

The are errors in the build logs https://github.com/Azure/AppConfiguration-JavaScriptProvider/actions/runs/17439118545/job/49517111057?pr=220. Are they expected?

Failed to load. Error message: Test Error. Retrying in 5000 ms.
Refresh failed: Error: All fallback clients failed to get configuration settings.
    at #executeWithFailoverPolicy (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:701:15)
    at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
    at async #getConfigurationSetting (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:633:24)
    at async #refreshKeyValues (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:544:30)
    at async Promise.allSettled (index 0)
    at async #refreshTasks (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:370:25)
    at async AzureAppConfigurationImpl.refresh (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/src/appConfigurationImpl.js:274:13)
    at async Context.<anonymous> (file:///home/runner/work/AppConfiguration-JavaScriptProvider/AppConfiguration-JavaScriptProvider/out/esm/test/requestTracing.test.js:177:13)
Failed to load. Error message: All fallback clients failed to get configuration settings.. Retrying in 5000 ms.
Failed to load. Error message: All fallback clients failed to get configuration settings.. Retrying in 5000 ms.

Junbo Wang (@juniwang) Yes.

This testcase The initial load and refresh will both fail, but we only need to check whether request contains the header we want, this is achieved through a pipeline policy.

@zhiyuanliang-ms
Zhiyuan Liang (zhiyuanliang-ms) merged commit a5756f9 into main Sep 8, 2025
6 checks passed
@zhiyuanliang-ms
Zhiyuan Liang (zhiyuanliang-ms) deleted the zhiyuanliang/browser-vitest branch September 8, 2025 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants