{"action":"created","issue":{"url":"https://api.github.com/repos/ga4gh/compliance/issues/51","labels_url":"https://api.github.com/repos/ga4gh/compliance/issues/51/labels{/name}","comments_url":"https://api.github.com/repos/ga4gh/compliance/issues/51/comments","events_url":"https://api.github.com/repos/ga4gh/compliance/issues/51/events","html_url":"https://github.com/ga4gh/compliance/issues/51","id":101929539,"number":51,"title":"Read tests in variants package","user":{"login":"sarahhunt","id":6123326,"avatar_url":"https://avatars.githubusercontent.com/u/6123326?v=3","gravatar_id":"","url":"https://api.github.com/users/sarahhunt","html_url":"https://github.com/sarahhunt","followers_url":"https://api.github.com/users/sarahhunt/followers","following_url":"https://api.github.com/users/sarahhunt/following{/other_user}","gists_url":"https://api.github.com/users/sarahhunt/gists{/gist_id}","starred_url":"https://api.github.com/users/sarahhunt/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sarahhunt/subscriptions","organizations_url":"https://api.github.com/users/sarahhunt/orgs","repos_url":"https://api.github.com/users/sarahhunt/repos","events_url":"https://api.github.com/users/sarahhunt/events{/privacy}","received_events_url":"https://api.github.com/users/sarahhunt/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"hjellinek","id":4968570,"avatar_url":"https://avatars.githubusercontent.com/u/4968570?v=3","gravatar_id":"","url":"https://api.github.com/users/hjellinek","html_url":"https://github.com/hjellinek","followers_url":"https://api.github.com/users/hjellinek/followers","following_url":"https://api.github.com/users/hjellinek/following{/other_user}","gists_url":"https://api.github.com/users/hjellinek/gists{/gist_id}","starred_url":"https://api.github.com/users/hjellinek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjellinek/subscriptions","organizations_url":"https://api.github.com/users/hjellinek/orgs","repos_url":"https://api.github.com/users/hjellinek/repos","events_url":"https://api.github.com/users/hjellinek/events{/privacy}","received_events_url":"https://api.github.com/users/hjellinek/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-08-19T16:05:48Z","updated_at":"2015-08-19T22:46:28Z","closed_at":null,"body":"The HTML report for org.ga4gh.cts.api.variants.CallsetsSearchResponseCheckIT is reporting checkSearchRouting errors for reads,readgroupsets and references\r\n\r\nAre there a few missplaced or unneccessary makeUrl calls in:\r\n\r\ncts-java/src/test/java/org/ga4gh/cts/api/variants/CallsetsSearchResponseCheckIT.java"},"comment":{"url":"https://api.github.com/repos/ga4gh/compliance/issues/comments/132815052","html_url":"https://github.com/ga4gh/compliance/issues/51#issuecomment-132815052","issue_url":"https://api.github.com/repos/ga4gh/compliance/issues/51","id":132815052,"user":{"login":"hjellinek","id":4968570,"avatar_url":"https://avatars.githubusercontent.com/u/4968570?v=3","gravatar_id":"","url":"https://api.github.com/users/hjellinek","html_url":"https://github.com/hjellinek","followers_url":"https://api.github.com/users/hjellinek/followers","following_url":"https://api.github.com/users/hjellinek/following{/other_user}","gists_url":"https://api.github.com/users/hjellinek/gists{/gist_id}","starred_url":"https://api.github.com/users/hjellinek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjellinek/subscriptions","organizations_url":"https://api.github.com/users/hjellinek/orgs","repos_url":"https://api.github.com/users/hjellinek/repos","events_url":"https://api.github.com/users/hjellinek/events{/privacy}","received_events_url":"https://api.github.com/users/hjellinek/received_events","type":"User","site_admin":false},"created_at":"2015-08-19T22:46:28Z","updated_at":"2015-08-19T22:46:28Z","body":"That may be a matter of opinion: though the schemas don't define them, we do make use of HTTP response codes to signal the success or failure of API calls.  There are some tests in `CallsetsSearchResponseCheckIT` that check server endpoint responses to various types of calls (`POST`, `GET`, `OPTIONS`). \r\n\r\nIf you're specifically referring to the unused variable `allSearchUrls`, it's there for future expansion purposes, but I wouldn't say it's unnecessary.\r\n\r\nPerhaps I'm not understanding the issue you're raising.  Can you can restate it in terms of what you're expecting to see, and what you did see?\r\n"}}