correctly write doclets at es6 class constructors

src/Emitter.ts

@@ -4,17 +4,18 @@ import { warn } from './logger';
 import { assertNever } from './assert_never';
 import {
     createClass,
-    createFunction,
+    createClassMember,
     createClassMethod,
-    createInterfaceMethod,
+    createConstructor,
+    createEnum,
+    createFunction,
     createInterface,
-    createClassMember,
     createInterfaceMember,
-    createNamespaceMember,
+    createInterfaceMethod,
     createModule,
     createNamespace,
+    createNamespaceMember,
     createTypedef,
-    createEnum,
 } from './create_helpers';
 
 interface IDocletTreeNode
@@ -41,10 +42,16 @@ function isEnum(doclet: TDoclet)
 
 function shouldMoveOutOfClass(doclet: TDoclet)
 {
-    return isClassLike(doclet)
+    return !isConstructor(doclet)
+        && (isClassLike(doclet)
         || isModuleLike(doclet)
         || isEnum(doclet)
-        || doclet.kind === 'typedef';
+        || doclet.kind === 'typedef');
+}
+
+function isConstructor(doclet: TDoclet)
+{
+    return doclet.kind === "class" && doclet.name === doclet.memberof
 }
 
 export class Emitter
@@ -238,7 +245,12 @@ export class Emitter
         switch (node.doclet.kind)
         {
             case 'class':
-                return createClass(node.doclet, children);
+                if (isConstructor(node.doclet)) {
+                    // constructor in es6 classes with own doclet
+                    return createConstructor(node.doclet)
+                } else {
+                    return createClass(node.doclet, children);
+                }
 
             case 'constant':
             case 'member':

src/create_helpers.ts

@@ -253,15 +253,9 @@ export function createEnum(doclet: IMemberDoclet): ts.EnumDeclaration
 
 export function createClassMember(doclet: IMemberDoclet): ts.PropertyDeclaration
 {
-    const mods: ts.Modifier[] = [];
     const type = resolveType(doclet.type, doclet);
 
-    if (doclet.access === 'private')
-        mods.push(ts.createModifier(ts.SyntaxKind.PrivateKeyword));
-    else if (doclet.access === 'protected')
-        mods.push(ts.createModifier(ts.SyntaxKind.ProtectedKeyword));
-    else if (doclet.access === 'public')
-        mods.push(ts.createModifier(ts.SyntaxKind.PublicKeyword));
+    const mods: ts.Modifier[] = getAccessModifiers(doclet);
 
     if (doclet.scope === 'static')
         mods.push(ts.createModifier(ts.SyntaxKind.StaticKeyword));
@@ -279,6 +273,31 @@ export function createClassMember(doclet: IMemberDoclet): ts.PropertyDeclaration
     ));
 }
 
+function getAccessModifiers(doclet: IMemberDoclet | IClassDoclet)
+{
+    const mods: ts.Modifier[] = [];
+
+    if (doclet.access === 'private' || doclet.access === 'package')
+        mods.push(ts.createModifier(ts.SyntaxKind.PrivateKeyword));
+    else if (doclet.access === 'protected')
+        mods.push(ts.createModifier(ts.SyntaxKind.ProtectedKeyword));
+    else if (doclet.access === 'public')
+        mods.push(ts.createModifier(ts.SyntaxKind.PublicKeyword));
+
+    return mods
+}
+
+export function createConstructor(doclet: IClassDoclet)
+{
+    const params = createFunctionParams(doclet);
+    return handleComment(doclet, ts.createConstructor(
+        undefined,  // decorators
+        getAccessModifiers(doclet),  // modifiers
+        params,     // parameters
+        undefined   // body
+    ))
+}
+
 export function createInterfaceMember(doclet: IMemberDoclet): ts.PropertySignature
 {
     const mods: ts.Modifier[] = [];

src/typings/jsdoc.d.ts

@@ -76,7 +76,7 @@ declare interface IDocletBase {
     tags?: IDocletTag[];
     memberof?: string;
     see?: string;
-    access?: ('public' | 'private' | 'protected');
+    access?: ('public' | 'private' | 'protected' | 'package');
     examples?: string;
     deprecated?: string;
     defaultvalue?: string;

test/expected/constructors.d.ts

@@ -0,0 +1,55 @@
+/**
+ * Class documentation.
+ */
+declare class MyClass {
+
+    /**
+     * Constructor documentation.
+     * @public
+     * @param {number} a
+     * @param {string} b
+     */
+    public constructor(a: number, b: string);
+}
+
+/**
+ * Class documentation.
+ */
+declare class MyClass2 {
+
+    /**
+     * Constructor documentation.
+     * @protected
+     * @param {number} a
+     * @param {string} b
+     */
+    protected constructor(a: number, b: string);
+}
+
+/**
+ * Class documentation.
+ */
+declare class MyClass3 {
+
+    /**
+     * Constructor documentation.
+     * @package
+     * @param {number} a
+     * @param {string} b
+     */
+    private constructor(a: number, b: string);
+}
+
+/**
+ * Class documentation.
+ */
+declare class MyClass4 {
+
+    /**
+     * Constructor documentation.
+     * @private
+     * @param {number} a
+     * @param {string} b
+     */
+    private constructor(a: number, b: string);
+}

test/fixtures/constructors.js

@@ -0,0 +1,55 @@
+/**
+ * Class documentation.
+ */
+class MyClass {
+
+    /**
+     * Constructor documentation.
+     * @public
+     * @param {number} a
+     * @param {string} b
+     */
+    constructor(a, b) {}
+}
+
+/**
+ * Class documentation.
+ */
+class MyClass2 {
+
+    /**
+     * Constructor documentation.
+     * @protected
+     * @param {number} a
+     * @param {string} b
+     */
+    constructor(a, b) {}
+}
+
+/**
+ * Class documentation.
+ */
+class MyClass3 {
+
+    /**
+     * Constructor documentation.
+     * @package
+     * @param {number} a
+     * @param {string} b
+     */
+    constructor(a, b) {}
+}
+
+/**
+ * Class documentation.
+ */
+class MyClass4 {
+
+    /**
+     * Constructor documentation.
+     * @private
+     * @param {number} a
+     * @param {string} b
+     */
+    constructor(a, b) {}
+}

test/specs/constructors.ts

@@ -0,0 +1,7 @@
+import { expectJsDoc } from '../lib';
+
+suite('Constructor Checks', () => {
+    test('All', () => {
+        expectJsDoc('constructors');
+    });
+});