Testfixes for new meta structure on tasks and groups

hashicorp · Dec 5, 2024 · 54c2ad0 · 54c2ad0
1 parent 2f5069a
commit 54c2ad0
Show file tree

Hide file tree

Showing 10 changed files with 60 additions and 19 deletions.
diff --git a/ui/app/models/task-group.js b/ui/app/models/task-group.js
@@ -68,7 +68,7 @@ export default class TaskGroup extends Fragment {
   get mergedMeta() {
     return {
       ...this.job.get('meta.raw'),
-      ...this.meta,
+      ...this.get('meta.raw'),
     };
   }
 

diff --git a/ui/app/models/task.js b/ui/app/models/task.js
@@ -29,7 +29,7 @@ export default class Task extends Fragment {
   get mergedMeta() {
     return {
       ...this.taskGroup.mergedMeta,
-      ...this.meta,
+      ...this.get('meta.raw'),
     };
   }
 

diff --git a/ui/mirage/factories/task-group.js b/ui/mirage/factories/task-group.js
@@ -57,6 +57,9 @@ export default Factory.extend({
   // When true, the group will simulate a "scheduled" block's paused state
   withPausedTasks: false,
 
+  // When true, the tasks will have metadata
+  withTaskMeta: false,
+
   afterCreate(group, server) {
     let taskIds = [];
     let volumes = Object.keys(group.volumes);
@@ -114,6 +117,7 @@ export default Factory.extend({
           })),
           createRecommendations: group.createRecommendations,
           withSchedule: group.withPausedTasks,
+          withMeta: group.withTaskMeta,
         });
       });
       taskIds = tasks.mapBy('id');

diff --git a/ui/mirage/factories/task.js b/ui/mirage/factories/task.js
@@ -26,6 +26,8 @@ export default Factory.extend({
   // Set in the TaskGroup factory
   volumeMounts: [],
 
+  meta: null,
+
   JobID: '',
 
   name: (id) => `task-${dasherize(faker.hacker.noun())}-${id}`,
@@ -128,5 +130,9 @@ export default Factory.extend({
       });
       task.update({ schedule: schedule });
     }
+
+    if (task.withMeta) {
+      task.update({ meta: { raw: { foo: 'bar' } } });
+    }
   },
 });
diff --git a/ui/tests/acceptance/task-detail-test.js b/ui/tests/acceptance/task-detail-test.js
@@ -12,7 +12,7 @@ import a11yAudit from 'nomad-ui/tests/helpers/a11y-audit';
 import Task from 'nomad-ui/tests/pages/allocations/task/detail';
 import Layout from 'nomad-ui/tests/pages/layout';
 import moment from 'moment';
-
+import { set } from '@ember/object';
 let allocation;
 let task;
 
@@ -213,6 +213,28 @@ module('Acceptance | task detail', function (hooks) {
     });
   });
 
+  test('when a task group has metadata, the metadata table is shown', async function (assert) {
+    const job = server.create('job', {
+      createAllocations: false,
+    });
+    const taskGroup = server.create('task-group', {
+      job,
+      name: 'scaling',
+      count: 1,
+      withTaskMeta: true,
+    });
+    job.update({ taskGroupIds: [taskGroup.id] });
+    allocation = server.db.allocations[1];
+    server.db.taskStates.update(
+      { allocationId: allocation.id },
+      { state: 'running' }
+    );
+    const jobTask = taskGroup.tasks.models[0];
+    task = jobTask;
+    await Task.visit({ id: allocation.id, name: task.name });
+    assert.ok(Task.hasMeta);
+  });
+
   test('each recent event should list the time, type, and description of the event', async function (assert) {
     const event = server.db.taskEvents.where({ taskStateId: task.id })[0];
     const recentEvent = Task.events.objectAt(Task.events.length - 1);

diff --git a/ui/tests/acceptance/task-group-detail-test.js b/ui/tests/acceptance/task-group-detail-test.js
@@ -454,6 +454,13 @@ module('Acceptance | task group detail', function (hooks) {
   });
 
   test('when the task group has metadata, the metadata table is shown', async function (assert) {
+    job = server.create('job', {
+      meta: { raw: { a: 'b' } },
+    });
+    taskGroup = server.create('task-group', {
+      job,
+      meta: { raw: { foo: 'bar' } },
+    });
     await TaskGroup.visit({ id: job.id, name: taskGroup.name });
 
     assert.ok(TaskGroup.hasMeta);

diff --git a/ui/tests/pages/allocations/task/detail.js b/ui/tests/pages/allocations/task/detail.js
@@ -60,6 +60,8 @@ export default create({
     clientSource: text('[data-test-volume-client-source]'),
   }),
 
+  hasMeta: isPresent('[data-test-meta]'),
+
   events: collection('[data-test-task-event]', {
     time: text('[data-test-task-event-time]'),
     type: text('[data-test-task-event-type]'),

diff --git a/ui/tests/pages/jobs/job/task-group.js b/ui/tests/pages/jobs/job/task-group.js
@@ -54,6 +54,8 @@ export default create({
     permissions: text('[data-test-volume-permissions]'),
   }),
 
+  hasMeta: isPresent('[data-test-meta]'),
+
   hasScaleEvents: isPresent('[data-test-scale-events]'),
   scaleEvents: collection(
     '[data-test-scale-events] [data-test-accordion-head]',

diff --git a/ui/tests/unit/models/task-group-test.js b/ui/tests/unit/models/task-group-test.js
@@ -76,24 +76,22 @@ module('Unit | Model | task-group', function (hooks) {
     const jobWithMeta = run(() =>
       store.createRecord('job', {
         name: 'example-with-meta',
-        meta: store.createFragment('structured-attributes', {
-          raw: { a: 'b' },
-        }),
+        meta: { raw: { a: 'b' } },
         taskGroups: [
           {
             name: 'one',
-            meta: { c: 'd' },
+            meta: { raw: { c: 'd' } },
           },
           {
             name: 'two',
           },
           {
             name: 'three',
-            meta: null,
+            meta: { raw: null },
           },
           {
             name: 'four',
-            meta: {},
+            meta: { raw: {} },
           },
         ],
       })
@@ -114,18 +112,18 @@ module('Unit | Model | task-group', function (hooks) {
         taskGroups: [
           {
             name: 'one',
-            meta: { c: 'd' },
+            meta: { raw: { c: 'd' } },
           },
           {
             name: 'two',
           },
           {
             name: 'three',
-            meta: null,
+            meta: { raw: null },
           },
           {
             name: 'four',
-            meta: {},
+            meta: { raw: {} },
           },
         ],
       })

diff --git a/ui/tests/unit/models/task-test.js b/ui/tests/unit/models/task-test.js
@@ -20,22 +20,22 @@ module('Unit | Model | task', function (hooks) {
         taskGroups: [
           {
             name: 'one',
-            meta: { a: 'b' },
+            meta: { raw: { a: 'b' } },
             tasks: [
               {
                 name: 'task-one',
-                meta: { c: 'd' },
+                meta: { raw: { c: 'd' } },
               },
               {
                 name: 'task-two',
               },
               {
                 name: 'task-three',
-                meta: null,
+                meta: { raw: null },
               },
               {
                 name: 'task-four',
-                meta: {},
+                meta: { raw: {} },
               },
             ],
           },
@@ -44,18 +44,18 @@ module('Unit | Model | task', function (hooks) {
             tasks: [
               {
                 name: 'task-one',
-                meta: { c: 'd' },
+                meta: { raw: { c: 'd' } },
               },
               {
                 name: 'task-two',
               },
               {
                 name: 'task-three',
-                meta: null,
+                meta: { raw: null },
               },
               {
                 name: 'task-four',
-                meta: {},
+                meta: { raw: {} },
               },
             ],
           },