Regarding the introduction of get_cursor(...): same as in the other patch: looks like a good idea, but then use that in the other tests, too. Also, like in the other patch, I'm still confused whether there's a scheme behind when you use ok_ and when you use assert...
In general: very nice tests :) Cheers, /Manuel On Fri, Feb 17, 2012 at 8:57 AM, Gregory Szorc <[email protected]> wrote: > I incorporated feedback from Tobias and exposed arguments as an > iterable and indexable container object. > > --- > bindings/python/clang/cindex.py | 47 +++++++++++++++++++ > bindings/python/tests/cindex/test_type.py | 69 > +++++++++++++++++++++++++++++ > 2 files changed, 116 insertions(+), 0 deletions(-) > > _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
