Fix Error reporting and parsing (fixes publiclab#1852) (publiclab#1857)

* Fix Error reporting and parsing (fixes publiclab#1852)

* Remove fraudlent failure

* Update .travis.yml

.travis.yml

@@ -19,8 +19,6 @@ install:
 
 script:
   - docker-compose run -e CI=TRUE web bash -c "GENERATE_REPORT=true rake test:all"
-  - mkdir -p ./test/reports
-  - touch ./test/reports/TEST_.xml
   - echo -e '<?xml version="1.0" encoding="UTF-8"?>' > output.xml
   - tail -n +2 -q ./test/reports/TEST*.xml >> output.xml
   - 'if [ "$TRAVIS_PULL_REQUEST" != "false" ]; then danger --verbose; fi'

test/test_helper.rb

@@ -8,15 +8,16 @@
 require 'i18n'
 
 require "minitest/reporters"
-Minitest::Reporters.use!
+MiniTest::Reporters.use! [MiniTest::Reporters::DefaultReporter.new,
+                          MiniTest::Reporters::JUnitReporter.new]
 
 class ActiveSupport::TestCase
   # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order.
   #
   # Note: You'll currently still have to declare fixtures explicitly in integration tests
   # -- they do not yet inherit this setting
 
-  # These can be replaced in Rails 5 with this syntax, in the fixture files: 
+  # These can be replaced in Rails 5 with this syntax, in the fixture files:
   # https://github.com/rails/rails/commit/2acec4657752d441ab87b9f5862d7918843d6409#diff-1ed2907b3b8f148c2533558a77673ffaR3
   # _fixture:
   #   model_class: 'DrupalUser'