{"action":"created","issue":{"url":"https://api.github.com/repos/Valloric/ycmd/issues/328","labels_url":"https://api.github.com/repos/Valloric/ycmd/issues/328/labels{/name}","comments_url":"https://api.github.com/repos/Valloric/ycmd/issues/328/comments","events_url":"https://api.github.com/repos/Valloric/ycmd/issues/328/events","html_url":"https://github.com/Valloric/ycmd/pull/328","id":129791564,"number":328,"title":"Allow skipping tests for specific completers","user":{"login":"mispencer","id":1434266,"avatar_url":"https://avatars.githubusercontent.com/u/1434266?v=3","gravatar_id":"","url":"https://api.github.com/users/mispencer","html_url":"https://github.com/mispencer","followers_url":"https://api.github.com/users/mispencer/followers","following_url":"https://api.github.com/users/mispencer/following{/other_user}","gists_url":"https://api.github.com/users/mispencer/gists{/gist_id}","starred_url":"https://api.github.com/users/mispencer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mispencer/subscriptions","organizations_url":"https://api.github.com/users/mispencer/orgs","repos_url":"https://api.github.com/users/mispencer/repos","events_url":"https://api.github.com/users/mispencer/events{/privacy}","received_events_url":"https://api.github.com/users/mispencer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2016-01-29T15:08:47Z","updated_at":"2016-02-01T21:40:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Valloric/ycmd/pulls/328","html_url":"https://github.com/Valloric/ycmd/pull/328","diff_url":"https://github.com/Valloric/ycmd/pull/328.diff","patch_url":"https://github.com/Valloric/ycmd/pull/328.patch"},"body":"Currently, you can skip tests for the clang completion engine. This pull requests extends this feature for the other completion engines.\r\n\r\nThe existing filter for clang was based on the test name. I attempted to use this method for the other completions, but not all tests were consistently named with the language or completer engine. So instead of refactoring all those name, I moved the test files so that all the tests for each completion engine were in their own directory, and, using a new nose plugin, excluded tests in said directory. As an additional benefit, directory filtering is less brittle to future changes.\r\n\r\nFor consistence, I also moved the other tests outside the test folder into the test folder.\n\n<!-- Reviewable:start -->\n[<img src=\"https://reviewable.io/review_button.svg\" height=\"40\" alt=\"Review on Reviewable\"/>](https://reviewable.io/reviews/valloric/ycmd/328)\n<!-- Reviewable:end -->\n"},"comment":{"url":"https://api.github.com/repos/Valloric/ycmd/issues/comments/178206606","html_url":"https://github.com/Valloric/ycmd/pull/328#issuecomment-178206606","issue_url":"https://api.github.com/repos/Valloric/ycmd/issues/328","id":178206606,"user":{"login":"vheon","id":90633,"avatar_url":"https://avatars.githubusercontent.com/u/90633?v=3","gravatar_id":"","url":"https://api.github.com/users/vheon","html_url":"https://github.com/vheon","followers_url":"https://api.github.com/users/vheon/followers","following_url":"https://api.github.com/users/vheon/following{/other_user}","gists_url":"https://api.github.com/users/vheon/gists{/gist_id}","starred_url":"https://api.github.com/users/vheon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vheon/subscriptions","organizations_url":"https://api.github.com/users/vheon/orgs","repos_url":"https://api.github.com/users/vheon/repos","events_url":"https://api.github.com/users/vheon/events{/privacy}","received_events_url":"https://api.github.com/users/vheon/received_events","type":"User","site_admin":false},"created_at":"2016-02-01T21:40:28Z","updated_at":"2016-02-01T21:40:28Z","body":"except the typo <img class=\"emoji\" title=\":lgtm:\" alt=\":lgtm:\" align=\"absmiddle\" src=\"https://reviewable.io/lgtm.png\" height=\"20\" width=\"61\"/> too\n\n---\n\nReview status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.\n\n---\n\n\n\n*Comments from the [review on Reviewable.io](https://reviewable.io:443/reviews/valloric/ycmd/328#-:-K9TlrezzNbt028PGg7a:-232293205)*\n<!-- Sent from Reviewable.io -->\n"}}