{"action":"created","comment":{"url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/comments/40162749","id":40162749,"diff_hunk":"@@ -55,35 +55,78 @@ def check_entry(entry, expected_name, expected_number, expected_email)\n     it 'imports the 1st entry' do\n       book.import_from_csv('entries.csv')\n       entry_one = book.entries[0]\n-      check_entry(entry_one, 'Bill', '555-555-4854', 'bill@blocmail.com')\n+      check_entry(entry_one, 'Bill',\n+                             '555-555-4854',\n+                             'bill@blocmail.com')\n     end\n \n     it 'imports the 2nd entry' do\n       book.import_from_csv('entries.csv')\n       # Check the second entry\n       entry_two = book.entries[1]\n-      check_entry(entry_two, 'Bob', '555-555-5415', 'bob@blocmail.com')\n+      check_entry(entry_two, 'Bob',\n+                             '555-555-5415',\n+                             'bob@blocmail.com')\n     end\n \n     it 'imports the 3rd entry' do\n       book.import_from_csv('entries.csv')\n       # Check the third entry\n       entry_three = book.entries[2]\n-      check_entry(entry_three, 'Joe', '555-555-3660', 'joe@blocmail.com')\n+      check_entry(entry_three, 'Joe',\n+                               '555-555-3660',\n+                               'joe@blocmail.com')\n     end\n \n     it 'imports the 4th entry' do\n       book.import_from_csv('entries.csv')\n       # Check the fourth entry\n       entry_four = book.entries[3]\n-      check_entry(entry_four, 'Sally', '555-555-4646', 'sally@blocmail.com')\n+      check_entry(entry_four, 'Sally',\n+                              '555-555-4646',\n+                              'sally@blocmail.com')\n     end\n \n     it 'imports the 5th entry' do\n       book.import_from_csv('entries.csv')\n       # Check the fifth entry\n       entry_five = book.entries[4]\n-      check_entry(entry_five, 'Sussie', '555-555-2036', 'sussie@blocmail.com')\n+      check_entry(entry_five, 'Sussie',\n+                              '555-555-2036',\n+                              'sussie@blocmail.com')\n+    end\n+  end\n+\n+  context 'Import from entries_2.csv' do\n+    it 'imports the correct number of entries from entries_2.csv' do\n+      book.import_from_csv('entries_2.csv')\n+      book_size = book.entries.size\n+\n+      expect(book_size).to eql 3\n+    end\n+\n+    it 'imports the 1st entry from csv #2' do\n+      book.import_from_csv('entries_2.csv')\n+      entry_one = book.entries[0]\n+      check_entry(entry_one, 'Gary',\n+                             '000-000-0000',\n+                             'gdawg@busey2016.com')\n+    end\n+\n+    it 'imports the 2nd entry from csv #2' do\n+      book.import_from_csv('entries_2.csv')\n+      entry_two = book.entries[1]\n+      check_entry(entry_two, 'Nate',\n+                             '555-1-ALLTHENUMBERS',\n+                             'nate@fictional.email')","path":"spec/address_book_spec.rb","position":72,"original_position":72,"commit_id":"7581cc761da185fa0905216803dd758b1ff90d9e","original_commit_id":"7581cc761da185fa0905216803dd758b1ff90d9e","user":{"login":"houndci","id":6697940,"avatar_url":"https://avatars.githubusercontent.com/u/6697940?v=3","gravatar_id":"","url":"https://api.github.com/users/houndci","html_url":"https://github.com/houndci","followers_url":"https://api.github.com/users/houndci/followers","following_url":"https://api.github.com/users/houndci/following{/other_user}","gists_url":"https://api.github.com/users/houndci/gists{/gist_id}","starred_url":"https://api.github.com/users/houndci/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/houndci/subscriptions","organizations_url":"https://api.github.com/users/houndci/orgs","repos_url":"https://api.github.com/users/houndci/repos","events_url":"https://api.github.com/users/houndci/events{/privacy}","received_events_url":"https://api.github.com/users/houndci/received_events","type":"User","site_admin":false},"body":"Align the parameters of a method call if they span more than one line.<br>Prefer double-quoted strings unless you need single quotes to avoid extra backslashes for escaping.","created_at":"2015-09-23T00:52:56Z","updated_at":"2015-09-23T00:52:56Z","html_url":"https://github.com/npauzenga/address-bloc/pull/3#discussion_r40162749","pull_request_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3","_links":{"self":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/comments/40162749"},"html":{"href":"https://github.com/npauzenga/address-bloc/pull/3#discussion_r40162749"},"pull_request":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3"}}},"pull_request":{"url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3","id":45734732,"html_url":"https://github.com/npauzenga/address-bloc/pull/3","diff_url":"https://github.com/npauzenga/address-bloc/pull/3.diff","patch_url":"https://github.com/npauzenga/address-bloc/pull/3.patch","issue_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/3","number":3,"state":"open","locked":false,"title":"Assignment 23 reading csv","user":{"login":"npauzenga","id":5838826,"avatar_url":"https://avatars.githubusercontent.com/u/5838826?v=3","gravatar_id":"","url":"https://api.github.com/users/npauzenga","html_url":"https://github.com/npauzenga","followers_url":"https://api.github.com/users/npauzenga/followers","following_url":"https://api.github.com/users/npauzenga/following{/other_user}","gists_url":"https://api.github.com/users/npauzenga/gists{/gist_id}","starred_url":"https://api.github.com/users/npauzenga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/npauzenga/subscriptions","organizations_url":"https://api.github.com/users/npauzenga/orgs","repos_url":"https://api.github.com/users/npauzenga/repos","events_url":"https://api.github.com/users/npauzenga/events{/privacy}","received_events_url":"https://api.github.com/users/npauzenga/received_events","type":"User","site_admin":false},"body":"I created a new context for importing from the second CSV. Not sure if that's preferable or if it should be all lumped into the first `#import_from_csv` context. Seemed to make sense because I had to re-check the size of `@entries` according to the new CSV size (3) and it doesn't seem tidy to lump that into the previous context.\r\n\r\nAlso, does the sorting happen at the `parse` method? I was confused why the order in my .CSV file was different than the order of `@entries`.","created_at":"2015-09-23T00:52:49Z","updated_at":"2015-09-23T00:52:56Z","closed_at":null,"merged_at":null,"merge_commit_sha":"b9c1327c48b9c617d3688c1a2c4defa034412dc8","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3/commits","review_comments_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3/comments","review_comment_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls/comments{/number}","comments_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/3/comments","statuses_url":"https://api.github.com/repos/npauzenga/address-bloc/statuses/7581cc761da185fa0905216803dd758b1ff90d9e","head":{"label":"npauzenga:assignment-23-ReadingCSV","ref":"assignment-23-ReadingCSV","sha":"7581cc761da185fa0905216803dd758b1ff90d9e","user":{"login":"npauzenga","id":5838826,"avatar_url":"https://avatars.githubusercontent.com/u/5838826?v=3","gravatar_id":"","url":"https://api.github.com/users/npauzenga","html_url":"https://github.com/npauzenga","followers_url":"https://api.github.com/users/npauzenga/followers","following_url":"https://api.github.com/users/npauzenga/following{/other_user}","gists_url":"https://api.github.com/users/npauzenga/gists{/gist_id}","starred_url":"https://api.github.com/users/npauzenga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/npauzenga/subscriptions","organizations_url":"https://api.github.com/users/npauzenga/orgs","repos_url":"https://api.github.com/users/npauzenga/repos","events_url":"https://api.github.com/users/npauzenga/events{/privacy}","received_events_url":"https://api.github.com/users/npauzenga/received_events","type":"User","site_admin":false},"repo":{"id":42790147,"name":"address-bloc","full_name":"npauzenga/address-bloc","owner":{"login":"npauzenga","id":5838826,"avatar_url":"https://avatars.githubusercontent.com/u/5838826?v=3","gravatar_id":"","url":"https://api.github.com/users/npauzenga","html_url":"https://github.com/npauzenga","followers_url":"https://api.github.com/users/npauzenga/followers","following_url":"https://api.github.com/users/npauzenga/following{/other_user}","gists_url":"https://api.github.com/users/npauzenga/gists{/gist_id}","starred_url":"https://api.github.com/users/npauzenga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/npauzenga/subscriptions","organizations_url":"https://api.github.com/users/npauzenga/orgs","repos_url":"https://api.github.com/users/npauzenga/repos","events_url":"https://api.github.com/users/npauzenga/events{/privacy}","received_events_url":"https://api.github.com/users/npauzenga/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/npauzenga/address-bloc","description":"","fork":false,"url":"https://api.github.com/repos/npauzenga/address-bloc","forks_url":"https://api.github.com/repos/npauzenga/address-bloc/forks","keys_url":"https://api.github.com/repos/npauzenga/address-bloc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/npauzenga/address-bloc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/npauzenga/address-bloc/teams","hooks_url":"https://api.github.com/repos/npauzenga/address-bloc/hooks","issue_events_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/events{/number}","events_url":"https://api.github.com/repos/npauzenga/address-bloc/events","assignees_url":"https://api.github.com/repos/npauzenga/address-bloc/assignees{/user}","branches_url":"https://api.github.com/repos/npauzenga/address-bloc/branches{/branch}","tags_url":"https://api.github.com/repos/npauzenga/address-bloc/tags","blobs_url":"https://api.github.com/repos/npauzenga/address-bloc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/npauzenga/address-bloc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/npauzenga/address-bloc/git/refs{/sha}","trees_url":"https://api.github.com/repos/npauzenga/address-bloc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/npauzenga/address-bloc/statuses/{sha}","languages_url":"https://api.github.com/repos/npauzenga/address-bloc/languages","stargazers_url":"https://api.github.com/repos/npauzenga/address-bloc/stargazers","contributors_url":"https://api.github.com/repos/npauzenga/address-bloc/contributors","subscribers_url":"https://api.github.com/repos/npauzenga/address-bloc/subscribers","subscription_url":"https://api.github.com/repos/npauzenga/address-bloc/subscription","commits_url":"https://api.github.com/repos/npauzenga/address-bloc/commits{/sha}","git_commits_url":"https://api.github.com/repos/npauzenga/address-bloc/git/commits{/sha}","comments_url":"https://api.github.com/repos/npauzenga/address-bloc/comments{/number}","issue_comment_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/comments{/number}","contents_url":"https://api.github.com/repos/npauzenga/address-bloc/contents/{+path}","compare_url":"https://api.github.com/repos/npauzenga/address-bloc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/npauzenga/address-bloc/merges","archive_url":"https://api.github.com/repos/npauzenga/address-bloc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/npauzenga/address-bloc/downloads","issues_url":"https://api.github.com/repos/npauzenga/address-bloc/issues{/number}","pulls_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls{/number}","milestones_url":"https://api.github.com/repos/npauzenga/address-bloc/milestones{/number}","notifications_url":"https://api.github.com/repos/npauzenga/address-bloc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/npauzenga/address-bloc/labels{/name}","releases_url":"https://api.github.com/repos/npauzenga/address-bloc/releases{/id}","created_at":"2015-09-19T21:24:48Z","updated_at":"2015-09-19T21:33:54Z","pushed_at":"2015-09-23T00:52:50Z","git_url":"git://github.com/npauzenga/address-bloc.git","ssh_url":"git@github.com:npauzenga/address-bloc.git","clone_url":"https://github.com/npauzenga/address-bloc.git","svn_url":"https://github.com/npauzenga/address-bloc","homepage":null,"size":0,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":3,"forks":0,"open_issues":3,"watchers":0,"default_branch":"master"}},"base":{"label":"npauzenga:master","ref":"master","sha":"9c074b849c2b8fd4922fa2ac539bf77b8ddb2968","user":{"login":"npauzenga","id":5838826,"avatar_url":"https://avatars.githubusercontent.com/u/5838826?v=3","gravatar_id":"","url":"https://api.github.com/users/npauzenga","html_url":"https://github.com/npauzenga","followers_url":"https://api.github.com/users/npauzenga/followers","following_url":"https://api.github.com/users/npauzenga/following{/other_user}","gists_url":"https://api.github.com/users/npauzenga/gists{/gist_id}","starred_url":"https://api.github.com/users/npauzenga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/npauzenga/subscriptions","organizations_url":"https://api.github.com/users/npauzenga/orgs","repos_url":"https://api.github.com/users/npauzenga/repos","events_url":"https://api.github.com/users/npauzenga/events{/privacy}","received_events_url":"https://api.github.com/users/npauzenga/received_events","type":"User","site_admin":false},"repo":{"id":42790147,"name":"address-bloc","full_name":"npauzenga/address-bloc","owner":{"login":"npauzenga","id":5838826,"avatar_url":"https://avatars.githubusercontent.com/u/5838826?v=3","gravatar_id":"","url":"https://api.github.com/users/npauzenga","html_url":"https://github.com/npauzenga","followers_url":"https://api.github.com/users/npauzenga/followers","following_url":"https://api.github.com/users/npauzenga/following{/other_user}","gists_url":"https://api.github.com/users/npauzenga/gists{/gist_id}","starred_url":"https://api.github.com/users/npauzenga/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/npauzenga/subscriptions","organizations_url":"https://api.github.com/users/npauzenga/orgs","repos_url":"https://api.github.com/users/npauzenga/repos","events_url":"https://api.github.com/users/npauzenga/events{/privacy}","received_events_url":"https://api.github.com/users/npauzenga/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/npauzenga/address-bloc","description":"","fork":false,"url":"https://api.github.com/repos/npauzenga/address-bloc","forks_url":"https://api.github.com/repos/npauzenga/address-bloc/forks","keys_url":"https://api.github.com/repos/npauzenga/address-bloc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/npauzenga/address-bloc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/npauzenga/address-bloc/teams","hooks_url":"https://api.github.com/repos/npauzenga/address-bloc/hooks","issue_events_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/events{/number}","events_url":"https://api.github.com/repos/npauzenga/address-bloc/events","assignees_url":"https://api.github.com/repos/npauzenga/address-bloc/assignees{/user}","branches_url":"https://api.github.com/repos/npauzenga/address-bloc/branches{/branch}","tags_url":"https://api.github.com/repos/npauzenga/address-bloc/tags","blobs_url":"https://api.github.com/repos/npauzenga/address-bloc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/npauzenga/address-bloc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/npauzenga/address-bloc/git/refs{/sha}","trees_url":"https://api.github.com/repos/npauzenga/address-bloc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/npauzenga/address-bloc/statuses/{sha}","languages_url":"https://api.github.com/repos/npauzenga/address-bloc/languages","stargazers_url":"https://api.github.com/repos/npauzenga/address-bloc/stargazers","contributors_url":"https://api.github.com/repos/npauzenga/address-bloc/contributors","subscribers_url":"https://api.github.com/repos/npauzenga/address-bloc/subscribers","subscription_url":"https://api.github.com/repos/npauzenga/address-bloc/subscription","commits_url":"https://api.github.com/repos/npauzenga/address-bloc/commits{/sha}","git_commits_url":"https://api.github.com/repos/npauzenga/address-bloc/git/commits{/sha}","comments_url":"https://api.github.com/repos/npauzenga/address-bloc/comments{/number}","issue_comment_url":"https://api.github.com/repos/npauzenga/address-bloc/issues/comments{/number}","contents_url":"https://api.github.com/repos/npauzenga/address-bloc/contents/{+path}","compare_url":"https://api.github.com/repos/npauzenga/address-bloc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/npauzenga/address-bloc/merges","archive_url":"https://api.github.com/repos/npauzenga/address-bloc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/npauzenga/address-bloc/downloads","issues_url":"https://api.github.com/repos/npauzenga/address-bloc/issues{/number}","pulls_url":"https://api.github.com/repos/npauzenga/address-bloc/pulls{/number}","milestones_url":"https://api.github.com/repos/npauzenga/address-bloc/milestones{/number}","notifications_url":"https://api.github.com/repos/npauzenga/address-bloc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/npauzenga/address-bloc/labels{/name}","releases_url":"https://api.github.com/repos/npauzenga/address-bloc/releases{/id}","created_at":"2015-09-19T21:24:48Z","updated_at":"2015-09-19T21:33:54Z","pushed_at":"2015-09-23T00:52:50Z","git_url":"git://github.com/npauzenga/address-bloc.git","ssh_url":"git@github.com:npauzenga/address-bloc.git","clone_url":"https://github.com/npauzenga/address-bloc.git","svn_url":"https://github.com/npauzenga/address-bloc","homepage":null,"size":0,"stargazers_count":0,"watchers_count":0,"language":"Ruby","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":3,"forks":0,"open_issues":3,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3"},"html":{"href":"https://github.com/npauzenga/address-bloc/pull/3"},"issue":{"href":"https://api.github.com/repos/npauzenga/address-bloc/issues/3"},"comments":{"href":"https://api.github.com/repos/npauzenga/address-bloc/issues/3/comments"},"review_comments":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3/comments"},"review_comment":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/npauzenga/address-bloc/pulls/3/commits"},"statuses":{"href":"https://api.github.com/repos/npauzenga/address-bloc/statuses/7581cc761da185fa0905216803dd758b1ff90d9e"}}}}