@andrian.yablonskyy/thub-coordinator 1.1.10 → 1.1.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@andrian.yablonskyy/thub-coordinator",
-  "version": "1.1.10",
+  "version": "1.1.11",
   "description": "TestHub Coordinator — job queue, resource registry, scheduler, heartbeat monitor, log/artifact store and web dashboard",
   "main": "src/server.js",
   "engines": {

package/public/css/thub.css

@@ -56,7 +56,7 @@
 }
 
 .resource-capabilities {
-  font-size: 0.75rem;
+  font-size: 0.875rem;
 }
 
 /* An overlay inside a modal (e.g. "Import to config"'s review note on the

package/test/groups-tab.test.js

@@ -340,3 +340,21 @@ test('Labels tab: in effect at once, sent to the Client as its next config revis
   const old = register(services, 'lab-old-30', { configFile: null });
   assert.throws(() => services.registry.setClientLabels(old, ['x'], { by: 'x' }), /too old/);
 });
+
+test('runner card: capabilities have their own tab, next to Details', async (t) => {
+  const { base, services, admin } = await start(t),
+    { resourceId } = services.registry.registerAuto({
+      clientId: 'hw8', name: 'lab-hw-08', type: 'hw', labels: [],
+      capabilities: { hw: { stlinks: [{ index: 1, path: '/dev/thub/dut1-stlink', present: false }], uarts: [], usbs: [] } }
+    }),
+    html = await (await fetch(`${base}/runners`, { headers: { cookie: admin } })).text(),
+    card = html.slice(html.indexOf(`id="resource-card-${resourceId}"`)),
+    details = card.slice(card.indexOf(`id="rc-${resourceId}-details"`), card.indexOf(`id="rc-${resourceId}-caps"`)),
+    caps = card.slice(card.indexOf(`id="rc-${resourceId}-caps"`)),
+    // The tab right after Details.
+    tabs = [...card.slice(0, card.indexOf('</ul>')).matchAll(/<button class="nav-link[^"]*" id="rc-[^"]*-(\w+)-tab"/g)].map((m) => m[1]);
+  assert.deepEqual(tabs.slice(0, 2), ['details', 'caps']);
+  assert.doesNotMatch(details, />Capabilities</); // no longer a Details row
+  assert.match(caps, new RegExp(`^id="rc-${resourceId}-caps"[^>]*data-live="rc-caps-${resourceId}"`)); // updates live
+  assert.match(caps, /\/dev\/thub\/dut1-stlink<\/code><span class="badge text-bg-danger[^"]*"[^>]*>missing/);
+});

package/views/help/_client-machines.pug

@@ -47,7 +47,7 @@
       +code('HW machine').
         sudo systemctl restart thub-client@client
         ls -l /dev/thub/
-    li Check the runner card's #[strong Capabilities]: each configured device must be present (a missing one is flagged).
+    li Check the runner card's #[strong Capabilities] tab: each configured device must be present (a missing one is flagged).
     li Your job's #[code --command] flashes the board. The Client never flashes by itself; it passes the ST-Link serials and the downloaded files in the environment (#[a(href="#env") Environment variables]).
   +code('A typical HW job').
     thub run --type hw --label board:nucleo-f401re \

package/views/help/_docker.pug

@@ -121,4 +121,4 @@
         td Files written to #[code /work] stay owned by the Client user, so the workspace can be cleaned up.
   +note('info').
     Docker access is effectively root access on that host. The Client host must have Docker and the service user must be in the
-    #[code docker] group. The runner card's #[strong Capabilities] show whether a Client can use Docker, and why not.
+    #[code docker] group. The runner card's #[strong Capabilities] tab shows whether a Client can use Docker, and why not.

package/views/mixins/resource.pug

@@ -298,8 +298,9 @@ mixin resourceCard(r, groupsById, returnTo)
         //- data-config-root: where public/js/client-config.js tracks unsaved
         //- changes (⚠ on the config tabs).
         .modal-body.pt-2(data-config-root)
-          //- Details: what the Client is and reports. Then its config tabs
-          //- (client-config.pug; read-only for viewers), then its USB devices.
+          //- Details: what the Client is and reports; Capabilities: what it can
+          //- drive. Then its config tabs (client-config.pug; read-only for
+          //- viewers), then its USB devices.
           ul.nav.nav-tabs.mb-3(role="tablist")
             li.nav-item(role="presentation")
               button.nav-link.active(
@@ -311,6 +312,16 @@ mixin resourceCard(r, groupsById, returnTo)
                 aria-controls=`rc-${r.id}-details`
                 aria-selected="true"
               ) Details
+            li.nav-item(role="presentation")
+              button.nav-link(
+                id=`rc-${r.id}-caps-tab`
+                type="button"
+                role="tab"
+                data-bs-toggle="tab"
+                data-bs-target=`#rc-${r.id}-caps`
+                aria-controls=`rc-${r.id}-caps`
+                aria-selected="false"
+              ) Capabilities
             +configTabItems(r)
             li.nav-item(role="presentation")
               button.nav-link(
@@ -326,6 +337,11 @@ mixin resourceCard(r, groupsById, returnTo)
           .tab-content
             .tab-pane.fade.show.active(id=`rc-${r.id}-details` role="tabpanel" aria-labelledby=`rc-${r.id}-details-tab` tabindex="0" data-live=`rc-details-${r.id}`)
               +resourceDetails(r, groupsById, returnTo, local, external)
+            //- Live too: a Client that restarts reports them again.
+            .tab-pane.fade(id=`rc-${r.id}-caps` role="tabpanel" aria-labelledby=`rc-${r.id}-caps-tab` tabindex="0" data-live=`rc-caps-${r.id}`)
+              p.small.text-body-secondary.mb-2 What the Client reported it can drive at its last start, checked against its host.
+              .resource-capabilities
+                +capabilitiesList(r.capabilities)
             +configPanes(r, returnTo, groupsById)
             .tab-pane.fade.thub-usb-pane(id=`rc-${r.id}-usb` role="tabpanel" aria-labelledby=`rc-${r.id}-usb-tab` tabindex="0")
               +usbScanPane(r)
@@ -496,9 +512,6 @@ mixin resourceDetails(r, groupsById, returnTo, local, external)
               data-bs-toggle="tooltip"
               data-bs-title="The address the Coordinator last saw this Client connect from"
             ) external
-    dt.col-sm-4 Capabilities
-    dd.col-sm-8.resource-capabilities
-      +capabilitiesList(r.capabilities)
     dt.col-sm-4 Labels
     dd.col-sm-8
       if r.labels.length