Skip to content

Commit d0d22ce

Browse files
authored
🐛 [Frontend] Fix: storage/async-jobs was retired (#7429)
1 parent a6d85cc commit d0d22ce

File tree

6 files changed

+38
-140
lines changed

6 files changed

+38
-140
lines changed

packages/pytest-simcore/src/pytest_simcore/helpers/playwright.py

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
from typing import Any, Final
1212

1313
from playwright._impl._sync_base import EventContextManager
14-
from playwright.sync_api import APIRequestContext, FrameLocator, Page, Request
14+
from playwright.sync_api import APIRequestContext, FrameLocator, Locator, Page, Request
1515
from playwright.sync_api import TimeoutError as PlaywrightTimeoutError
1616
from playwright.sync_api import WebSocket
1717
from pydantic import AnyUrl
@@ -508,3 +508,22 @@ def app_mode_trigger_next_app(page: Page) -> None:
508508
):
509509
# Move to next step (this auto starts the next service)
510510
page.get_by_test_id("AppMode_NextBtn").click()
511+
512+
513+
def wait_for_label_text(
514+
page: Page,
515+
locator: str,
516+
substring: str,
517+
timeout: int = 10000
518+
) -> Locator:
519+
page.locator(locator).wait_for(
520+
state="visible",
521+
timeout=timeout
522+
)
523+
524+
page.wait_for_function(
525+
f"() => document.querySelector('{locator}').innerText.includes('{substring}')",
526+
timeout=timeout
527+
)
528+
529+
return page.locator(locator)

services/static-webserver/client/source/class/osparc/data/PollTask.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@
2222
qx.Class.define("osparc.data.PollTask", {
2323
extend: qx.core.Object,
2424

25-
construct: function(taskData, interval) {
25+
construct: function(taskData, interval = 1000) {
2626
this.base(arguments);
2727

28-
interval ? this.setPollInterval(interval) : this.initPollInterval();
28+
this.setPollInterval(interval);
2929

3030
if (taskData && "task_id" in taskData) {
3131
this.set({

services/static-webserver/client/source/class/osparc/data/Resources.js

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1241,26 +1241,6 @@ qx.Class.define("osparc.data.Resources", {
12411241
}
12421242
}
12431243
},
1244-
/*
1245-
* STORAGE ASYNC
1246-
*/
1247-
"storageAsyncJobs": {
1248-
useCache: false,
1249-
endpoints: {
1250-
jobStatus: {
1251-
method: "GET",
1252-
url: statics.API + "/storage/async-jobs/{jobId}/status"
1253-
},
1254-
jobResult: {
1255-
method: "GET",
1256-
url: statics.API + "/storage/async-jobs/{jobId}/result"
1257-
},
1258-
abortJob: {
1259-
method: "POST",
1260-
url: statics.API + "/storage/async-jobs/{jobId}/abort"
1261-
},
1262-
}
1263-
},
12641244
/*
12651245
* ACTIVITY
12661246
*/

services/static-webserver/client/source/class/osparc/file/StorageAsyncJob.js

Lines changed: 0 additions & 98 deletions
This file was deleted.

services/static-webserver/client/source/class/osparc/file/TreeFolderView.js

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ qx.Class.define("osparc.file.TreeFolderView", {
6262
icon: "@FontAwesome5Solid/spinner/14",
6363
allowGrowX: false
6464
});
65+
osparc.utils.Utils.setIdToWidget(control.getChildControl("label"), "totalSizeLabel");
6566
this.getChildControl("header-layout").add(control);
6667
break;
6768
case "tree-folder-layout":
@@ -160,22 +161,18 @@ qx.Class.define("osparc.file.TreeFolderView", {
160161
const totalSize = this.getChildControl("total-size-label");
161162
totalSize.getChildControl("icon").getContentElement().addClass("rotate");
162163

163-
osparc.data.Resources.fetch("storagePaths", "requestSize", { url: { pathId } })
164-
.then(resp => {
165-
const jobId = resp["job_id"];
166-
if (jobId) {
167-
const asyncJob = new osparc.file.StorageAsyncJob(jobId);
168-
asyncJob.addListener("resultReceived", e => {
169-
const size = e.getData();
170-
totalSize.set({
171-
icon: null,
172-
label: this.tr("Total size: ") + osparc.utils.Utils.bytesToSize(size),
173-
});
164+
const pollTasks = osparc.store.PollTasks.getInstance();
165+
const fetchPromise = osparc.data.Resources.fetch("storagePaths", "requestSize", { url: { pathId } })
166+
pollTasks.createPollingTask(fetchPromise)
167+
.then(task => {
168+
task.addListener("resultReceived", e => {
169+
const size = e.getData();
170+
totalSize.set({
171+
icon: null,
172+
label: this.tr("Total size: ") + osparc.utils.Utils.bytesToSize(size),
174173
});
175-
asyncJob.addListener("pollingError", e => {
176-
totalSize.hide();
177-
});
178-
}
174+
});
175+
task.addListener("pollingError", e => totalSize.hide());
179176
})
180177
.catch(err => {
181178
console.error(err);

services/static-webserver/client/source/class/osparc/store/Services.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ qx.Class.define("osparc.store.Services", {
5151
const services = this.__servicesCached;
5252
if (key in services) {
5353
const latestMetadata = Object.values(services[key])[0];
54-
if (!osparc.service.Utils.isRetired(latestMetadata)) {
54+
if (!osparc.service.Utils.isDeprecated(latestMetadata)) {
5555
return latestMetadata;
5656
}
5757
}
@@ -168,8 +168,8 @@ qx.Class.define("osparc.store.Services", {
168168
continue;
169169
}
170170
if (excludeDeprecated) {
171-
if (osparc.service.Utils.isRetired(serviceLatest)) {
172-
// first check if a previous version of this service isn't retired
171+
if (osparc.service.Utils.isDeprecated(serviceLatest)) {
172+
// first check if a previous version of this service isn't deprecated
173173
// getService to get its history
174174
await this.getService(serviceLatest["key"], serviceLatest["version"]);
175175
const serviceMetadata = this.__servicesCached[key][serviceLatest["version"]];
@@ -185,7 +185,7 @@ qx.Class.define("osparc.store.Services", {
185185
}
186186
}
187187
}
188-
if (osparc.service.Utils.isRetired(serviceLatest)) {
188+
if (osparc.service.Utils.isDeprecated(serviceLatest)) {
189189
// do not add retired services
190190
continue;
191191
}

0 commit comments

Comments
 (0)