Skip to content

Add junit formatter to the elixir tests#1888

Merged
garrensmith merged 2 commits intoapache:masterfrom
garrensmith:elixir-junit
Jan 31, 2019
Merged

Add junit formatter to the elixir tests#1888
garrensmith merged 2 commits intoapache:masterfrom
garrensmith:elixir-junit

Conversation

@garrensmith
Copy link
Copy Markdown
Member

Overview

Adds the junit formatter for the elixir tests so that Jenkins can read the test output nicely

Testing recommendations

run mix test and then look under _build/test/lib/foo/test-junit-report.xml for the xml output

Checklist

  • Code is written and works correctly;
  • Changes are covered by tests;
  • Documentation reflects the changes;

@eiri
Copy link
Copy Markdown
Member

eiri commented Jan 31, 2019

The check formatter (mix 1.8.0) fails on test_helper.exs changes, insist on config to look like

@@ -1,3 +1,7 @@
-ExUnit.configure(exclude: [pending: true], formatters: [JUnitFormatter, ExUnit.CLIFormatter])
+ExUnit.configure(
+  exclude: [pending: true],
+  formatters: [JUnitFormatter, ExUnit.CLIFormatter]
+)
+

Credo fails with [R] ↗ Variable names should be written in snake_case. for lib/couch.ex:58:5 #(Couch.process_url), sorry no idea how to fix this, so can't actually run the tests.

Add the junit formatter so that Jenkins can read the elixir tests
@garrensmith garrensmith merged commit f22e71a into apache:master Jan 31, 2019
@wohali
Copy link
Copy Markdown
Member

wohali commented Jan 31, 2019

@garrensmith This might be something we want to piece into the top-level Jenkinsfile, not sure.

@garrensmith
Copy link
Copy Markdown
Member Author

garrensmith commented Jan 31, 2019 via email

@garrensmith garrensmith deleted the elixir-junit branch December 12, 2019 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants