Skip to content
This repository was archived by the owner on May 18, 2026. It is now read-only.

Commit 0490ae2

Browse files
takaokoujiGemini
andcommitted
fix: ensure all extensions are visible during integration tests
- Modified SeleniumHelper.loadUri to automatically append showAllExtensions=true to the test URI. - This ensures that hidden extensions like EV3 are accessible to tests that interact with the extension library UI. - Fixes CI failure in test/integration/connection-modal.test.js. Co-Authored-By: Gemini <noreply@google.com>
1 parent 668c067 commit 0490ae2

1 file changed

Lines changed: 11 additions & 1 deletion

File tree

test/helpers/selenium-helper.js

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -249,8 +249,8 @@ class SeleniumHelper {
249249
* @returns {Promise} A promise that resolves when the URI is loaded.
250250
*/
251251
async loadUri (uri) {
252+
const locale = 'locale=en';
252253
if (uri.indexOf('locale=') < 0) {
253-
const locale = 'locale=en';
254254
if (uri.indexOf('?') >= 0) {
255255
uri = uri.replace('?', `?${locale}&`);
256256
} else if (uri.indexOf('#') >= 0) {
@@ -259,6 +259,16 @@ class SeleniumHelper {
259259
uri = `${uri}?${locale}`;
260260
}
261261
}
262+
if (uri.indexOf('showAllExtensions=') < 0) {
263+
const showAllExtensions = 'showAllExtensions=true';
264+
if (uri.indexOf('?') >= 0) {
265+
uri = uri.replace('?', `?${showAllExtensions}&`);
266+
} else if (uri.indexOf('#') >= 0) {
267+
uri = uri.replace('#', `?${showAllExtensions}#`);
268+
} else {
269+
uri = `${uri}?${showAllExtensions}`;
270+
}
271+
}
262272

263273
const outerError = new Error(`loadUri failed with arguments:\n\turi: ${uri}`);
264274
try {

0 commit comments

Comments
 (0)