bugGnash - The GNU Flash player - Bugs: bug #31594, gravity_embedded-TestRunner...

 
 

bug #31594: gravity_embedded-TestRunner testsuite test fail some times

Submitted by:  Petter Reinholdtsen <pere>
Submitted on:  Tue 09 Nov 2010 08:29:59 AM UTC  
 
Category: testsuiteSeverity: 3 - Normal
Release: masterStatus: Fixed
Privacy: PublicAssigned to: Sandro Santilli <strk>
Open/Closed: Closed

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 16 Nov 2010 07:43:51 PM UTC, comment #5:

Thanks for testing!

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 16 Nov 2010 07:24:08 PM UTC, comment #4:

I tested the patch with four builds in a row. all of them succeeded, while it used to fail every second time. I guess it work. :)

Petter Reinholdtsen <pere>
Project Member
Tue 16 Nov 2010 06:57:28 PM UTC, comment #3:

For the record:
http://git.savannah.gnu.org/cgit/gnash.git/commit/?id=22a70dc0672b85a0fb83d28e6cb2bd6db0252f96

/me wonders if there's an automatic link to there, somehow

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 16 Nov 2010 06:56:23 PM UTC, comment #2:

I've pushed the (non-definitive) improvement (commit 22a70dc)

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 16 Nov 2010 06:14:27 PM UTC, comment #1:

The test runner sleeps 5milliseconds and does a single advance, expecting the movie to be loaded by that time.
I've sent a patch to pere for testing. The patch makes 5 advances and 1ms sleep each.

Generally, to avoid timing issues, the runner should instead just
periodically check for the existance of a 'ball' children in the loaded-to container and proceed when that is available...

Sandro Santilli <strk>
Project MemberIn charge of this item.
Tue 09 Nov 2010 08:29:59 AM UTC, original submission:

The test suite fail some times with this error. I was told on IRC that this is because of race conditions in the thread handling of this test.

--=[ testsuite/movies.all ]=--
FAIL: gravity_embedded-TestRunner: tmp.strictly_equals(50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:79]
FAIL: gravity_embedded-TestRunner: loaded->getBounds().height() == '0' (expected: 2056) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:81]
FAIL: gravity_embedded-TestRunner: loaded->getBounds().width() == '0' (expected: 2056) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:82]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == '' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:90]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == '' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:98]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:100]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:102]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:104]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:106]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:108]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:110]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:112]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:114]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 44) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:116]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 44) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:118]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 44) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:120]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 44) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:126]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:128]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:130]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:132]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 46) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:134]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:136]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:138]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:140]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 48) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:142]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:144]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:146]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:148]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 50) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:150]
FAIL: gravity_embedded-TestRunner: string(text->get_text_value()) == 'NaN' (expected: 52) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:152]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 52) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:154]
FAIL: gravity_embedded-TestRunner: round(toNumber(tmp, vm)) == '100' (expected: 52) [/var/tmp/gnash/gnashgit/testsuite/movies.all/gravity_embedded-TestRunner.cpp:156]

make[5]: *** [anal] Error 1
make[5]: Leaving directory `/var/tmp/gnash/gnashgit/_build'
make[4]: *** [check-local] Error 2

Petter Reinholdtsen <pere>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by strk (Posted a comment)
  • -unavailable- added by pere (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 16 Nov 2010 07:43:51 PM UTCstrkStatusReady For Test=>Fixed
      Open/ClosedOpen=>Closed
    Tue 16 Nov 2010 06:56:23 PM UTCstrkStatusIn Progress=>Ready For Test
      Assigned toNone=>strk
    Tue 16 Nov 2010 06:14:27 PM UTCstrkStatusNone=>In Progress

    Back to the top


    Powered by Savane 3.1-cleanup1