{"action":"created","issue":{"url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/1053","labels_url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/1053/labels{/name}","comments_url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/1053/comments","events_url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/1053/events","html_url":"https://github.com/jboss-reddeer/reddeer/pull/1053","id":71293454,"number":1053,"title":"ljelinko_Add possibility to execute javascript inside Browser.","user":{"login":"rhopp","id":1215011,"avatar_url":"https://avatars.githubusercontent.com/u/1215011?v=3","gravatar_id":"","url":"https://api.github.com/users/rhopp","html_url":"https://github.com/rhopp","followers_url":"https://api.github.com/users/rhopp/followers","following_url":"https://api.github.com/users/rhopp/following{/other_user}","gists_url":"https://api.github.com/users/rhopp/gists{/gist_id}","starred_url":"https://api.github.com/users/rhopp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhopp/subscriptions","organizations_url":"https://api.github.com/users/rhopp/orgs","repos_url":"https://api.github.com/users/rhopp/repos","events_url":"https://api.github.com/users/rhopp/events{/privacy}","received_events_url":"https://api.github.com/users/rhopp/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/jboss-reddeer/reddeer/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":{"login":"rhopp","id":1215011,"avatar_url":"https://avatars.githubusercontent.com/u/1215011?v=3","gravatar_id":"","url":"https://api.github.com/users/rhopp","html_url":"https://github.com/rhopp","followers_url":"https://api.github.com/users/rhopp/followers","following_url":"https://api.github.com/users/rhopp/following{/other_user}","gists_url":"https://api.github.com/users/rhopp/gists{/gist_id}","starred_url":"https://api.github.com/users/rhopp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rhopp/subscriptions","organizations_url":"https://api.github.com/users/rhopp/orgs","repos_url":"https://api.github.com/users/rhopp/repos","events_url":"https://api.github.com/users/rhopp/events{/privacy}","received_events_url":"https://api.github.com/users/rhopp/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/jboss-reddeer/reddeer/milestones/14","html_url":"https://github.com/jboss-reddeer/reddeer/milestones/0.8.0","labels_url":"https://api.github.com/repos/jboss-reddeer/reddeer/milestones/14/labels","id":1014551,"number":14,"title":"0.8.0","description":"","creator":{"login":"rawagner","id":2078045,"avatar_url":"https://avatars.githubusercontent.com/u/2078045?v=3","gravatar_id":"","url":"https://api.github.com/users/rawagner","html_url":"https://github.com/rawagner","followers_url":"https://api.github.com/users/rawagner/followers","following_url":"https://api.github.com/users/rawagner/following{/other_user}","gists_url":"https://api.github.com/users/rawagner/gists{/gist_id}","starred_url":"https://api.github.com/users/rawagner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rawagner/subscriptions","organizations_url":"https://api.github.com/users/rawagner/orgs","repos_url":"https://api.github.com/users/rawagner/repos","events_url":"https://api.github.com/users/rawagner/events{/privacy}","received_events_url":"https://api.github.com/users/rawagner/received_events","type":"User","site_admin":false},"open_issues":81,"closed_issues":55,"state":"open","created_at":"2015-03-10T11:38:26Z","updated_at":"2015-04-30T07:06:33Z","due_on":"2015-04-30T22:00:00Z","closed_at":null},"comments":2,"created_at":"2015-04-27T14:01:59Z","updated_at":"2015-04-30T10:55:25Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jboss-reddeer/reddeer/pulls/1053","html_url":"https://github.com/jboss-reddeer/reddeer/pull/1053","diff_url":"https://github.com/jboss-reddeer/reddeer/pull/1053.diff","patch_url":"https://github.com/jboss-reddeer/reddeer/pull/1053.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/comments/97737298","html_url":"https://github.com/jboss-reddeer/reddeer/pull/1053#issuecomment-97737298","issue_url":"https://api.github.com/repos/jboss-reddeer/reddeer/issues/1053","id":97737298,"user":{"login":"ljelinkova","id":2078072,"avatar_url":"https://avatars.githubusercontent.com/u/2078072?v=3","gravatar_id":"","url":"https://api.github.com/users/ljelinkova","html_url":"https://github.com/ljelinkova","followers_url":"https://api.github.com/users/ljelinkova/followers","following_url":"https://api.github.com/users/ljelinkova/following{/other_user}","gists_url":"https://api.github.com/users/ljelinkova/gists{/gist_id}","starred_url":"https://api.github.com/users/ljelinkova/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ljelinkova/subscriptions","organizations_url":"https://api.github.com/users/ljelinkova/orgs","repos_url":"https://api.github.com/users/ljelinkova/repos","events_url":"https://api.github.com/users/ljelinkova/events{/privacy}","received_events_url":"https://api.github.com/users/ljelinkova/received_events","type":"User","site_admin":false},"created_at":"2015-04-30T10:55:25Z","updated_at":"2015-04-30T10:55:25Z","body":"* please remove System.out.println from BrowserTest. \r\n* in log messages, please use present form (evaluate) not continuous form (evaluating)\r\n* why do you catch SWTException in AbstractBrowser.evaluate/execute methods? Display should wrap the exception into CoreException so you should never see SWTException\r\n* why do you return null from evaluate method (this is different in SWT implementation where an exception is thrown). How would you differenciate if the code failed or returned null?\r\n* why do you need wait conditions in the test? If you need to wait for the script to execute, you should put wait into AbstractBrowser"}}