From 7d80d0ed37664e88c49e6b4ea699dc2198e6330b Mon Sep 17 00:00:00 2001 From: Michael Lange Date: Tue, 20 Jun 2023 13:50:33 -0700 Subject: [PATCH] Rip out the xUnit test reporter This was used to integrate with Circle CI's deeper test reporting (failures, flakes, reporting). It's strictly vestigial now that we're on GHA. --- .github/workflows/_ember-exam.yml | 5 +---- ui/testem.js | 26 -------------------------- 2 files changed, 1 insertion(+), 30 deletions(-) diff --git a/.github/workflows/_ember-exam.yml b/.github/workflows/_ember-exam.yml index 437930dce..2012cbd32 100644 --- a/.github/workflows/_ember-exam.yml +++ b/.github/workflows/_ember-exam.yml @@ -21,7 +21,4 @@ jobs: - name: Ember tests env: PERCY_TOKEN: ${{ secrets.PERCY_TOKEN }} - run: mkdir -p /tmp/test-reports && cd ui && yarn exam:parallel -- --split=${{ inputs.split }} --partition=${{ inputs.partition }} - - uses: actions/upload-artifact@0b7f8abb1508181956e8e162db84b466c27e18ce # v3.1.2 - with: - path: "/tmp/test-reports" + run: cd ui && yarn exam:parallel -- --split=${{ inputs.split }} --partition=${{ inputs.partition }} diff --git a/ui/testem.js b/ui/testem.js index 9f5394f28..3365884eb 100644 --- a/ui/testem.js +++ b/ui/testem.js @@ -5,11 +5,6 @@ 'use strict'; -const MultiReporter = require('testem-multi-reporter'); -const TapReporter = require('testem/lib/reporters/tap_reporter'); -const XunitReporter = require('testem/lib/reporters/xunit_reporter'); -const fs = require('fs'); - const config = { test_page: 'tests/index.html?hidepassed', disable_watching: true, @@ -37,25 +32,4 @@ const config = { }, }; -if (process.env.CI) { - const reporters = [ - { - ReporterClass: TapReporter, - args: [false, null, { get: () => false }], - }, - { - ReporterClass: XunitReporter, - args: [ - false, - fs.createWriteStream('/tmp/test-reports/ui.xml'), - { get: () => false }, - ], - }, - ]; - - const multiReporter = new MultiReporter({ reporters }); - - config.reporter = multiReporter; -} - module.exports = config;