CLOUDP-306161: Improve single resource path evaluation

Author: yelizhenden-mdb

tools/spectral/ipa/__tests__/utils/resourceEvaluation.test.js

@@ -227,6 +227,11 @@ describe('tools/spectral/ipa/rulesets/functions/utils/resourceEvaluation.js', ()
         path: '/resourceOne/{id}/{id}',
         isSingleResourceIdentifier: false,
       },
+      {
+        description: 'invalid single identifier',
+        path: '/resource/resource/{id}',
+        isSingleResourceIdentifier: false,
+      },
       {
         description: 'single identifier',
         path: '/resource/{id}',
@@ -236,7 +241,7 @@ describe('tools/spectral/ipa/rulesets/functions/utils/resourceEvaluation.js', ()
         description: 'single identifier child',
         path: '/resource/{id}/child/{id}',
         isSingleResourceIdentifier: true,
-      },
+      }
     ];
 
     testCases.forEach((testCase) => {

tools/spectral/ipa/rulesets/functions/utils/resourceEvaluation.js

@@ -27,8 +27,24 @@ export function isResourceCollectionIdentifier(path) {
  * @returns {boolean} true if the path represents a single resource, false otherwise
  */
 export function isSingleResourceIdentifier(path) {
-  const pattern = new RegExp(`^.*/[a-zA-Z]+/{[a-zA-Z]+}$`);
-  return pattern.test(path);
+  const p = removePrefix(path);
+
+  // Check if the path ends with /{paramName} pattern
+  const endsWithParamPattern = /\/\{[a-zA-Z][a-zA-Z0-9]*}$/;
+
+  if (!endsWithParamPattern.test(p)) {
+    return false;
+  }
+
+  // Extract the part before the final parameter
+  const lastSlashBeforeParam = p.lastIndexOf('/');
+  if (lastSlashBeforeParam === -1) {
+    return false;
+  }
+
+  // Check if the preceding part is a valid resource collection identifier
+  const collectionPath = p.substring(0, lastSlashBeforeParam);
+  return isResourceCollectionIdentifier(collectionPath);
 }
 
 export function isCustomMethodIdentifier(path) {