kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Merge pull request #42785 from akshaymohite/smart-parallel-tests-patch

Browse source 
Used already defined method parallelized? and fixed semantics of test name.
This commit is contained in:
Eileen M. Uchitelle 2021-07-15 13:09:42 -04:00 committed by GitHub
commit c2669b85ba
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
activesupport/lib/active_support/testing/parallelize_executor.rb
View file

@ -61,7 +61,7 @@ module ActiveSupport
end
def execution_info
if should_parallelize?
if parallelized?
"Running #{tests_count} tests in parallel using #{parallel_executor.size} #{parallelize_with}"
else
"Running #{tests_count} tests in a single process (parallelization threshold is #{ActiveSupport.test_parallelization_minimum_number_of_tests})"

2
railties/test/application/test_runner_test.rb
View file

@ -571,7 +571,7 @@ module ApplicationTests
assert_no_match "create_table(:users)", output
end
def test_avoid_paralleling_when_number_of_tests_if_below_threshold
def test_avoid_parallelizing_when_number_of_tests_is_below_threshold
exercise_parallelization_regardless_of_machine_core_count(with: :processes, threshold: 100)
file_name = create_parallel_processes_test_file