{"action":"created","issue":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/177","labels_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/177/labels{/name}","comments_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/177/comments","events_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/177/events","html_url":"https://github.com/tomchadwin/qgis2web/pull/177","id":128346245,"number":177,"title":"Testing Refactoring","user":{"login":"akbargumbira","id":992519,"avatar_url":"https://avatars.githubusercontent.com/u/992519?v=3","gravatar_id":"","url":"https://api.github.com/users/akbargumbira","html_url":"https://github.com/akbargumbira","followers_url":"https://api.github.com/users/akbargumbira/followers","following_url":"https://api.github.com/users/akbargumbira/following{/other_user}","gists_url":"https://api.github.com/users/akbargumbira/gists{/gist_id}","starred_url":"https://api.github.com/users/akbargumbira/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/akbargumbira/subscriptions","organizations_url":"https://api.github.com/users/akbargumbira/orgs","repos_url":"https://api.github.com/users/akbargumbira/repos","events_url":"https://api.github.com/users/akbargumbira/events{/privacy}","received_events_url":"https://api.github.com/users/akbargumbira/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2016-01-23T19:04:32Z","updated_at":"2016-01-23T22:45:35Z","closed_at":"2016-01-23T21:58:31Z","pull_request":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/pulls/177","html_url":"https://github.com/tomchadwin/qgis2web/pull/177","diff_url":"https://github.com/tomchadwin/qgis2web/pull/177.diff","patch_url":"https://github.com/tomchadwin/qgis2web/pull/177.patch"},"body":"* Add test_data_path, load_layer, load_wfs_layer functions for testing\r\n* Organize test data better\r\n\r\n@tomchadwin: I added test_data_path, load_layer, and load_wfs_layer functions. The data path was hardcoded using the path in Travis, which is not good. Developers should be able to run the tests without problem on their local computer. Please, look at those 3 functions and see how to use them for future tests.\r\n\r\nI also moved the test data in to 3 subdirectories: control (for control files), layer, and style.\r\n\r\nThanks"},"comment":{"url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/comments/174228089","html_url":"https://github.com/tomchadwin/qgis2web/pull/177#issuecomment-174228089","issue_url":"https://api.github.com/repos/tomchadwin/qgis2web/issues/177","id":174228089,"user":{"login":"tomchadwin","id":7840213,"avatar_url":"https://avatars.githubusercontent.com/u/7840213?v=3","gravatar_id":"","url":"https://api.github.com/users/tomchadwin","html_url":"https://github.com/tomchadwin","followers_url":"https://api.github.com/users/tomchadwin/followers","following_url":"https://api.github.com/users/tomchadwin/following{/other_user}","gists_url":"https://api.github.com/users/tomchadwin/gists{/gist_id}","starred_url":"https://api.github.com/users/tomchadwin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomchadwin/subscriptions","organizations_url":"https://api.github.com/users/tomchadwin/orgs","repos_url":"https://api.github.com/users/tomchadwin/repos","events_url":"https://api.github.com/users/tomchadwin/events{/privacy}","received_events_url":"https://api.github.com/users/tomchadwin/received_events","type":"User","site_admin":false},"created_at":"2016-01-23T22:45:35Z","updated_at":"2016-01-23T22:45:35Z","body":"Comparing the full files rather than the checksums has one benefit. If the `assert` fails, test and control are both output. That makes it easy to debug. So, is there a clause in an `assert` which is only triggered on fail? If so, and we can print out test and control output in such a clause,  then using checksums is fine."}}