{"comment":{"url":"https://api.github.com/repos/caskdata/cdap/comments/13058300","html_url":"https://github.com/caskdata/cdap/commit/2ae4f12ebbd41afd8bafb33c56c570e307348244#commitcomment-13058300","id":13058300,"user":{"login":"rohitsinha54","id":1472867,"avatar_url":"https://avatars.githubusercontent.com/u/1472867?v=3","gravatar_id":"","url":"https://api.github.com/users/rohitsinha54","html_url":"https://github.com/rohitsinha54","followers_url":"https://api.github.com/users/rohitsinha54/followers","following_url":"https://api.github.com/users/rohitsinha54/following{/other_user}","gists_url":"https://api.github.com/users/rohitsinha54/gists{/gist_id}","starred_url":"https://api.github.com/users/rohitsinha54/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rohitsinha54/subscriptions","organizations_url":"https://api.github.com/users/rohitsinha54/orgs","repos_url":"https://api.github.com/users/rohitsinha54/repos","events_url":"https://api.github.com/users/rohitsinha54/events{/privacy}","received_events_url":"https://api.github.com/users/rohitsinha54/received_events","type":"User","site_admin":false},"position":252,"line":282,"path":"cdap-app-fabric/src/test/java/co/cask/cdap/internal/app/runtime/schedule/SchedulerServiceTest.java","commit_id":"2ae4f12ebbd41afd8bafb33c56c570e307348244","created_at":"2015-09-03T21:46:04Z","updated_at":"2015-09-03T21:46:04Z","body":"I will just get rid of System.currentTimeMillis() comparison as it looks like it will lead to issue. In this case of update test all we want to test is the schedule got updated and for that comparing with the old one is enough. "}}