process implements tags before augments tags (#906)

lib/jsdoc/augment.js

@@ -513,7 +513,7 @@ exports.addImplemented = function(doclets) {
  * @return {void}
  */
 exports.augmentAll = function(doclets) {
-    exports.addInherited(doclets);
     exports.addMixedIn(doclets);
     exports.addImplemented(doclets);
+    exports.addInherited(doclets);
 };

test/fixtures/augmentall.js

@@ -0,0 +1,27 @@
+/**
+ * Parent interface.
+ * @interface
+ */
+function Connection() {}
+
+/**
+ * Open the connection.
+ */
+Connection.prototype.open = function() {};
+
+/**
+ * Child class.
+ * @class
+ * @implements {Connection}
+ */
+function Socket() {}
+
+/** @inheritdoc */
+Socket.prototype.open = function() {};
+
+/**
+ * Extension of child class.
+ * @class
+ * @extends {Socket}
+ */
+function EncryptedSocket() {}

test/jasmine-jsdoc.js

@@ -142,7 +142,7 @@ jasmine.asyncSpecDone = function() {
     jasmine.asyncSpecWait.done = true;
 };
 
-jasmine.getDocSetFromFile = function(filename, parser, validate) {
+jasmine.getDocSetFromFile = function(filename, parser, validate, augment) {
     var doclets;
     var validationResult;
 
@@ -154,8 +154,9 @@ jasmine.getDocSetFromFile = function(filename, parser, validate) {
     doclets = testParser.parse('javascript:' + sourceCode);
     jsdoc.borrow.indexAll(doclets);
 
-    jsdoc.augment.addInherited(doclets);
-    jsdoc.augment.addImplemented(doclets);
+    if (augment !== false) {
+        jsdoc.augment.augmentAll(doclets);
+    }
 
     // test assume borrows have not yet been resolved
     // require('jsdoc/borrow').resolveBorrows(doclets);

test/specs/jsdoc/augment.js

@@ -39,7 +39,7 @@ describe('jsdoc/augment', function() {
 
     describe('augmentAll', function() {
         it('should call all other methods that the module exports', function() {
-            var docSet = jasmine.getDocSetFromFile('test/fixtures/mixintag2.js');
+            var docSet = jasmine.getDocSetFromFile('test/fixtures/mixintag2.js', null, null, false);
             var methodNames = Object.keys(augment).filter(function(name) {
                 return name !== 'augmentAll';
             });
@@ -48,11 +48,22 @@ describe('jsdoc/augment', function() {
                 spyOn(augment, name);
             });
 
-            augment.augmentAll(docSet);
+            augment.augmentAll(docSet.doclets);
 
             methodNames.forEach(function(name) {
                 expect(augment[name]).toHaveBeenCalled();
             });
         });
+
+        it('should process @implements tags before @augments tags', function() {
+            var docSet = jasmine.getDocSetFromFile('test/fixtures/augmentall.js', null, null, false);
+            var open;
+
+            augment.augmentAll(docSet.doclets);
+
+            open = docSet.getByLongname('EncryptedSocket#open')[0];
+
+            expect(open.description).toBe('Open the connection.');
+        });
     });
 });