diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-09-23 22:16:57 -0500 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-09-23 22:16:57 -0500 |
commit | 2ba804e6db52637fbe23fbecfd0e65b4db1a49e2 (patch) | |
tree | 3ee26fc1116d7c4e50dd0385640ad6c8d613f43c /python/tidy/servo_tidy_tests/test_tidy.py | |
parent | 28ebab43987b0f9829a03a3c043e621cae2db1fe (diff) | |
parent | 0ac212afb18c98ea929368e9ccf21101e7a8921c (diff) | |
download | servo-2ba804e6db52637fbe23fbecfd0e65b4db1a49e2.tar.gz servo-2ba804e6db52637fbe23fbecfd0e65b4db1a49e2.zip |
Auto merge of #13361 - Mylainos:issue-13350, r=Wafflespeanut
Report tidy error for space after (
<!-- Please describe your changes on the following line: -->
Add report in tidy for code which have a space after the `(` like `some_function( argument)`
---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #13350 (github issue number if applicable).
<!-- Either: -->
- [X] There are tests for these changes OR
- [ ] These changes do not require tests because _____
<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/13361)
<!-- Reviewable:end -->
Diffstat (limited to 'python/tidy/servo_tidy_tests/test_tidy.py')
-rw-r--r-- | python/tidy/servo_tidy_tests/test_tidy.py | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/python/tidy/servo_tidy_tests/test_tidy.py b/python/tidy/servo_tidy_tests/test_tidy.py index 998967c617f..52371b944bb 100644 --- a/python/tidy/servo_tidy_tests/test_tidy.py +++ b/python/tidy/servo_tidy_tests/test_tidy.py @@ -109,6 +109,9 @@ class CheckTidiness(unittest.TestCase): self.assertEqual('use &T instead of &Root<T>', errors.next()[2]) self.assertEqual('operators should go at the end of the first line', errors.next()[2]) self.assertEqual('else braces should be on the same line', errors.next()[2]) + self.assertEqual('extra space after (', errors.next()[2]) + self.assertEqual('extra space after (', errors.next()[2]) + self.assertEqual('extra space after (', errors.next()[2]) self.assertNoMoreErrors(errors) def test_spec_link(self): |