From 9c50beb364eced0a3a7e9eebb978a813cf7195a2 Mon Sep 17 00:00:00 2001
From: Erik Poul <erik.poul@mgid.com>
Date: Thu, 24 Dec 2020 16:56:41 +0200
Subject: [PATCH] fix tags parsing into namespace

---
 lib/gen-utils.ts            |  4 ++--
 test/all-operations.json    |  8 +++++++-
 test/all-operations.spec.ts | 12 ++++++++++--
 3 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/lib/gen-utils.ts b/lib/gen-utils.ts
index d720956..13b3a4d 100644
--- a/lib/gen-utils.ts
+++ b/lib/gen-utils.ts
@@ -51,14 +51,14 @@ export function modelFile(pathToModels: string, name: string, options: Options):
 }
 
 /**
- * Returns the namespace path, that is, the part before the last '.' splitted by '/' instead of '.'.
+ * Returns the namespace path, that is, the part before the last '.' split by '/' instead of '.'.
  * If there's no namespace, returns undefined.
  */
 export function namespace(name: string): string | undefined {
   name = name.replace(/^\.+/g, '');
   name = name.replace(/\.+$/g, '');
   const pos = name.lastIndexOf('.');
-  return pos < 0 ? undefined : name.substring(0, pos).replace('.', '/');
+  return pos < 0 ? undefined : name.substring(0, pos).replace(/\./g, '/');
 }
 
 /**
diff --git a/test/all-operations.json b/test/all-operations.json
index 35617e2..cf6df0c 100644
--- a/test/all-operations.json
+++ b/test/all-operations.json
@@ -22,6 +22,9 @@
     {
       "name": "tag2"
     },
+    {
+      "name": "tag.tag2.tag3.tag4.tag5"
+    },
     {
       "name": "unused",
       "description": "Unused description"
@@ -340,6 +343,9 @@
     },
     "/path5": {
       "get": {
+        "tags": [
+          "tag.tag2.tag3.tag4.tag5"
+        ],
         "summary": "A path that contains a reference to response objects",
         "responses": {
           "200": {
@@ -409,4 +415,4 @@
       }
     }
   }
-}
\ No newline at end of file
+}
diff --git a/test/all-operations.spec.ts b/test/all-operations.spec.ts
index aa62352..591f10a 100644
--- a/test/all-operations.spec.ts
+++ b/test/all-operations.spec.ts
@@ -15,7 +15,7 @@ describe('Generation tests using all-operations.json', () => {
   });
 
   it('Tags', () => {
-    expect(gen.services.size).toBe(3);
+    expect(gen.services.size).toBe(4);
   });
 
   it('Tag 1', done => {
@@ -98,11 +98,19 @@ describe('Generation tests using all-operations.json', () => {
     });
   });
 
+  it('Tag with nesting', () => {
+    const tagWithNesting = gen.services.get('tag.tag2.tag3.tag4.tag5');
+    expect(tagWithNesting).toBeDefined();
+    if (!tagWithNesting) return;
+
+    expect(tagWithNesting.namespace).toBe('tag/tag2/tag3/tag4');
+  });
+
   it('No tag', done => {
     const noTag = gen.services.get('noTag');
     expect(noTag).toBeDefined();
     if (!noTag) return;
-    expect(noTag.operations.length).toBe(4);
+    expect(noTag.operations.length).toBe(3);
 
     const ts = gen.templates.apply('service', noTag);
     const parser = new TypescriptParser();