From 048ecb9c841f03f9914cad5c4ca1b5dc6c41d92d Mon Sep 17 00:00:00 2001
From: Ronnie Geraghty <rgeraghty@microsoft.com>
Date: Wed, 8 Feb 2023 17:37:20 -0800
Subject: [PATCH] Inventory Script Bug Fix (#5551)

* Added additional logging around line causing error

* add logging for read of spec dir

* removed extra logging and added check to test if item is dir before trying a dir read
---
 eng/scripts/inventory-dashboard/src/addEmptyDataPoints.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/eng/scripts/inventory-dashboard/src/addEmptyDataPoints.ts b/eng/scripts/inventory-dashboard/src/addEmptyDataPoints.ts
index 4575b65f4..1dacf2d5b 100644
--- a/eng/scripts/inventory-dashboard/src/addEmptyDataPoints.ts
+++ b/eng/scripts/inventory-dashboard/src/addEmptyDataPoints.ts
@@ -84,8 +84,11 @@ async function getServicesFromSpecRepo(packages: PackageList): Promise<PackageLi
     const serviceName = serviceNameMap[serviceSpecDir] === undefined ? serviceSpecDir : serviceNameMap[serviceSpecDir];
     // skip service api spec if it's in the list of services to hide
     if (servicesToHide[serviceName]) continue;
+    // test if service spec dir is a dir, if not move on. 
+    if (!fs.lstatSync(path.join(specsDirPath, serviceSpecDir)).isDirectory()) continue;
     // list of plane dirs in service spec dir, should be either data-plane and/or resource-manager
-    const planeSpecDirs = fs.readdirSync(path.join(specsDirPath, serviceSpecDir));
+    let planeSpecDirs: string[] = fs.readdirSync(path.join(specsDirPath, serviceSpecDir));
+    // loop through plane dirs in the service spec dir
     for (let planeSpecDir of planeSpecDirs) {
       // determine plane and SDK name
       let plane: Plane = "UNABLE TO BE DETERMINED";