{"action":"created","issue":{"url":"https://api.github.com/repos/Intrepidd/working_hours/issues/28","labels_url":"https://api.github.com/repos/Intrepidd/working_hours/issues/28/labels{/name}","comments_url":"https://api.github.com/repos/Intrepidd/working_hours/issues/28/comments","events_url":"https://api.github.com/repos/Intrepidd/working_hours/issues/28/events","html_url":"https://github.com/Intrepidd/working_hours/issues/28","id":123063482,"number":28,"title":"WorkingHours.return_to_working_time() doesn't pass `.in_working_hours?`","user":{"login":"rafalyesware","id":4246934,"avatar_url":"https://avatars.githubusercontent.com/u/4246934?v=3","gravatar_id":"","url":"https://api.github.com/users/rafalyesware","html_url":"https://github.com/rafalyesware","followers_url":"https://api.github.com/users/rafalyesware/followers","following_url":"https://api.github.com/users/rafalyesware/following{/other_user}","gists_url":"https://api.github.com/users/rafalyesware/gists{/gist_id}","starred_url":"https://api.github.com/users/rafalyesware/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rafalyesware/subscriptions","organizations_url":"https://api.github.com/users/rafalyesware/orgs","repos_url":"https://api.github.com/users/rafalyesware/repos","events_url":"https://api.github.com/users/rafalyesware/events{/privacy}","received_events_url":"https://api.github.com/users/rafalyesware/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-12-19T05:24:27Z","updated_at":"2015-12-19T08:52:14Z","closed_at":null,"body":"When one returns a time to working time, it would seem sensible that the time actually passes the `.in_working_hours?` test in all cases.  \r\n\r\nHowever, if the time passed to `return_to_working_time` was outside working hours to start with, then `WorkingHours.return_to_working_time` will return a time that does not pass the `.in_working_hours?` predicate (because it returns the exact closing time).\r\n\r\nSee for example:\r\n```\r\n2.2.2 :006 > Time.now.in_working_hours?\r\n=> false\r\n2.2.2 :007 > WorkingHours.return_to_working_time(Time.now).in_working_hours? \r\n=> false\r\n2.2.2 :008 > (WorkingHours.return_to_working_time(Time.now) - 1.second).in_working_hours?\r\n=> true\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/Intrepidd/working_hours/issues/comments/165961467","html_url":"https://github.com/Intrepidd/working_hours/issues/28#issuecomment-165961467","issue_url":"https://api.github.com/repos/Intrepidd/working_hours/issues/28","id":165961467,"user":{"login":"jarthod","id":201687,"avatar_url":"https://avatars.githubusercontent.com/u/201687?v=3","gravatar_id":"","url":"https://api.github.com/users/jarthod","html_url":"https://github.com/jarthod","followers_url":"https://api.github.com/users/jarthod/followers","following_url":"https://api.github.com/users/jarthod/following{/other_user}","gists_url":"https://api.github.com/users/jarthod/gists{/gist_id}","starred_url":"https://api.github.com/users/jarthod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jarthod/subscriptions","organizations_url":"https://api.github.com/users/jarthod/orgs","repos_url":"https://api.github.com/users/jarthod/repos","events_url":"https://api.github.com/users/jarthod/events{/privacy}","received_events_url":"https://api.github.com/users/jarthod/received_events","type":"User","site_admin":false},"created_at":"2015-12-19T08:52:14Z","updated_at":"2015-12-19T08:52:14Z","body":"I see, then maybe we should rename this method, and have another one with time name returning an offseted time to be inside the period ? "}}