Skip to content

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Dec 30, 2017

  1. The with_setup decorator does not appear to be designed to be used
    without parenthesis. Remove it, since it is not needed/used anyway.
  2. Fix the actual test.

1. The `with_setup` decorator does not appear to be designed to be used
   without parenthesis.  Remove it, since it is not needed/used anyway.
2. Fix the actual test.
@blueyed
Copy link
Contributor Author

blueyed commented Dec 30, 2017

btw: this makes the tests run on pytest.

What do you think about using pytest as main test runner?

@justinmk
Copy link
Member

justinmk commented Jan 1, 2018

What do you think about using pytest as main test runner

Definitely in favor, the work was done in #266 but it's blocked by a build failure that no one has investigated.

@justinmk justinmk merged commit 70dfc84 into neovim:master Apr 12, 2018
justinmk added a commit to justinmk/neovim that referenced this pull request Apr 12, 2018
justinmk added a commit to neovim/neovim that referenced this pull request Apr 12, 2018
@blueyed blueyed deleted the fix-test_command_output branch April 12, 2018 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants