Removed maxDepth. Let server run the validation

Author: rg911

package-lock.json

@@ -35,21 +35,20 @@ export class IdGenerator {
     /**
      * Parses a unified namespace name into a path.
      * @param {string} name The unified namespace name.
-     * @param {number} maxDepth The max namespace depth (network configuration, default: 3)
      * @returns {array<module:coders/uint64~uint64>} The namespace path.
      */
-    public static generateNamespacePath = (name: string, maxDepth: number = idGeneratorConst.default_namespace_max_depth) => {
+    public static generateNamespacePath = (name: string) => {
         if (0 >= name.length) {
             utilities.throwInvalidFqn('having zero length', name);
         }
         let namespaceId = utilities.idGeneratorConst.namespace_base_id;
         const path = [];
         const start = utilities.split(name, (substringStart, size) => {
             namespaceId = utilities.generateNamespaceId(namespaceId, utilities.extractPartName(name, substringStart, size));
-            utilities.append(path, namespaceId, name, maxDepth);
+            utilities.append(path, namespaceId, name);
         });
         namespaceId = utilities.generateNamespaceId(namespaceId, utilities.extractPartName(name, start, name.length - start));
-        utilities.append(path, namespaceId, name, maxDepth);
+        utilities.append(path, namespaceId, name);
         return path;
     }
 }

src/core/format/IdGenerator.ts

@@ -94,7 +94,6 @@ export const tryParseUint = (str) => {
 
 export const idGeneratorConst = {
     namespace_base_id: [0, 0],
-    default_namespace_max_depth: 3,
     name_pattern: /^[a-z0-9][a-z0-9-_]*$/,
 };
 
@@ -113,10 +112,7 @@ export const extractPartName = (name, start, size) => {
     return partName;
 };
 
-export const append = (path, id, name, maxDepth) => {
-    if (maxDepth === path.length) {
-        this.throwInvalidFqn('too many parts', name);
-    }
+export const append = (path, id, name) => {
     path.push(id);
 };
 

src/core/format/Utilities.ts

@@ -234,21 +234,11 @@ describe('id generator', () => {
             expect(idGenerator.generateNamespacePath('foo.bar.baz')).to.deep.equal(expected);
         });
 
-        it('rejects names with too many parts', () => {
-            // Assert:
-            ['a.b.c.d', 'a.b.c.d.e'].forEach((name) =>
-                expect(() => idGenerator.generateNamespacePath(name), `name ${name}`).to.throw('too many parts'));
-        });
-
         it('rejects improper qualified names', () => {
             // Assert:
             ['a:b:c', 'a::b'].forEach((name) =>
                 expect(() => idGenerator.generateNamespacePath(name), `name ${name}`).to.throw('invalid part name'));
         });
-
-        it('rejects names depth exceeds maxDepth', () => {
-            expect(() => idGenerator.generateNamespacePath('nem.xem', 1)).to.throw(Error, 'too many parts');
-        });
         addBasicTests(idGenerator.generateNamespacePath);
     });
 });