{"pull_request":{"state":"open","deletions":116,"merged_by":null,"user":{"avatar_url":"https://secure.gravatar.com/avatar/443e2cbd92193bdd4b578cc9f26ce125?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"443e2cbd92193bdd4b578cc9f26ce125","url":"https://api.github.com/users/lnpat139","login":"lnpat139","id":1083005},"updated_at":"2011-11-10T03:22:14Z","commits":1,"base":{"ref":"master","repo":{"name":"expertiza","pushed_at":"2011-11-09T13:30:45Z","fork":false,"master_branch":null,"clone_url":"https://github.com/expertiza/expertiza.git","updated_at":"2011-11-10T02:50:07Z","open_issues":82,"homepage":"http://wikis.lib.ncsu.edu/index.php/Expertiza","forks":49,"private":false,"svn_url":"https://svn.github.com/expertiza/expertiza","language":"Ruby","url":"https://api.github.com/repos/expertiza/expertiza","owner":{"avatar_url":"https://secure.gravatar.com/avatar/d41d8cd98f00b204e9800998ecf8427e?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-orgs.png","gravatar_id":"d41d8cd98f00b204e9800998ecf8427e","url":"https://api.github.com/users/expertiza","login":"expertiza","id":427458},"html_url":"https://github.com/expertiza/expertiza","description":"Expertiza is a web application where students can submit and peer-review learning objects (articles, code, web sites, etc).","size":352,"id":962564,"watchers":50,"git_url":"git://github.com/expertiza/expertiza.git","created_at":"2010-10-05T03:37:29Z","ssh_url":"git@github.com:expertiza/expertiza.git"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/d41d8cd98f00b204e9800998ecf8427e?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-orgs.png","gravatar_id":"d41d8cd98f00b204e9800998ecf8427e","url":"https://api.github.com/users/expertiza","login":"expertiza","id":427458},"label":"expertiza:master","sha":"12e06db23af01fe72c8e7776e42e8203afe6e171"},"closed_at":null,"diff_url":"https://github.com/expertiza/expertiza/pull/142.diff","comments":0,"merged":false,"head":{"ref":"master","repo":{"name":"expertiza","pushed_at":"2011-11-10T03:10:55Z","fork":true,"master_branch":null,"clone_url":"https://github.com/lnpat139/expertiza.git","updated_at":"2011-11-10T03:10:57Z","open_issues":0,"homepage":"http://wikis.lib.ncsu.edu/index.php/Expertiza","forks":0,"private":false,"svn_url":"https://svn.github.com/lnpat139/expertiza","language":"Ruby","url":"https://api.github.com/repos/lnpat139/expertiza","owner":{"avatar_url":"https://secure.gravatar.com/avatar/443e2cbd92193bdd4b578cc9f26ce125?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"443e2cbd92193bdd4b578cc9f26ce125","url":"https://api.github.com/users/lnpat139","login":"lnpat139","id":1083005},"html_url":"https://github.com/lnpat139/expertiza","description":"Expertiza is a web application where students can submit and peer-review learning objects (articles, code, web sites, etc).","size":6356,"id":2746185,"watchers":1,"git_url":"git://github.com/lnpat139/expertiza.git","created_at":"2011-11-10T02:50:07Z","ssh_url":"git@github.com:lnpat139/expertiza.git"},"user":{"avatar_url":"https://secure.gravatar.com/avatar/443e2cbd92193bdd4b578cc9f26ce125?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png","gravatar_id":"443e2cbd92193bdd4b578cc9f26ce125","url":"https://api.github.com/users/lnpat139","login":"lnpat139","id":1083005},"label":"lnpat139:master","sha":"4ebeda0a74fb037bdb461a87b71d590c48b7b488"},"url":"https://api.github.com/repos/expertiza/expertiza/pulls/142","additions":1939,"merged_at":null,"body":"REFACTOR assignment_participant.rb\r\nThe following were the tasks performed as part of this project:\r\n1) In this project we have a defined a new module called \"Artifact\".\r\nThe Artifact module holds all information regarding submission and retrieval of files and hyperlinks\r\nThe Artifact module is located under the lib folder.\r\nThe Artifact module contains the following methods after refactoring:\r\na) submit_hyperlink()\r\nb) get_submitted_files()\r\nc) submitted_hyperlinks=()\r\nd) get_hyperlinks\r\ne) get_hyperlinks_array\r\nf) get_files()\r\n\r\n2) Refactor the methods get_scores() to get_scores_given() and get_topic_string() to get_topic_name(), in order tomaintain the code conventions and avoid discrepnancies.\r\n\r\n3) Modify the access of submitted_hyperlink() and submitted_file() methods to public and check if it causes any discrepnancies.\r\nTest cases:\r\nSince we defined a module and included this module in the assignment_participant we do not need to specify any new testcases , since thereare no new functionality is defined.\r\nHowever since the methods are now called in a different way now , it is required to ensure that all the testcases are passedfrom the model.\r\nFor e.g: The model assignment_participant.rb does not contain any methods pertaining to submit_hyperlink.\r\nHowever it is required that all the testcases pass, since the module Artifact is included in the model as a mixin.\r\nThis is checked.\r\nIn order to verify that the module is being mixed into the class we can remove the \"include Artifact\" statement from theasssignment_participant.rb class.\r\nIn this case, the following cases fail:\r\na) test_reject_empty_hyperlink\r\nb) test_reject_remove_hyperlink\r\nc) test_remove_one_hyperlink\r\nd) test_remove_second_hyperlink\r\ne) test_submit_first_hyperlink\r\nf) test_submit_third_hyperlink\r\n\r\nThe rest of the testcases i.e test_import,test_publishing_rights, test_publishing_rights_negative_tests and regenerate_keys must pass.\r\n\r\n","html_url":"https://github.com/expertiza/expertiza/pull/142","_links":{"comments":{"href":"https://api.github.com/repos/expertiza/expertiza/issues/142/comments"},"html":{"href":"https://github.com/expertiza/expertiza/pull/142"},"review_comments":{"href":"https://api.github.com/repos/expertiza/expertiza/pulls/142/comments"},"self":{"href":"https://api.github.com/repos/expertiza/expertiza/pulls/142"}},"review_comments":0,"issue_url":"https://github.com/expertiza/expertiza/issues/142","title":"E203 Refactor assignment_participant (abalasu3, lpatil, dnyacha)","number":142,"mergeable":null,"id":480788,"patch_url":"https://github.com/expertiza/expertiza/pull/142.patch","changed_files":10,"created_at":"2011-11-10T03:22:14Z"},"action":"opened","number":142}