Bug 1102854 - runTest (tests.test_parted_parted.GetLabelsTestCase) fails
Summary: runTest (tests.test_parted_parted.GetLabelsTestCase) fails
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: pyparted
Version: rawhide
Hardware: aarch64
OS: Linux
unspecified
unspecified
Target Milestone: ---
Assignee: Dave Cantrell
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-05-29 17:11 UTC by D. Marlin
Modified: 2014-06-12 13:53 UTC (History)
2 users (show)

Fixed In Version: pyparted-3.9.3-3.fc21
Clone Of:
Environment:
Last Closed: 2014-06-12 13:53:43 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
patch to fix failing testcase GetLabelsTestCase (1.77 KB, patch)
2014-05-29 17:12 UTC, D. Marlin
no flags Details | Diff

Description D. Marlin 2014-05-29 17:11:22 UTC
Description of problem:

GetLabelsTestCase fails when building pyparted on aarch64.  AArch64 uses GPT partitions, but apparently pyparted does not yet recognize this for AArch64.


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

pyparted-3.9.3-1.fc21


How reproducible:

consistently


Steps to Reproduce:
1.  Example koji build: 
  http://arm.koji.fedoraproject.org/koji/taskinfo?taskID=2354502


Actual results:

FAIL: runTest (tests.test_parted_parted.GetLabelsTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/builddir/build/BUILD/pyparted-3.9.3/tests/test_parted_parted.py", line 50, in runTest
    self.assertGreater(len(parted.getLabels()), 0)
AssertionError: 0 not greater than 0


Expected results:

test passes


Additional info:

The attached patch adds AArch64 support for GPT and fixes the test failure.

Comment 1 D. Marlin 2014-05-29 17:12:27 UTC
Created attachment 900443 [details]
patch to fix failing testcase GetLabelsTestCase

Comment 2 Dave Cantrell 2014-06-12 13:32:13 UTC
Resolving this today.  Have a backlog of pyparted fixes like this that I was waiting on until after 7.0 GA.  Will get this pushed out quickly.


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