From 350134e1877fe0de745862a5142e445c2a6bfbb2 Mon Sep 17 00:00:00 2001
From: John Wu <76966357+jwu-fisker@users.noreply.github.com>
Date: Wed, 6 Apr 2022 12:34:39 -0700
Subject: [PATCH] CEC-532 Display manifest file properties (#133)

* CEC-532 Display update file properties

* npm audit fix
---
 package-lock.json                             | 30 ++++++++++---------
 package.json                                  |  2 +-
 .../Details/__snapshots__/index.test.jsx.snap | 12 +++-----
 src/components/Manifest/Details/index.jsx     |  9 ++++--
 4 files changed, 28 insertions(+), 25 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index 846badd..001349c 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -5328,6 +5328,16 @@
         "css-what": "^3.2.1",
         "domutils": "^1.7.0",
         "nth-check": "^1.0.2"
+      },
+      "dependencies": {
+        "nth-check": {
+          "version": "1.0.2",
+          "resolved": "https://registry.npmjs.org/nth-check/-/nth-check-1.0.2.tgz",
+          "integrity": "sha512-WeBOdju8SnzPN5vTUJYxYUxLeXpCaVP5i5e0LF8fg7WORF2Wd7wFX/pk0tYZk7s8T+J7VLy0Da6J1+wCT0AtHg==",
+          "requires": {
+            "boolbase": "~1.0.0"
+          }
+        }
       }
     },
     "css-select-base-adapter": {
@@ -8939,9 +8949,9 @@
       }
     },
     "minimist": {
-      "version": "1.2.5",
-      "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.5.tgz",
-      "integrity": "sha512-FM9nNUYrRBAELZQT3xeZQ7fmMOBg6nWNmJKTcgsJeaLstP/UODVpGsr5OhXhhXg6f+qtJ8uiZ+PUxkDWcgIXLw=="
+      "version": "1.2.6",
+      "resolved": "https://registry.npmjs.org/minimist/-/minimist-1.2.6.tgz",
+      "integrity": "sha512-Jsjnk4bw3YJqYzbdyBiNsPWHPfO++UGG749Cxs6peCu5Xg4nrena6OVxOYxrQTqww0Jmwt+Ref8rggumkTLz9Q=="
     },
     "mkdirp": {
       "version": "0.5.5",
@@ -9000,9 +9010,9 @@
       }
     },
     "node-forge": {
-      "version": "1.2.1",
-      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.2.1.tgz",
-      "integrity": "sha512-Fcvtbb+zBcZXbTTVwqGA5W+MKBj56UjVRevvchv5XrcyXbmNdesfZL37nlcWOfpgHhgmxApw3tQbTr4CqNmX4w=="
+      "version": "1.3.1",
+      "resolved": "https://registry.npmjs.org/node-forge/-/node-forge-1.3.1.tgz",
+      "integrity": "sha512-dPEtOeMvF9VMcYV/1Wb8CPoVAXtp6MKMlcbAt4ddqmGqUJ6fQZFXkNZNkNlfevtNkGtaSoXf/vNNNSvgrdXwtA=="
     },
     "node-int64": {
       "version": "0.4.0",
@@ -9037,14 +9047,6 @@
         "path-key": "^3.0.0"
       }
     },
-    "nth-check": {
-      "version": "2.0.1",
-      "resolved": "https://registry.npmjs.org/nth-check/-/nth-check-2.0.1.tgz",
-      "integrity": "sha512-it1vE95zF6dTT9lBsYbxvqh0Soy4SPowchj0UBGj/V6cTPnXXtQOPUbhZ6CmGzAD/rW22LQK6E96pcdJXk4A4w==",
-      "requires": {
-        "boolbase": "^1.0.0"
-      }
-    },
     "nwsapi": {
       "version": "2.2.0",
       "resolved": "https://registry.npmjs.org/nwsapi/-/nwsapi-2.2.0.tgz",
diff --git a/package.json b/package.json
index 30e73af..7739340 100644
--- a/package.json
+++ b/package.json
@@ -71,4 +71,4 @@
       "lcov"
     ]
   }
-}
\ No newline at end of file
+}
diff --git a/src/components/Manifest/Details/__snapshots__/index.test.jsx.snap b/src/components/Manifest/Details/__snapshots__/index.test.jsx.snap
index 98022d4..8184647 100644
--- a/src/components/Manifest/Details/__snapshots__/index.test.jsx.snap
+++ b/src/components/Manifest/Details/__snapshots__/index.test.jsx.snap
@@ -116,7 +116,7 @@ Object {
               ECUs: 
               <ul>
                 <li>
-                  ICC, SW Version: 1000, HW Version: None, Mode: D
+                  ICC, SW Version: 1000, HW Version: none, Mode: D
                   <br />
                   <ul>
                     <li>
@@ -125,9 +125,7 @@ Object {
                       >
                         LARGE.jpg
                       </a>
-                       [
-                      b0cda514c94080b4
-                      ]
+                      , File ID: b0cda514c94080b4, Type: ODX Data, Download Size: 14559274, File Size: 14488498, Offset: 0, CRC32: 00000000
                     </li>
                   </ul>
                 </li>
@@ -161,7 +159,7 @@ Object {
             ECUs: 
             <ul>
               <li>
-                ICC, SW Version: 1000, HW Version: None, Mode: D
+                ICC, SW Version: 1000, HW Version: none, Mode: D
                 <br />
                 <ul>
                   <li>
@@ -170,9 +168,7 @@ Object {
                     >
                       LARGE.jpg
                     </a>
-                     [
-                    b0cda514c94080b4
-                    ]
+                    , File ID: b0cda514c94080b4, Type: ODX Data, Download Size: 14559274, File Size: 14488498, Offset: 0, CRC32: 00000000
                   </li>
                 </ul>
               </li>
diff --git a/src/components/Manifest/Details/index.jsx b/src/components/Manifest/Details/index.jsx
index 80b08da..32ca3e9 100644
--- a/src/components/Manifest/Details/index.jsx
+++ b/src/components/Manifest/Details/index.jsx
@@ -12,7 +12,12 @@ const ECUFileList = (files) => {
     <ul>
       {files.map((file) => (
         <li key={file.filename}>
-          <a href={file.url}>{file.filename}</a> [{file.file_id}]
+          <a href={file.url}>{file.filename}</a>
+          {`, File ID: ${file.file_id}, Type: ${file.type}, Download Size: ${
+            file.file_size
+          }, File Size: ${file.size}, Offset: ${file.offset ?? 0}, CRC32: ${
+            file.checksum ?? "00000000"
+          }`}
         </li>
       ))}
     </ul>
@@ -26,7 +31,7 @@ const ECUList = (ecus) => {
       {ecus.map((ecu) => (
         <li key={ecu.name}>
           {`${ecu.name}, SW Version: ${ecu.version ?? "None"}, HW Version: ${
-            ecu.hw_version ?? "None"
+            ecu.hw_version ?? "none"
           }, Mode: ${ecu.mode ?? "D"}${
             ecu.self_download ? ", Self Download" : ""
           }`}