{"action":"created","issue":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/2071","labels_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/2071/labels{/name}","comments_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/2071/comments","events_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/2071/events","html_url":"https://github.com/RedHatQE/cfme_tests/pull/2071","id":92068598,"number":2071,"title":"Add the service catalog item ordering via REST.","user":{"login":"mfalesni","id":2163040,"avatar_url":"https://avatars.githubusercontent.com/u/2163040?v=3","gravatar_id":"","url":"https://api.github.com/users/mfalesni","html_url":"https://github.com/mfalesni","followers_url":"https://api.github.com/users/mfalesni/followers","following_url":"https://api.github.com/users/mfalesni/following{/other_user}","gists_url":"https://api.github.com/users/mfalesni/gists{/gist_id}","starred_url":"https://api.github.com/users/mfalesni/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mfalesni/subscriptions","organizations_url":"https://api.github.com/users/mfalesni/orgs","repos_url":"https://api.github.com/users/mfalesni/repos","events_url":"https://api.github.com/users/mfalesni/events{/privacy}","received_events_url":"https://api.github.com/users/mfalesni/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/labels/lint-ok","name":"lint-ok","color":"328832"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-06-30T11:35:07Z","updated_at":"2015-08-03T20:50:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/pulls/2071","html_url":"https://github.com/RedHatQE/cfme_tests/pull/2071","diff_url":"https://github.com/RedHatQE/cfme_tests/pull/2071.diff","patch_url":"https://github.com/RedHatQE/cfme_tests/pull/2071.patch"},"body":"Also adds a new feature, `pytest.wait_for` wrapper. Usage in the newly written test. @psav has already seen it, @seandst ?\r\n\r\n{{pytest: cfme/tests/services/test_service_catalogs.py -v --long-running -k rest}}"},"comment":{"url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/comments/127403116","html_url":"https://github.com/RedHatQE/cfme_tests/pull/2071#issuecomment-127403116","issue_url":"https://api.github.com/repos/RedHatQE/cfme_tests/issues/2071","id":127403116,"user":{"login":"seandst","id":441252,"avatar_url":"https://avatars.githubusercontent.com/u/441252?v=3","gravatar_id":"","url":"https://api.github.com/users/seandst","html_url":"https://github.com/seandst","followers_url":"https://api.github.com/users/seandst/followers","following_url":"https://api.github.com/users/seandst/following{/other_user}","gists_url":"https://api.github.com/users/seandst/gists{/gist_id}","starred_url":"https://api.github.com/users/seandst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/seandst/subscriptions","organizations_url":"https://api.github.com/users/seandst/orgs","repos_url":"https://api.github.com/users/seandst/repos","events_url":"https://api.github.com/users/seandst/events{/privacy}","received_events_url":"https://api.github.com/users/seandst/received_events","type":"User","site_admin":false},"created_at":"2015-08-03T20:50:53Z","updated_at":"2015-08-03T20:50:53Z","body":"@mfalesni, I poked a little bit at `parsedatetime` and I'm not sure it works as well as we need/want, though I might be making the requirements a little too broad.\r\n\r\nFirst, I'm sad that a module called `parsedatetime` returns time tuples. Second, notice how putting multiple fragments breaks the time parsing:\r\n\r\n```\r\n17 >>> c.parse('3d')\r\n   17: \r\n(time.struct_time(tm_year=2015, tm_mon=8, tm_mday=6, tm_hour=16, tm_min=43, tm_sec=57, tm_wday=3, tm_yday=218, tm_isdst=-1),\r\n 1)\r\n\r\n18 >>> c.parse('3d1h')\r\n   18: \r\n(time.struct_time(tm_year=2015, tm_mon=8, tm_mday=3, tm_hour=16, tm_min=44, tm_sec=1, tm_wday=0, tm_yday=215, tm_isdst=1),\r\n 0)\r\n\r\n19 >>> c.parse('1h')\r\n   19: \r\n(time.struct_time(tm_year=2015, tm_mon=8, tm_mday=3, tm_hour=17, tm_min=44, tm_sec=33, tm_wday=0, tm_yday=215, tm_isdst=-1),\r\n 2)\r\n```\r\n\r\n`3d`(prompt 17) adds three days, as you'd expect. `1h` (prompt 19) adds 1 hour, again, just like you'd expect. But `3d1h` (prompt 18) seems to completely give up. Neither the `3d` nor the `1h` is evaluated. The [`parse` docs] (https://bear.im/code/parsedatetime/docs/parsedatetime.Calendar-class.html#parse) lead me to believe that this usage should work, but I could be misreading them or simply looking at the wrong method.\r\n\r\nLet me know what you think."}}