mgorny added a comment.
Thanks. I'll update it to incorporate the lately committed test fixes, and push
it later today.
Repository:
rL LLVM
https://reviews.llvm.org/D39763
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm
bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
I can't really check if all the tests are equivalent to the old ones, but not
having to install nose is a major usability improvement. Let's ship this.
We should totally fix the existing fai
mgorny added a comment.
Before:
==
FAIL: tests.cindex.test_code_completion.test_code_complete_availability
--
Traceback (most recent call last):
File
mgorny added a comment.
The diffs aren't very useful since I had to add a class for each test unit and
so everything needed reindenting. You can take my word that changes boil down
to:
- adding `unittest` import,
- adding class for each test unit and converting the functions into methods
(exce