fix for package annotations in sync command

adobe · Jun 12, 2020 · e8a90bf · e8a90bf
1 parent a6ce900
commit e8a90bf
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 14 deletions.
diff --git a/src/runtime-helpers.js b/src/runtime-helpers.js
@@ -998,20 +998,12 @@ async function getProjectEntities (project, isProjectHash, ow) {
 async function addManagedProjectAnnotations (entities, manifestPath, projectName, projectHash) {
   // add whisk managed annotations
   for (const pkg of entities.pkgAndDeps) {
-    const options = {}
-    options['name'] = pkg.name
-    options['package'] = {
-      annotations: [
-        {
-          key: 'whisk-managed',
-          value: {
-            file: manifestPath,
-            projectDeps: [],
-            projectHash: projectHash,
-            projectName: projectName
-          }
-        }
-      ]
+    pkg.annotations = pkg.annotations || {}
+    pkg.annotations['whisk-managed'] = {
+      file: manifestPath,
+      projectDeps: [],
+      projectHash: projectHash,
+      projectName: projectName
     }
   }
   for (const action of entities.actions) {

diff --git a/test/commands/runtime/deploy/sync.test.js b/test/commands/runtime/deploy/sync.test.js
@@ -352,6 +352,18 @@ describe('instance methods', () => {
       return command.run()
         .then(() => {
           expect(cmd).toHaveBeenCalled()
+          expect(cmd).toHaveBeenCalledWith(
+            expect.objectContaining({
+              annotations: expect.objectContaining({
+                'whisk-managed': {
+                  file: '/deploy/deployment_actionMissingInputs.yaml',
+                  projectDeps: [],
+                  projectHash: 'xyz',
+                  projectName: 'proj'
+                }
+              })
+            })
+          )
           expect(stdout.output).toMatch('')
         })
     })