mirror of
https://github.com/ruby/ruby.git
synced 2025-09-18 18:13:58 +02:00
merge revision(s) 674db715f5a141891a66e67004ca138696d3b0ad,1f1b9b0942ec12dde1af8000f8cb84692904fccc:
[ruby/rdoc] test/rdoc/test_rdoc_generator_json_index.rb: Use assert_equal instead of assert. It's better because assert_equal prints the values when it fails.91d40ce8f8
--- test/rdoc/test_rdoc_generator_json_index.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) [ruby/rdoc] test/rdoc/test_rdoc_generator_json_index.rb: pend in test_generate in ppc64le. We observed that this test randomly fails in the ruby/ruby Travis ppc64le case. This commit is to pend the test_generate if the assertion for the generated file's modified time fails in a ppc64le environment. Note that I didn't use the word "Travis CI" or Travis CI specific environment variables such as `TRAVIS` and `TRAVIS_CPU_ARCH`[1] in the code. Because I wanted to prioritize the rdoc's independence from the ruby/ruby. [1] https://docs.travis-ci.com/user/environment-variables/#default-environment-variables42cdad1cf2
--- test/rdoc/test_rdoc_generator_json_index.rb | 12 ++++++++++++ 1 file changed, 12 insertions(+)
This commit is contained in:
parent
5e9948336e
commit
d9f4f321c6
2 changed files with 14 additions and 2 deletions
|
@ -104,8 +104,20 @@ class TestRDocGeneratorJsonIndex < RDoc::TestCase
|
|||
orig_file = Pathname(File.join srcdir, 'generator/template/json_index/js/navigation.js')
|
||||
generated_file = Pathname(File.join @tmpdir, 'js/navigation.js')
|
||||
|
||||
# The following assertion for the generated file's modified time randomly
|
||||
# fails in a ppc64le environment.
|
||||
# https://github.com/ruby/rdoc/issues/1048
|
||||
if orig_file.mtime.inspect != generated_file.mtime.inspect &&
|
||||
RUBY_PLATFORM =~ /powerpc64le/
|
||||
pend <<~EOC
|
||||
Unstable test in ppc64le.
|
||||
<#{orig_file.mtime.inspect}> expected but was
|
||||
<#{generated_file.mtime.inspect}>.
|
||||
EOC
|
||||
end
|
||||
|
||||
# This is dirty hack on JRuby
|
||||
assert orig_file.mtime.inspect == generated_file.mtime.inspect,
|
||||
assert_equal orig_file.mtime.inspect, generated_file.mtime.inspect,
|
||||
'.js files should be the same timestamp of original'
|
||||
|
||||
json = File.read 'js/search_index.js'
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue