firstSegmentSplit case application

j-d-carmichael · Sep 18, 2023 · e31345f · e31345f
1 parent 4a738bf
commit e31345f
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/src/UniqueOperationIds.ts b/src/UniqueOperationIds.ts
@@ -41,7 +41,6 @@ class UniqueOperationIds {
     let iterationToNotCaseChange = -1;
     if (firstSegmentSplit) {
       iterationToNotCaseChange = 0;
-      filePathParts[0] = filePathParts[0] + firstSegmentSplit;
     }
 
     // inject the prefixes if given
@@ -53,7 +52,7 @@ class UniqueOperationIds {
     }
 
     for (let i = 0; i < filePathParts.length; ++i) {
-      if (filePathParts[i] !== sep && i !== iterationToNotCaseChange) {
+      if (filePathParts[i] !== sep) {
         switch (segmentStyle) {
           case StringStyle.snakeCase:
             filePathParts[i] = _.snakeCase(this.removeCurlys(filePathParts[i]));
@@ -72,6 +71,10 @@ class UniqueOperationIds {
         if ([StringStyle.camelCase, StringStyle.PascalCase].includes(style)) {
           filePathParts[i] = ucFirst(filePathParts[i]);
         }
+        // Add the split highlight
+        if (i === iterationToNotCaseChange) {
+          filePathParts[i] += firstSegmentSplit;
+        }
       }
     }
 

diff --git a/src/__tests__/UniqueOperationIds.spec.ts b/src/__tests__/UniqueOperationIds.spec.ts
@@ -186,14 +186,14 @@ describe('firstSegmentSplit tests', () => {
     ).toBe('weather_IsBadGet');
   });
 
-  it('should separate with -', async () => {
+  it('should separate with - but also the camel case should be applied to the first segment even though there is a split highlight passed', async () => {
     expect(
       UniqueOperationIds.getUniqueOperationIdFromPath({
-        filePath: 'src/paths/weather/is-bad/get.yml',
+        filePath: 'src/paths/weather-is/bad/get.yml',
         stripValue: 'src/paths/',
         cwd: 'some/dir/',
         firstSegmentSplit: '-'
       })
-    ).toBe('weather-IsBadGet');
+    ).toBe('weatherIs-BadGet');
   });
 });