Adding tests for parsing one line functions/ Fixing parser bug.

src/pydocstyle/parser.py

@@ -683,7 +683,9 @@ def parse_definition(self, class_):
         else:  # one-liner definition
             skipped_error_codes = ''
             docstring = self.parse_docstring()
-            decorators = []  # TODO
+            decorators = self._accumulated_decorators
+            self.log.debug("current accumulated decorators: %s", decorators)
+            self._accumulated_decorators = []
             children = []
             end = self.line
             self.leapfrog(tk.NEWLINE)

src/tests/parser_test.py

@@ -880,6 +880,19 @@ def test():
         def test():
             pass
     """),
+    CodeSnippet("""\
+        '''Test this'''
+
+        @property
+        def test(): pass
+    """),
+    CodeSnippet("""\
+    '''Test this'''
+
+    @first_decorator
+    @property
+    def test(): pass
+   """),
 ))
 def test_parsing_function_decorators(code):
     """Test to ensure we are correctly parsing function decorators."""

src/tests/test_cases/test.py

@@ -337,6 +337,33 @@ def oneliner_d102(): return
 def oneliner_withdoc(): """One liner"""
 
 
+def ignored_decorator(func):   # noqa: D400,D401,D415
+    """Runs something"""
+    func()
+    pass
+
+
+def decorator_for_test(func):   # noqa: D400,D401,D415
+    """Runs something"""
+    func()
+    pass
+
+
+@ignored_decorator
+def oneliner_ignored_decorator(): """One liner"""
+
+
+@decorator_for_test
+@expect("D400: First line should end with a period (not 'r')")
+@expect("D415: First line should end with a period, question mark,"
+        " or exclamation point (not 'r')")
+def oneliner_with_decorator_expecting_errors(): """One liner"""
+
+
+@decorator_for_test
+def valid_oneliner_with_decorator(): """One liner."""
+
+
 @expect("D207: Docstring is under-indented")
 @expect('D213: Multi-line docstring summary should start at the second line')
 def docstring_start_in_same_line(): """First Line.

src/tests/test_definitions.py

@@ -37,7 +37,8 @@ def test_complex_file(test_case):
                                   test_case + '.py')
     results = list(check([test_case_file],
                          select=set(ErrorRegistry.get_error_codes()),
-                         ignore_decorators=re.compile('wraps')))
+                         ignore_decorators=re.compile(
+                             'wraps|ignored_decorator')))
     for error in results:
         assert isinstance(error, Error)
     results = {(e.definition.name, e.message) for e in results}