-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add rake task to compute a percentage of skippable tests for RSpec #194
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #194 +/- ##
==========================================
- Coverage 98.93% 98.92% -0.01%
==========================================
Files 262 263 +1
Lines 12440 12457 +17
Branches 560 561 +1
==========================================
+ Hits 12307 12323 +16
- Misses 133 134 +1 ☔ View full report in Codecov by Sentry. |
namespace :datadog do | ||
namespace :ci do | ||
namespace :rspec do | ||
task :skippable_percentage do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
task :skippable_percentage do | |
task skippable_percentage: :environment do |
end | ||
|
||
rspec_cli_options = %w[ | ||
--dry-run |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
--dry-run | |
--dry-run | |
--require | |
rails_helper |
(or spec_helper)
print((itr.skipped_tests_count.to_f / itr.total_tests_count).floor(2)) | ||
end | ||
|
||
task :skippable_percentage_estimate do |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
task :skippable_percentage_estimate do | |
task skippable_percentage_estimate: :environment do |
6b21d41
to
c050ca4
Compare
…y test optimisation
d411a96
to
4d8a92a
Compare
Datadog.logger.warn { "Intelligent test runner is not supported for forking test runners yet" } | ||
return | ||
end | ||
|
||
@mutex.synchronize do | ||
Telemetry.itr_skipped |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FIXME: this is a bug, need to be moved down to count skipped tests correctly
What does this PR do?
Motivation
Additional Notes
How to test the change?