Bug 755787

Summary: RFE: rebase CD/DVD/Blu-ray tests
Product: [Retired] Red Hat Hardware Certification Program Reporter: Caspar Zhang <czhang>
Component: Test Suite (tests)Assignee: Caspar Zhang <czhang>
Status: CLOSED ERRATA QA Contact: Guangze Bai <gbai>
Severity: medium Docs Contact:
Priority: high    
Version: 1.4CC: gbai, gnichols, qcai, rlandry, yshao
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
No tech note needed.
 Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-05 21:36:48 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 767775, 773757    
Attachments:
Description Flags
01-opticalDiskTest.patch
none
02-cdrom.patch
none
03-dvd.patch
none
04-bluray.patch none

Description Caspar Zhang 2011-11-22 03:26:40 UTC 
Description of problem:

Current CD/DVD/Blu-ray test processes are different:

CD:
  InitializeDeviceInfo()->
  (old codes of confirmation) ->
  testDisk()

DVD:
  InitializeDeviceInfo()->
  getMediaChoice()->
  testDisk()

Bluray:
  InitializeDeviceInfo()->
  getMediaChoice()->
  echoTest()->
  checkMediaType()->
  testDisk()

Here we get some problems, e.g. in bluray test, the tray will be opened and closed twice, which is not necessary.

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

Additional info:
Comment 1 Caspar Zhang 2011-11-22 07:55:30 UTC 
Now I've composed a initial version of patch to update cdrom/dvd/bluray tests and opticalDiskTest.py

In my patch, I did the following changes:

1. move echoTest() and checkMediaType() to opticalDiskTest

  echoTest() is enabled in cdrom/dvd/bluray tests.

  echoTest() takes over the request of putting a disk into device; so I 
  removed the similar codes to ask user to insert a disk in testDisk() 
  function. 

  This change avoids opening the tray to ask user to put disk twice.

  checkMediaType() is enabled in bluray test and dvd+ part in dvd test.

  So I removed the check media part codes in rewriter() and writer() 
  functions in dvd test.

2. I make test sleep for 25 seconds after everytime tray closed. This solves bug 707144

3. I re-adjusted bluray test sequence to bd-re, bd-r, bd. This solves bug 754919

Patch is still under modification and testing. Will post it soon.
Comment 2 Caspar Zhang 2011-11-22 09:06:17 UTC 
Created attachment 534951 [details]
01-opticalDiskTest.patch
Comment 3 Caspar Zhang 2011-11-22 09:06:37 UTC 
Created attachment 534952 [details]
02-cdrom.patch
Comment 4 Caspar Zhang 2011-11-22 09:06:59 UTC 
Created attachment 534953 [details]
03-dvd.patch
Comment 5 Caspar Zhang 2011-11-22 09:07:22 UTC 
Created attachment 534954 [details]
04-bluray.patch
Comment 11 Caspar Zhang 2012-06-04 07:32:21 UTC 
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
No tech note needed.
Comment 13 errata-xmlrpc 2012-06-05 21:36:48 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2012-0711.html