Skip to content

Add RSpec Queue Formatter to hide duplicated pending and failed tests in Queue Mode#31

Merged
ArturT merged 9 commits intomasterfrom
rspec-queue-formatter
Apr 23, 2017
Merged

Add RSpec Queue Formatter to hide duplicated pending and failed tests in Queue Mode#31
ArturT merged 9 commits intomasterfrom
rspec-queue-formatter

Conversation

@ArturT
Copy link
Copy Markdown
Member

@ArturT ArturT commented Apr 23, 2017

  • Add RSpec queue formatter and show summary of failed and pending tests at the end of queue mode run.
  • Hide pending and failed tests for intermediate tests runs after each fetched test files from Knapsack Pro API. In order to do that we monkey patch default rspec formatters.
  • Allow user to disable monkey patching for default rspec formatters with flag KNAPSACK_PRO_MODIFY_DEFAULT_RSPEC_FORMATTERS

Related to: rspec/rspec-core#2416

@ArturT ArturT changed the title Add RSpec Queue Formatter to hide duplicated pending and failed tests Add RSpec Queue Formatter to hide duplicated pending and failed tests in Queue Mode Apr 23, 2017

if KnapsackPro::Config::Env.modify_default_rspec_formatters?
class RSpec::Core::Formatters::BaseTextFormatter
prepend KnapsackPro::Formatters::RSpecHideFailuresAndPendingExtension
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ArturT ArturT merged commit 29286d2 into master Apr 23, 2017
@ArturT ArturT deleted the rspec-queue-formatter branch April 23, 2017 20:04
@toolmantim
Copy link
Copy Markdown

Thanks so much for this @ArturT! It's so much better.

One thing I'm still noticing is duplicate --profile output. Perhaps profile output should be disabled for queue mode?

@ArturT
Copy link
Copy Markdown
Member Author

ArturT commented May 1, 2017

@toolmantim I did fix for --profile and if you use profile formatter then you will see only single summary at the very end of the RSpec Queue Mode output.

Please update knapsack_pro to the newest version. ;)

Tech note:
Change in v0.34.0...v0.35.0

@toolmantim
Copy link
Copy Markdown

Amazing! ❤️

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.

2 participants