{"action":"created","issue":{"url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/98","repository_url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite","labels_url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/98/labels{/name}","comments_url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/98/comments","events_url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/98/events","html_url":"https://github.com/adlnet/lrs-conformance-test-suite/pull/98","id":195089541,"number":98,"title":"Delay sequence","user":{"login":"cr8onski","id":6187779,"avatar_url":"https://avatars.githubusercontent.com/u/6187779?v=3","gravatar_id":"","url":"https://api.github.com/users/cr8onski","html_url":"https://github.com/cr8onski","followers_url":"https://api.github.com/users/cr8onski/followers","following_url":"https://api.github.com/users/cr8onski/following{/other_user}","gists_url":"https://api.github.com/users/cr8onski/gists{/gist_id}","starred_url":"https://api.github.com/users/cr8onski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cr8onski/subscriptions","organizations_url":"https://api.github.com/users/cr8onski/orgs","repos_url":"https://api.github.com/users/cr8onski/repos","events_url":"https://api.github.com/users/cr8onski/events{/privacy}","received_events_url":"https://api.github.com/users/cr8onski/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-12-12T21:23:36Z","updated_at":"2016-12-14T18:58:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/pulls/98","html_url":"https://github.com/adlnet/lrs-conformance-test-suite/pull/98","diff_url":"https://github.com/adlnet/lrs-conformance-test-suite/pull/98.diff","patch_url":"https://github.com/adlnet/lrs-conformance-test-suite/pull/98.patch"},"body":"fixes #96"},"comment":{"url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/comments/267123027","html_url":"https://github.com/adlnet/lrs-conformance-test-suite/pull/98#issuecomment-267123027","issue_url":"https://api.github.com/repos/adlnet/lrs-conformance-test-suite/issues/98","id":267123027,"user":{"login":"cr8onski","id":6187779,"avatar_url":"https://avatars.githubusercontent.com/u/6187779?v=3","gravatar_id":"","url":"https://api.github.com/users/cr8onski","html_url":"https://github.com/cr8onski","followers_url":"https://api.github.com/users/cr8onski/followers","following_url":"https://api.github.com/users/cr8onski/following{/other_user}","gists_url":"https://api.github.com/users/cr8onski/gists{/gist_id}","starred_url":"https://api.github.com/users/cr8onski/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cr8onski/subscriptions","organizations_url":"https://api.github.com/users/cr8onski/orgs","repos_url":"https://api.github.com/users/cr8onski/repos","events_url":"https://api.github.com/users/cr8onski/events{/privacy}","received_events_url":"https://api.github.com/users/cr8onski/received_events","type":"User","site_admin":false},"created_at":"2016-12-14T18:58:09Z","updated_at":"2016-12-14T18:58:09Z","body":"It was just in these tests.  The problem was that the GET would start (and sometimes finish) before the POST ended.  It happened in cases where the requests were continuously chained:\r\n```\r\nrequest(helper.getEndpointAndAuth())\r\n.post(helper.getEndpointStatements())\r\n...\r\n.end()\r\n.get(helper.getEndpointStatements())\r\n.wait(helper.genDelay(...))\r\n...\r\n```\r\nNesting the GET inside of the .end() of the POST takes care of that problem and wait and genDelay work fine from there.\r\nThat's what these fixes are, nesting wait/genDelays which had been chained. So now the GET does not start until the POST completes, and then genDelay delays while looking for the statement to register in the LRS or verifying against the consistent-through header."}}
{"actor":{"display_login":"cr8onski"}}