danger-xcode_summary 0.5.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: e222df89970744f0a9e0e9ec76408480aaa6020c
-  data.tar.gz: f07bbe188cf33ef8cd5707742bbd4d68b965da8f
+SHA256:
+  metadata.gz: e30986734f1f4563b6c7f58e5765cb2b00be9bf4cc86ae03c88485d429009e3e
+  data.tar.gz: 570891d22c3fbb5a9b2cdbb61dab1833ac8570013e0c2689886b78702cbff368
 SHA512:
-  metadata.gz: fb7531300b28f2307a50ce66562c28e3bec695d6e70223220f811b822b5522137e5022677359f92509a7fa651f0fd9beaa3a6d084733df7e76016ab5ef5e8d0d
-  data.tar.gz: 70d9df94ff7603adb1b340650cb5b4f6b350c228797aa5b89ea790b99de1c5792439ed0ac5eaf7663c94ec45f9261865c827828a58150e27437856a463971d11
+  metadata.gz: dac17c30dbeaf27f6b026c82f213a8d6ede70623c05bd811ab352957dde61b9a7c402ebca595146b161b8a82c3405e7cc93002c950363aa4262bee9d17267f76
+  data.tar.gz: 32db5f742803b7cc51d46be5825a644be65d40c89b790db6cfad3e15312b099f58912b7febb542767ee0dd8ef63fd703a059ae20c2b80ee1765bdea8dbdbf05b

data/.gitattributes

@@ -0,0 +1 @@
+*.xcresult binary

data/.rubocop.yml

@@ -1,11 +1,11 @@
+AllCops:
+  NewCops: enable
+  SuggestExtensions: false
+
 # kind_of? is a good way to check a type
 Style/ClassCheck:
   EnforcedStyle: kind_of?
 
-# It's better to be more explicit about the type
-Style/BracesAroundHashParameters:
-  Enabled: false
-
 # specs sometimes have useless assignments, which is fine
 Lint/UselessAssignment:
   Exclude:
@@ -54,8 +54,8 @@ Metrics/CyclomaticComplexity:
   Max: 17
 
 # Configuration parameters: AllowURI, URISchemes.
-Metrics/LineLength:
-  Max: 120
+Layout/LineLength:
+  Max: 130
 
 # Configuration parameters: CountKeywordArgs.
 Metrics/ParameterLists:

data/.travis.yml

@@ -3,9 +3,17 @@ cache:
   directories:
     - bundle
 
+os: osx
+osx_image: xcode11
+
 rvm:
-  - 2.1.3
-  - 2.3.1
+  - 2.6.0
+  - 2.5.3
+
+# install bundler 2.x
+before_install:
+    - gem update --system
+    - gem install bundler
 
 script:
-    - bundle exec rake spec
+    - bundle exec rake spec

data/Gemfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in danger-xcode_summary.gemspec

data/Gemfile.lock

@@ -1,47 +1,61 @@
 PATH
   remote: .
   specs:
-    danger-xcode_summary (0.5.0)
+    danger-xcode_summary (1.0.0)
       danger-plugin-api (~> 1.0)
+      xcresult (~> 0.2)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.5.2)
-      public_suffix (>= 2.0.2, < 4.0)
-    ast (2.4.0)
-    claide (1.0.2)
+    addressable (2.7.0)
+      public_suffix (>= 2.0.2, < 5.0)
+    ast (2.4.2)
+    claide (1.0.3)
     claide-plugins (0.9.2)
       cork
       nap
       open4 (~> 1.3)
-    coderay (1.1.2)
+    coderay (1.1.3)
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (5.5.10)
+    danger (8.2.3)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
-      faraday (~> 0.9)
-      faraday-http-cache (~> 1.0)
-      git (~> 1)
-      kramdown (~> 1.5)
+      faraday (>= 0.9.0, < 2.0)
+      faraday-http-cache (~> 2.0)
+      git (~> 1.7)
+      kramdown (~> 2.3)
+      kramdown-parser-gfm (~> 1.0)
       no_proxy_fix
       octokit (~> 4.7)
-      terminal-table (~> 1)
+      terminal-table (>= 1, < 4)
     danger-plugin-api (1.0.0)
       danger (> 2.0)
-    diff-lcs (1.3)
-    faraday (0.14.0)
+    diff-lcs (1.4.4)
+    faraday (1.4.2)
+      faraday-em_http (~> 1.0)
+      faraday-em_synchrony (~> 1.0)
+      faraday-excon (~> 1.1)
+      faraday-net_http (~> 1.0)
+      faraday-net_http_persistent (~> 1.1)
       multipart-post (>= 1.2, < 3)
-    faraday-http-cache (1.3.1)
-      faraday (~> 0.8)
-    ffi (1.9.23)
+      ruby2_keywords (>= 0.0.4)
+    faraday-em_http (1.0.0)
+    faraday-em_synchrony (1.0.0)
+    faraday-excon (1.1.0)
+    faraday-http-cache (2.2.0)
+      faraday (>= 0.8)
+    faraday-net_http (1.0.1)
+    faraday-net_http_persistent (1.1.0)
+    ffi (1.15.1)
     formatador (0.2.5)
-    git (1.3.0)
-    guard (2.14.2)
+    git (1.8.1)
+      rchardet (~> 1.8)
+    guard (2.17.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -55,81 +69,93 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    kramdown (1.16.2)
+    kramdown (2.3.1)
+      rexml
+    kramdown-parser-gfm (1.1.0)
+      kramdown (~> 2.0)
     listen (3.0.7)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9.7)
-    lumberjack (1.0.12)
-    method_source (0.9.0)
-    multipart-post (2.0.0)
+    lumberjack (1.2.8)
+    method_source (1.0.0)
+    multipart-post (2.1.1)
     nap (1.1.0)
     nenv (0.3.0)
     no_proxy_fix (0.1.2)
-    notiffany (0.1.1)
+    notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (4.8.0)
+    octokit (4.21.0)
+      faraday (>= 0.9)
       sawyer (~> 0.8.0, >= 0.5.3)
     open4 (1.3.4)
-    parallel (1.12.1)
-    parser (2.5.0.2)
-      ast (~> 2.4.0)
-    powerpack (0.1.1)
-    pry (0.11.3)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    public_suffix (3.0.2)
+    parallel (1.20.1)
+    parser (3.0.1.1)
+      ast (~> 2.4.1)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (4.0.6)
     rainbow (3.0.0)
-    rake (10.5.0)
-    rb-fsevent (0.10.3)
-    rb-inotify (0.9.10)
-      ffi (>= 0.5.0, < 2)
-    rspec (3.7.0)
-      rspec-core (~> 3.7.0)
-      rspec-expectations (~> 3.7.0)
-      rspec-mocks (~> 3.7.0)
-    rspec-core (3.7.1)
-      rspec-support (~> 3.7.0)
-    rspec-expectations (3.7.0)
+    rake (13.0.3)
+    rb-fsevent (0.11.0)
+    rb-inotify (0.10.1)
+      ffi (~> 1.0)
+    rchardet (1.8.0)
+    regexp_parser (2.1.1)
+    rexml (3.2.5)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.7.0)
-    rspec-mocks (3.7.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.7.0)
-    rspec-support (3.7.1)
-    rubocop (0.52.1)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
+    rubocop (1.15.0)
       parallel (~> 1.10)
-      parser (>= 2.4.0.2, < 3.0)
-      powerpack (~> 0.1)
+      parser (>= 3.0.0.0)
       rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml
+      rubocop-ast (>= 1.5.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.9.0)
-    sawyer (0.8.1)
-      addressable (>= 2.3.5, < 2.6)
-      faraday (~> 0.8, < 1.0)
+      unicode-display_width (>= 1.4.0, < 3.0)
+    rubocop-ast (1.5.0)
+      parser (>= 3.0.1.1)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.4)
+    sawyer (0.8.2)
+      addressable (>= 2.3.5)
+      faraday (> 0.8, < 2.0)
     shellany (0.0.1)
-    terminal-table (1.8.0)
-      unicode-display_width (~> 1.1, >= 1.1.1)
-    thor (0.20.0)
-    unicode-display_width (1.3.0)
-    yard (0.9.12)
+    terminal-table (3.0.1)
+      unicode-display_width (>= 1.1.1, < 3)
+    thor (1.1.0)
+    unicode-display_width (2.0.0)
+    xcresult (0.2.1)
+    yard (0.9.26)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  bundler (~> 1.3)
+  bundler (>= 2.2.10)
   danger
   danger-xcode_summary!
   guard (~> 2.14)
   guard-rspec (~> 4.7)
   listen (= 3.0.7)
   pry
-  rake (~> 10.0)
+  rake (~> 13.0)
   rspec (~> 3.4)
   rubocop
   yard
 
 BUNDLED WITH
-   1.16.1
+   2.2.26

data/Guardfile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # A guardfile for making Danger Plugins
 # For more info see https://github.com/guard/guard#readme
 

data/README.md

@@ -3,14 +3,9 @@
 [![License](http://img.shields.io/badge/license-MIT-green.svg?style=flat)](LICENSE.txt)
 [![Gem Version](https://badge.fury.io/rb/danger-xcode_summary.svg)](https://badge.fury.io/rb/danger-xcode_summary)
 [![Build Status](https://travis-ci.org/diogot/danger-xcode_summary.svg?branch=master)](https://travis-ci.org/diogot/danger-xcode_summary)
-[![Dependency Status](https://dependencyci.com/github/diogot/danger-xcode_summary/badge)](https://dependencyci.com/github/diogot/danger-xcode_summary)
 
 A [Danger](http://danger.systems) plugin that shows all build errors, warnings and unit tests results generated from `xcodebuild`.
 
-You need to use [xcpretty](https://github.com/supermarin/xcpretty) with 
-[xcpretty-json-formatter](https://github.com/marcelofabri/xcpretty-json-formatter) 
-to generate a JSON file that this plugin can read.
-
 ## How does it look?
 
 <table>
@@ -61,24 +56,6 @@ to generate a JSON file that this plugin can read.
   </tbody>
 </table>
 
-<table>
-  <thead>
-    <tr>
-      <th width="50"></th>
-      <th width="100%">
-          1 Message
-      </th>
-     </tr>
-  </thead>
-  <tbody>
-    <tr>
-      <td><g-emoji alias="book" fallback-src="https://assets-cdn.github.com/images/icons/emoji/unicode/1f4d6.png">📖</g-emoji></td>
-      <td>Executed 5 tests, with 1 failure (0 unexpected) in 0.032 (0.065) seconds</td>
-    </tr>
-      </tr>
-  </tbody>
-</table>
-
 ## Installation
 
 Add this line to your Gemfile:
@@ -92,9 +69,13 @@ gem 'danger-xcode_summary'
 Just add this line to your `Dangerfile`:
 
 ```ruby
-xcode_summary.report 'xcodebuild.json'
+xcode_summary.report 'MyApp.xcresult'
 ```
 
+You need to pass the path of the `xcresult` generated after compiling your app.
+By default, this is inside the `DerivedData` for your project, but you can use the `-resultBundlePath`
+flag when calling `xcodebuild` to customize its path. You can read more about it in this [blog post from the folks at PSPDFKit](https://pspdfkit.com/blog/2021/deflaking-ci-tests-with-xcresults/#using-xcresult-bundles).
+
 You can also ignore warnings from certain files by setting `ignored_files`: 
 Warning: `ignored_files` patterns applied on relative paths.  
 
@@ -104,10 +85,10 @@ xcode_summary.ignored_files = 'Pods/**'
 
 # Ignoring specific warnings
 xcode_summary.ignored_results { |result|
-  result.message.start_with 'ld' # Ignore ld_warnings
+  result.message.include? 'ld' # Ignore ld_warnings
 }
 
-xcode_summary.report 'xcodebuild.json'
+xcode_summary.report 'MyApp.xcresult'
 ```
 
 You can use `ignores_warnings` to supress warnings and shows only errors.
@@ -122,13 +103,13 @@ When this value is enabled, each warnings and errors are commented on each lines
 ```ruby
 # Comment on each lines
 xcode_summary.inline_mode = true
-xcode_summary.report 'xcodebuild.json'
+xcode_summary.report 'MyApp.xcresult'
 ```
 
-You can get warning and error number by calling `warning_error_count`. The return will be a JSON string contains warning and error count, e.g {"warnings":1,"errors":3}:
+You can get warning and error number by calling `warning_error_count`. The return will be a JSON string contains warning and error count, e.g `{"warnings":1,"errors":3}`:
 
 ```ruby
-result = xcode_summary.warning_error_count 'xcodebuild.json'
+result = xcode_summary.warning_error_count 'MyApp.xcresult'
 ```
 
 ## License

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 require 'rspec/core/rake_task'
 require 'rubocop/rake_task'

data/danger-xcode_summary.gemspec

@@ -1,6 +1,6 @@
-# coding: utf-8
+# frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
 require 'xcode_summary/gem_version.rb'
@@ -21,11 +21,14 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
+  spec.required_ruby_version = ">= 2.4.0"
+
+  spec.add_dependency 'xcresult', '~> 0.2'
   spec.add_runtime_dependency 'danger-plugin-api', '~> 1.0'
 
   # General ruby development
-  spec.add_development_dependency 'bundler', '~> 1.3'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'bundler', '>= 2.2.10'
+  spec.add_development_dependency 'rake', '~> 13.0'
 
   #  Testing support
   spec.add_development_dependency 'rspec', '~> 3.4'

data/lib/danger_plugin.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require 'xcode_summary/plugin'

data/lib/danger_xcode_summary.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 require 'xcode_summary/gem_version'

data/lib/xcode_summary/gem_version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module XcodeSummary
-  VERSION = '0.5.0'.freeze
+  VERSION = '1.0.0'
 end

data/lib/xcode_summary/plugin.rb

@@ -1,18 +1,19 @@
+# frozen_string_literal: true
+
 require 'json'
+require 'xcresult'
 
 module Danger
   # Shows all build errors, warnings and unit tests results generated from `xcodebuild`.
-  # You need to use [xcpretty](https://github.com/supermarin/xcpretty)
-  # with [xcpretty-json-formatter](https://github.com/marcelofabri/xcpretty-json-formatter)
-  # to generate a JSON file that this plugin can read.
+  # You need to use the `xcresult` produced by Xcode 11. It's located in the Derived Data folder.
   # @example Showing summary
   #
-  #          xcode_summary.report 'xcodebuild.json'
+  #          xcode_summary.report 'build.xcresult'
   #
   # @example Filtering warnings in Pods
   #
   #          xcode_summary.ignored_files = '**/Pods/**'
-  #          xcode_summary.report 'xcodebuild.json'
+  #          xcode_summary.report 'build.xcresult'
   #
   # @see  diogot/danger-xcode_summary
   # @tags xcode, xcodebuild, format
@@ -42,18 +43,6 @@ module Danger
     # @return   [Block]
     attr_accessor :ignored_results
 
-    # Defines if the test summary will be sticky or not.
-    # Defaults to `false`.
-    # @param    [Boolean] value
-    # @return   [Boolean]
-    attr_accessor :sticky_summary
-
-    # Defines if the build summary is shown or not.
-    # Defaults to `true`.
-    # @param    [Boolean] value
-    # @return   [Boolean]
-    attr_accessor :test_summary
-
     # Defines if using inline comment or not.
     # Defaults to `false`.
     # @param    [Boolean] value
@@ -81,14 +70,6 @@ module Danger
       @ignored_results ||= block
     end
 
-    def sticky_summary
-      @sticky_summary || false
-    end
-
-    def test_summary
-      @test_summary.nil? ? true : @test_summary
-    end
-
     def inline_mode
       @inline_mode || false
     end
@@ -96,30 +77,41 @@ module Danger
     def ignores_warnings
       @ignores_warnings || false
     end
+
+    # Pick a Dangerfile plugin for a chosen request_source and cache it
+    # based on https://github.com/danger/danger/blob/master/lib/danger/plugin_support/plugin.rb#L31
+    def plugin
+      plugins = Plugin.all_plugins.select { |plugin| Dangerfile.essential_plugin_classes.include? plugin }
+      @plugin ||= plugins.select { |p| p.method_defined? :html_link }.map { |p| p.new(@dangerfile) }.compact.first
+    end
     # rubocop:enable Lint/DuplicateMethods
 
-    # Reads a file with JSON Xcode summary and reports it.
+    # Reads a `.xcresult` and reports it.
     #
-    # @param    [String] file_path Path for Xcode summary in JSON format.
+    # @param    [String] file_path Path for xcresult bundle.
     # @return   [void]
     def report(file_path)
-      if File.file?(file_path)
-        xcode_summary = JSON.parse(File.read(file_path), symbolize_names: true)
+      if File.exist?(file_path)
+        xcode_summary = XCResult::Parser.new(path: file_path)
         format_summary(xcode_summary)
       else
         fail 'summary file not found'
       end
     end
 
-    # Reads a file with JSON Xcode summary and reports its warning and error count.
+    # Reads a `.xcresult` and reports its warning and error count.
     #
-    # @param    [String] file_path Path for Xcode summary in JSON format.
+    # @param    [String] file_path Path for xcresult bundle.
     # @return   [String] JSON string with warningCount and errorCount
     def warning_error_count(file_path)
-      if File.file?(file_path)
-        xcode_summary = JSON.parse(File.read(file_path), symbolize_names: true)
-        warning_count = warnings(xcode_summary).count
-        error_count = errors(xcode_summary).count
+      if File.exist?(file_path)
+        xcode_summary = XCResult::Parser.new(path: file_path)
+        warning_count = 0
+        error_count = 0
+        xcode_summary.actions_invocation_record.actions.each do |action|
+          warning_count += warnings(action).count
+          error_count += errors(action).count
+        end
         result = { warnings: warning_count, errors: error_count }
         result.to_json
       else
@@ -130,111 +122,90 @@ module Danger
     private
 
     def format_summary(xcode_summary)
-      messages(xcode_summary).each { |s| message(s, sticky: sticky_summary) }
-      warnings(xcode_summary).each do |result|
-        if inline_mode && result.location
-          warn(result.message, sticky: false, file: result.location.file_name, line: result.location.line)
-        else
-          warn(result.message, sticky: false)
+      xcode_summary.actions_invocation_record.actions.each do |action|
+        warnings(action).each do |result|
+          if inline_mode && result.location
+            warn(result.message, sticky: false, file: result.location.file_path, line: result.location.line)
+          else
+            warn(result.message, sticky: false)
+          end
         end
-      end
-      errors(xcode_summary).each do |result|
-        if inline_mode && result.location
-          fail(result.message, sticky: false, file: result.location.file_name, line: result.location.line)
-        else
-          fail(result.message, sticky: false)
+        # rubocop:disable Lint/UnreachableLoop
+        errors(action).each do |result|
+          if inline_mode && result.location
+            fail(result.message, sticky: false, file: result.location.file_path, line: result.location.line)
+          else
+            fail(result.message, sticky: false)
+          end
         end
+        # rubocop:enable Lint/UnreachableLoop
       end
     end
 
-    def messages(xcode_summary)
-      if test_summary
-        [
-          xcode_summary[:tests_summary_messages]
-        ].flatten.uniq.compact.map(&:strip)
-      else
-        []
-      end
-    end
-
-    def warnings(xcode_summary)
-      if ignores_warnings
-        return []
-      end
+    def warnings(action)
+      return [] if ignores_warnings
 
       warnings = [
-        xcode_summary.fetch(:warnings, []).map { |message| Result.new(message, nil) },
-        xcode_summary.fetch(:ld_warnings, []).map { |message| Result.new(message, nil) },
-        xcode_summary.fetch(:compile_warnings, {}).map do |h|
-          Result.new(format_compile_warning(h), parse_location(h))
-        end
-      ].flatten.uniq.compact.reject { |result| result.message.nil? }
+        action.action_result.issues.warning_summaries,
+        action.build_result.issues.warning_summaries
+      ].flatten.compact.map do |summary|
+        result = Result.new(summary.message, parse_location(summary.document_location_in_creating_workspace))
+        Result.new(format_warning(result), result.location)
+      end
+      warnings = warnings.uniq.reject { |result| result.message.nil? }
       warnings.delete_if(&ignored_results)
     end
 
-    def errors(xcode_summary)
+    def errors(action)
       errors = [
-        xcode_summary.fetch(:errors, []).map { |message| Result.new(message, nil) },
-        xcode_summary.fetch(:compile_errors, {}).map do |h|
-          Result.new(format_compile_warning(h), parse_location(h))
-        end,
-        xcode_summary.fetch(:file_missing_errors, {}).map do |h|
-          Result.new(format_format_file_missing_error(h), parse_location(h))
-        end,
-        xcode_summary.fetch(:undefined_symbols_errors, {}).map do |h|
-          Result.new(format_undefined_symbols(h), nil)
-        end,
-        xcode_summary.fetch(:duplicate_symbols_errors, {}).map do |h|
-          Result.new(format_duplicate_symbols(h), nil)
-        end,
-        xcode_summary.fetch(:tests_failures, {}).map do |test_suite, failures|
-          failures.map do |failure|
-            Result.new(format_test_failure(test_suite, failure), parse_test_location(failure))
-          end
-        end
-      ].flatten.uniq.compact.reject { |result| result.message.nil? }
-      errors.delete_if(&ignored_results)
-    end
+        action.action_result.issues.error_summaries,
+        action.build_result.issues.error_summaries
+      ].flatten.compact.map do |summary|
+        result = Result.new(summary.message, parse_location(summary.document_location_in_creating_workspace))
+        Result.new(format_warning(result), result.location)
+      end
 
-    def parse_location(h)
-      file_path, line, _column = h[:file_path].split(':')
-      Location.new(h[:file_name], file_path, line.to_i)
-    end
+      test_failures = [
+        action.action_result.issues.test_failure_summaries,
+        action.build_result.issues.test_failure_summaries
+      ].flatten.compact.map do |summary|
+        result = Result.new(summary.message, parse_location(summary.document_location_in_creating_workspace))
+        Result.new(format_test_failure(result, summary.producing_target, summary.test_case_name),
+                   result.location)
+      end
 
-    def parse_test_location(failure)
-      path, line = failure[:file_path].split(':')
-      file_name = relative_path(path)
-      Location.new(file_name, path, line.to_i)
+      results = (errors + test_failures).uniq.reject { |result| result.message.nil? }
+      results.delete_if(&ignored_results)
     end
 
-    def format_path(path)
-      clean_path, line = parse_filename(path)
-      path = clean_path + '#L' + line if clean_path && line
-
-      # Pick a Dangerfile plugin for a chosen request_source
-      # based on https://github.com/danger/danger/blob/master/lib/danger/plugin_support/plugin.rb#L31
-      plugins = Plugin.all_plugins.select { |plugin| Dangerfile.essential_plugin_classes.include? plugin }
-      plugin = plugins.select { |p| p.method_defined? :html_link }.map { |p| p.new(@dangerfile) }.compact.first
+    def parse_location(document_location)
+      return nil if document_location.nil?
 
-      plugin.html_link(path)
+      file_path = document_location.url.gsub('file://', '').split('#').first
+      file_name = file_path.split('/').last
+      fragment = document_location.url.split('#').last
+      params = CGI.parse(fragment).transform_values(&:first)
+      line = params['StartingLineNumber'].to_i + 1 # StartingLineNumber is 0-based, but we need a 1-based value
+      Location.new(file_name, relative_path(file_path), line)
     end
 
-    def parse_filename(path)
-      regex = /^(.*?):(\d*):?\d*$/
-      match = path.match(regex)
-      if match
-        match.captures
+    def format_path(file_path, line)
+      if plugin
+        path = file_path
+        path += "#L#{line}" if line
+        plugin.html_link(path)
+      else
+        file_path
       end
     end
 
     def relative_path(path)
       return nil if project_root.nil?
+
       path.gsub(project_root, '')
     end
 
     def should_ignore_warning?(path)
-      parsed = parse_filename(path)
-      path = parsed.first || path
       ignored_files.any? { |pattern| File.fnmatch(pattern, path) }
     end
 
@@ -242,44 +213,22 @@ module Danger
       reason.gsub('>', '\>').gsub('<', '\<')
     end
 
-    def format_compile_warning(h)
-      path = relative_path(h[:file_path])
-      return nil if should_ignore_warning?(path)
-
-      path_link = format_path(path)
+    def format_warning(result)
+      return escape_reason(result.message) if result.location.nil?
 
-      warning = "**#{path_link}**: #{escape_reason(h[:reason])}  <br />"
-      if h[:line] && !h[:line].empty?
-        "#{warning}" \
-          "```\n" \
-          "#{h[:line]}\n" \
-          '```'
-      else
-        warning
-      end
-    end
-
-    def format_format_file_missing_error(h)
-      path = relative_path(h[:file_path])
-      path_link = format_path(path)
-      "**#{escape_reason(h[:reason])}**: #{path_link}"
-    end
+      path = result.location.file_path
+      return nil if should_ignore_warning?(path)
 
-    def format_undefined_symbols(h)
-      "#{h[:message]}  <br />" \
-        "> Symbol: #{h[:symbol]}  <br />" \
-        "> Referenced from: #{h[:reference]}"
-    end
+      path_link = format_path(path, result.location.line)
 
-    def format_duplicate_symbols(h)
-      "#{h[:message]}  <br />" \
-        "> #{h[:file_paths].map { |path| path.split('/').last }.join('<br /> ')}"
+      "**#{path_link}**: #{escape_reason(result.message)}"
     end
 
-    def format_test_failure(suite_name, failure)
-      path = relative_path(failure[:file_path])
-      path_link = format_path(path)
-      "**#{suite_name}**: #{failure[:test_case]}, #{escape_reason(failure[:reason])}  <br />  #{path_link}"
+    def format_test_failure(result, producing_target, test_case_name)
+      path = result.location.file_path
+      path_link = format_path(path, result.location.line)
+      suite_name = "#{producing_target}.#{test_case_name}"
+      "**#{suite_name}**: #{escape_reason(result.message)}  <br />  #{path_link}"
     end
   end
 end