Project

General

Profile

Actions

Bug #12491

closed

TestRefinement#test_prepend_after_refine_wb_miss too slow

Added by zeha (Christian Hofstaedtler) almost 8 years ago. Updated over 7 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.3.1p112 (2016-04-26) [x86_64-linux-gnu]
[ruby-core:76031]

Description

In Debian we've recently enabled the full test suite (make test-all) on all architectures.
As part of that, we found that TestRefinement#test_prepend_after_refine_wb_miss times out on slower archs (think armhf, mips, ...).

  Error:
TestRefinement#test_prepend_after_refine_wb_miss:
Timeout::Error: execution of assert_normal_exit expired
pid 30320 exit 1
| -:38:in `block in <main>': NameError
| 	from -:3:in `times'
| 	from -:3:in `<main>'
| 

    /«PKGBUILDDIR»/test/ruby/test_refinement.rb:786:in `test_prepend_after_refine_wb_miss'

Please consider bumping the timeout and/or excluding the test?

Please also backport this to 2.3. Thanks!

Updated by shugo (Shugo Maeda) almost 8 years ago

  • Status changed from Open to Assigned
  • Assignee set to shugo (Shugo Maeda)

Christian Hofstaedtler wrote:

In Debian we've recently enabled the full test suite (make test-all) on all architectures.
As part of that, we found that TestRefinement#test_prepend_after_refine_wb_miss times out on slower archs (think armhf, mips, ...).
(snip)
Please consider bumping the timeout and/or excluding the test?

Does the following fix work?

diff --git a/test/ruby/test_refinement.rb b/test/ruby/test_refinement.rb
index 1984d2c..c9de8e8 100644
--- a/test/ruby/test_refinement.rb
+++ b/test/ruby/test_refinement.rb
@@ -785,6 +785,9 @@ class C
   eval PrependAfterRefine_CODE
 
   def test_prepend_after_refine_wb_miss
+    if /\A(arm|mips)/ =~ RUBY_PLATFORM
+      skip "too slow cpu"
+    end
     assert_normal_exit %Q{
       GC.stress = true
       10.times{
Actions #2

Updated by shugo (Shugo Maeda) over 7 years ago

  • Status changed from Assigned to Closed

Applied in changeset r55532.


  • test/ruby/test_refinement.rb: skip
    test_prepend_after_refine_wb_miss on ARM or MIPS.
    [ruby-core:76031] [Bug #12491]

Updated by shugo (Shugo Maeda) over 7 years ago

  • Backport changed from 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: REQUIRED to 2.1: DONTNEED, 2.2: DONTNEED, 2.3: REQUIRED

Updated by nagachika (Tomoyuki Chikanaga) over 7 years ago

  • Backport changed from 2.1: DONTNEED, 2.2: DONTNEED, 2.3: REQUIRED to 2.1: DONTNEED, 2.2: DONTNEED, 2.3: DONE

ruby_2_3 r55790 merged revision(s) 55532.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0