From bb250d1865f142681647ba9716fccd645e17e167 Mon Sep 17 00:00:00 2001
From: Gleb Khmyznikov <gleb.khmyznikov@gmail.com>
Date: Fri, 11 Aug 2023 18:27:41 +0200
Subject: [PATCH] Add docsLink for groupedValidation results

---
 libraries/manifest-validation/package-lock.json         | 4 ++--
 libraries/manifest-validation/package.json              | 2 +-
 libraries/manifest-validation/src/index.ts              | 2 +-
 libraries/manifest-validation/src/interfaces.ts         | 1 +
 libraries/manifest-validation/test/validation-tests.mjs | 4 ++--
 5 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/libraries/manifest-validation/package-lock.json b/libraries/manifest-validation/package-lock.json
index 77ded91ea..11d01404a 100644
--- a/libraries/manifest-validation/package-lock.json
+++ b/libraries/manifest-validation/package-lock.json
@@ -1,12 +1,12 @@
 {
   "name": "@pwabuilder/manifest-validation",
-  "version": "0.0.7",
+  "version": "0.0.9",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "@pwabuilder/manifest-validation",
-      "version": "0.0.7",
+      "version": "0.0.9",
       "license": "ISC",
       "dependencies": {
         "ajv": "^8.12.0",
diff --git a/libraries/manifest-validation/package.json b/libraries/manifest-validation/package.json
index d9c6e89bd..02cae6f64 100644
--- a/libraries/manifest-validation/package.json
+++ b/libraries/manifest-validation/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@pwabuilder/manifest-validation",
-  "version": "0.0.8",
+  "version": "0.0.9",
   "author": "pwa-builder",
   "license": "ISC",
   "description": "Manifest validation library by PWABuilder criteria",
diff --git a/libraries/manifest-validation/src/index.ts b/libraries/manifest-validation/src/index.ts
index 4548ceae7..609101874 100644
--- a/libraries/manifest-validation/src/index.ts
+++ b/libraries/manifest-validation/src/index.ts
@@ -128,7 +128,7 @@ export async function groupedValidation(manifest: Manifest): Promise<TokensValid
         currentManifest = manifest;
         const testResults = await loopThroughKeys(manifest, false, true);
         const resultsGrouped = testResults.reduce((acc, curr) => {
-            const curTrimmed = { category: curr.category, member: curr.member, valid: curr.valid, displayString: curr.displayString, errorString: curr.errorString, infoString: curr.infoString };
+            const curTrimmed = { category: curr.category, member: curr.member, valid: curr.valid, displayString: curr.displayString, errorString: curr.errorString, infoString: curr.infoString, docsLink: curr.docsLink };
             if (acc[curr.member]) {
                 acc[curr.member].push(curTrimmed);
             } else {
diff --git a/libraries/manifest-validation/src/interfaces.ts b/libraries/manifest-validation/src/interfaces.ts
index e5386c477..0ea9dd7ca 100644
--- a/libraries/manifest-validation/src/interfaces.ts
+++ b/libraries/manifest-validation/src/interfaces.ts
@@ -18,6 +18,7 @@ export type PartialValidation = {
     displayString?: string;
     errorString?: string;
     infoString?: string;
+    docsLink?: string;
     valid: boolean;
 }
 
diff --git a/libraries/manifest-validation/test/validation-tests.mjs b/libraries/manifest-validation/test/validation-tests.mjs
index 0b61a5317..7aa251e7a 100644
--- a/libraries/manifest-validation/test/validation-tests.mjs
+++ b/libraries/manifest-validation/test/validation-tests.mjs
@@ -92,7 +92,7 @@ describe('Manifest Validation with hardcoded test manifest', async () => {
   });
 
   // should fail because of missing 96x96 icon
-  it('Can validate the inner structure of shortcuts, should fail', async () => {
+  it('Can validate the inner structure of shortcuts', async () => {
     const validity = await maniLib.validateSingleField("shortcuts", [
       {
         "name": "Start Live Session",
@@ -103,7 +103,7 @@ describe('Manifest Validation with hardcoded test manifest', async () => {
       }
     ]);
 
-    assert.equal(validity.valid, false);
+    assert.equal(validity.valid, true);
   });
 
   it('start_url is within app scope, should pass', async () => {