{"action":"created","issue":{"url":"https://api.github.com/repos/IQSS/dataverse/issues/4896","repository_url":"https://api.github.com/repos/IQSS/dataverse","labels_url":"https://api.github.com/repos/IQSS/dataverse/issues/4896/labels{/name}","comments_url":"https://api.github.com/repos/IQSS/dataverse/issues/4896/comments","events_url":"https://api.github.com/repos/IQSS/dataverse/issues/4896/events","html_url":"https://github.com/IQSS/dataverse/issues/4896","id":344881284,"node_id":"MDU6SXNzdWUzNDQ4ODEyODQ=","number":4896,"title":"Classifying junit tests for testing versatility","user":{"login":"matthew-a-dunlap","id":8901015,"node_id":"MDQ6VXNlcjg5MDEwMTU=","avatar_url":"https://avatars3.githubusercontent.com/u/8901015?v=4","gravatar_id":"","url":"https://api.github.com/users/matthew-a-dunlap","html_url":"https://github.com/matthew-a-dunlap","followers_url":"https://api.github.com/users/matthew-a-dunlap/followers","following_url":"https://api.github.com/users/matthew-a-dunlap/following{/other_user}","gists_url":"https://api.github.com/users/matthew-a-dunlap/gists{/gist_id}","starred_url":"https://api.github.com/users/matthew-a-dunlap/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthew-a-dunlap/subscriptions","organizations_url":"https://api.github.com/users/matthew-a-dunlap/orgs","repos_url":"https://api.github.com/users/matthew-a-dunlap/repos","events_url":"https://api.github.com/users/matthew-a-dunlap/events{/privacy}","received_events_url":"https://api.github.com/users/matthew-a-dunlap/received_events","type":"User","site_admin":false},"labels":[{"id":435707518,"node_id":"MDU6TGFiZWw0MzU3MDc1MTg=","url":"https://api.github.com/repos/IQSS/dataverse/labels/Status:%20Code%20Review","name":"Status: Code Review","color":"e6e6e6","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":16,"created_at":"2018-07-26T14:58:34Z","updated_at":"2018-08-20T19:03:04Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"There is a need to organize our tests such that only some are run during development while others are run by Travis. Some tests can be pretty slow and running the full suite every time can slow down dev work.\r\n\r\nLooking into this a bit, there are at least two JUnit options to choose from: @suite and @category annotations . It may be best to choose the option that does not require annotating every test class, but maybe only annotating the slow ones? \r\n\r\nMore investigation is needed but either way it shouldn't be more work than adding some annotations and doing some minor edits to the build process and travis process"},"comment":{"url":"https://api.github.com/repos/IQSS/dataverse/issues/comments/414427047","html_url":"https://github.com/IQSS/dataverse/issues/4896#issuecomment-414427047","issue_url":"https://api.github.com/repos/IQSS/dataverse/issues/4896","id":414427047,"node_id":"MDEyOklzc3VlQ29tbWVudDQxNDQyNzA0Nw==","user":{"login":"pdurbin","id":21006,"node_id":"MDQ6VXNlcjIxMDA2","avatar_url":"https://avatars2.githubusercontent.com/u/21006?v=4","gravatar_id":"","url":"https://api.github.com/users/pdurbin","html_url":"https://github.com/pdurbin","followers_url":"https://api.github.com/users/pdurbin/followers","following_url":"https://api.github.com/users/pdurbin/following{/other_user}","gists_url":"https://api.github.com/users/pdurbin/gists{/gist_id}","starred_url":"https://api.github.com/users/pdurbin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pdurbin/subscriptions","organizations_url":"https://api.github.com/users/pdurbin/orgs","repos_url":"https://api.github.com/users/pdurbin/repos","events_url":"https://api.github.com/users/pdurbin/events{/privacy}","received_events_url":"https://api.github.com/users/pdurbin/received_events","type":"User","site_admin":false},"created_at":"2018-08-20T19:03:04Z","updated_at":"2018-08-20T19:03:04Z","author_association":"MEMBER","body":"@benjamin-martinez do we really need a git submodule for this? Here's a screenshot:\r\n\r\n![screen shot 2018-08-20 at 2 59 55 pm](https://user-images.githubusercontent.com/21006/44360727-be00d300-a489-11e8-8de9-0bf7b3c9db3b.png)\r\n\r\nAlso, over 100 files were changed but it seems like mostly whitespace was changed. I'm sort of wondering if we can make a cleaned up version of your pull request.\r\n\r\n@pameyer did you catch that \"IT\" (integration tests) are enumerated in src/test/java/edu/harvard/iq/dataverse/NonEssentialTestSuite.java ? It makes we wonder about how we already enumerate them in `conf/docker-aio/run-test-suite.sh`.\r\n\r\nAnd speaking of the API test suite, I saw a note it the pull request that is was failing. @benjamin-martinez is it still failing?"}}
{"actor":{"display_login":"pdurbin"}}