web_dev_style: add check for ending jsdoc comments with **/ instead of */.

chrome/browser/resources/test_presubmit.py

 #!/usr/bin/env python
 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Unit tests for Web Development Style Guide checker."""
 
 import os
 import re
 import sys
@@ skipping 109 matching lines @@
   def testChromeSendPasses(self):
     lines = [
         "chrome.send('message', constructArgs('foo', []));",
         "  chrome.send('message', constructArgs('foo', []));",
         "chrome.send('message', constructArgs([]));",
         "  chrome.send('message', constructArgs([]));",
     ]
     for line in lines:
       self.ShouldPassChromeSendCheck(line)
 
+  def ShouldFailEndJsDocCommentCheck(self, line):
+    """Checks that the **/ checker flags |line| as a style error."""
+    error = self.checker.EndJsDocCommentCheck(1, line)
+    self.assertNotEqual('', error,
+        'Should be flagged as style error: ' + line)
+    self.assertEqual(self.GetHighlight(line, error), '**/')
+
+  def ShouldPassEndJsDocCommentCheck(self, line):
+    """Checks that the **/ checker doesn't flag |line| as a style error."""
+    self.assertEqual('', self.checker.EndJsDocCommentCheck(1, line),
+        'Should not be flagged as style error: ' + line)
+
+  def testEndJsDocCommentFails(self):
+    lines = [
+        "/** @override **/",
+        "/** @type {number} @const **/",
+        "  **/",
+        "**/  ",
+    ]
+    for line in lines:
+      self.ShouldFailEndJsDocCommentCheck(line)
+
+  def testEndJsDocCommentPasses(self):
+    lines = [
+        "/***************/",  # visual separators
+        "  */",  # valid JSDoc comment ends
+        "*/  ",
+        "/**/",  # funky multi-line comment enders
+        "/** @override */",  # legit JSDoc one-liners
+    ]
+    for line in lines:
+      self.ShouldPassEndJsDocCommentCheck(line)
+
   def ShouldFailGetElementByIdCheck(self, line):
     """Checks that the 'getElementById' checker flags |line| as a style
        error.
     """
     error = self.checker.GetElementByIdCheck(1, line)
     self.assertNotEqual('', error,
         'Should be flagged as style error: ' + line)
     self.assertEqual(self.GetHighlight(line, error), 'document.getElementById')
 
   def ShouldPassGetElementByIdCheck(self, line):
@@ skipping 484 matching lines @@
     opacity: .0;
     opacity: 0.0;
     opacity: 0.;
     border-width: 0mm;
     height: 0cm;
     width: 0in;
 """)
 
 if __name__ == '__main__':
   unittest.main()