Fix parsing of author in payload send by GitHub

Signed-off-by: Simon Rozet <simon@rozet.name>

[integrity#81 state:resolved]

lib/integrity/project.rb

@@ -119,7 +119,7 @@ def head_of_remote_repo
 
       def create_commit_from(data)
         commits.create(:identifier   => data["id"],
-                       :author       => data["author"],
+                       :author       => "#{data["author"]["name"]} <#{data["author"]["email"]}>",
                        :message      => data["message"],
                        :committed_at => data["timestamp"])
       end
@@ -139,4 +139,4 @@ def delete_code
         Integrity.log "Problem while trying to deleting code: #{error}"
       end
   end
-end
+end

test/acceptance/api_test.rb

@@ -7,6 +7,15 @@ class ApiTest < Test::Unit::AcceptanceTestCase
     So that my project is built everytime I push to the Holy Hub
   EOF
 
+  it "parses payload author correctly" do
+    repo = git_repo(:my_test_project) # initial commit && successful commit
+    Project.gen(:my_test_project, :uri => repo.path)
+    basic_auth "admin", "test"
+    post "/my-test-project/push", :payload => payload(repo.head, "master", repo.commits)
+    visit "/my-test-project"
+    response_body.should have_tag(".who", "by: John Doe")
+  end
+
   scenario "it only build commits for the branch being monitored" do
     repo = git_repo(:my_test_project) # initial commit && successful commit
     Project.gen(:my_test_project, :uri => repo.path, :branch => "my-branch")

test/helpers/acceptance/git_helper.rb

@@ -43,7 +43,7 @@ def commits
       Dir.chdir(@path) do
         commits = `git log --pretty=oneline`.collect { |line| line.split(" ").first }
         commits.inject([]) do |commits, sha1|
-          format  = %Q(---%n:message: >-%n  %s%n:timestamp: %ci%n:id: %H%n:author: %an <%ae>)
+          format  = %Q(---%n:message: >-%n  %s%n:timestamp: %ci%n:id: %H%n:author: %n :name: %an%n :email: %ae%n)
           commits << YAML.load(`git show -s --pretty=format:"#{format}" #{sha1}`)
         end
       end