Bug 812320 - when phase is not finished, although FAIL was reported, test ends as PASS
Summary: when phase is not finished, although FAIL was reported, test ends as PASS
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Fedora
Classification: Fedora
Component: beakerlib
Version: 16
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
Assignee: Petr Muller
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 545868
TreeView+ depends on / blocked
 
Reported: 2012-04-13 11:35 UTC by Jan Hutař
Modified: 2016-09-20 02:08 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-02-14 02:15:48 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Jan Hutař 2012-04-13 11:35:38 UTC
Description of problem:
When, because of test creator mistake, phase is not finished by rlPhaseEnd, although FAIL was reported earlier in the phase, test finishes as PASS.


Version-Release number of selected component (if applicable):
Current


How reproducible:
always


Steps to Reproduce:
1. Consider this (of-course, this is incorrect, but test should not PASS):
   for i in 1 2 3; do
     rlPhaseStartTest "test-$i"
     rlRun "fail"
     if fail:
       continue
     rlPhaseEnd
   done


Actual results:
[...]

::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: Satellite 5.4.1 i386 RHEL5 check
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

ls: Satellite-5.4.1-RHEL5-*: No such file or directory
:: [   FAIL   ] :: Not a directory: /i386/i386 

::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: Satellite 5.4.1 x86_64 RHEL5 check
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

[...]

::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::
:: [   LOG    ] :: Satellite 5.4.1 i386 RHEL5 check
::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::

:: [   FAIL   ] :: Not a directory: /i386/i386 
:: [   LOG    ] :: Duration: 18s
:: [   LOG    ] :: Assertions: 0 good, 1 bad
:: [unfinished] :: RESULT: Satellite 5.4.1 i386 RHEL5 check

[...]


Beaker WebUI or XML with results say "PASS" for the test.


Expected results:
Test should finish as FAIL as this is dangerous situation (gives false feeling all is OK, although there were FAIL). Even if no FAIL would be reported, IMO it is better to end with FAIL.

Comment 1 Petr Muller 2012-04-13 13:45:09 UTC
At first I wanted to immediately NOTABUG it, as I don't give much about the broken tests. But I guess it should not do any harm if we (in rlJournalEnd) checked if there's any unfinished phase, and issue a WARN about it. Tests not using rlJournalEnd are on their own, as there is no other place where such check could be placed.

Comment 2 Fedora End Of Life 2013-02-14 02:15:58 UTC
Fedora 16 changed to end-of-life (EOL) status on 2013-02-12. Fedora 16 is 
no longer maintained, which means that it will not receive any further 
security or bug fix updates. As a result we are closing this bug.

If you can reproduce this bug against a currently maintained version of 
Fedora please feel free to reopen this bug against that version.

Thank you for reporting this bug and we are sorry it could not be fixed.


Note You need to log in before you can comment on or make changes to this bug.