[MRG] Make knownfailure work even without parentheses

[Akshay0724]

Fixes #731

Changes made in this Pull Request:

• modified mdanalysis/testsuite/MDAnalysisTests/plugins/knownfailure.py to support knownfailure decorator to work with and without parentheses.

PR Checklist

• Tests?
• Docs?
• CHANGELOG updated?
• Issue raised/referenced?

[richardjgowers]

@Akshay0724 looks good, can you also change where we currently use knownfailure() to not use the brackets? git grep 'knownfailure() gives me:

testsuite/MDAnalysisTests/__init__.py:plugin provides the `@knownfailure()` decorator, which can be used to mark tests
testsuite/MDAnalysisTests/coordinates/test_dcd.py:    @knownfailure()
testsuite/MDAnalysisTests/plugins/knownfailure.py:Beware that the decorator must be used as a function call: `@knownfai
testsuite/MDAnalysisTests/test_streamio.py:    @knownfailure()

The docs in __init__.py need updating to reflect that we don't need the brackets any more

[richardjgowers]

Looks good if travis passes. You can add yourself to the AUTHORS list if you want to as well.

[Akshay0724]

Is there is a way to reproduce this build failed using docker? If so then please tell me about docker image to be used. Thanks

…

On Feb 24, 2017 6:57 PM, "Richard Gowers" ***@***.***> wrote: Looks good if travis passes. You can add yourself to the AUTHORS <https://github.com/MDAnalysis/mdanalysis/blob/develop/package/AUTHORS> list if you want to as well. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1218 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ANfGMMa4KnXu9bBHdjG8VZqnaQyT02ftks5rftqsgaJpZM4MK0ZE> .