Bug 1514929 - Review Request: python-parameterized - Parameterized testing with any Python test framework
Summary: Review Request: python-parameterized - Parameterized testing with any Python ...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Neal Gompa
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1514289
TreeView+ depends on / blocked
 
Reported: 2017-11-19 14:19 UTC by Igor Gnatenko
Modified: 2017-11-20 16:56 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-11-20 16:56:41 UTC
Type: ---
Embargoed:
ngompa13: fedora-review+


Attachments (Terms of Use)

Description Igor Gnatenko 2017-11-19 14:19:19 UTC
Spec URL: https://ignatenkobrain.fedorapeople.org/for-review/python-parameterized.spec
SRPM URL: https://ignatenkobrain.fedorapeople.org/for-review/python-parameterized-0.6.1-1.fc28.src.rpm
Description: Parameterized testing with any Python test framework.
Fedora Account System Username: ignatenkobrain

Comment 1 Neal Gompa 2017-11-19 14:20:09 UTC
Taking this review.

Comment 2 Neal Gompa 2017-11-19 14:44:47 UTC
I get this error while attempting to build the package (while running the unit tests):

======================================================================
Traceback (most recent call last):
  File "/usr/bin/nosetests-2.7", line 11, in <module>
    load_entry_point('nose==1.3.7', 'console_scripts', 'nosetests')()
  File "/usr/lib/python2.7/site-packages/nose/core.py", line 121, in __init__
    **extra_args)
  File "/usr/lib64/python2.7/unittest/main.py", line 95, in __init__
    self.runTests()
  File "/usr/lib/python2.7/site-packages/nose/core.py", line 207, in runTests
    result = self.testRunner.run(self.test)
  File "/usr/lib/python2.7/site-packages/nose/core.py", line 66, in run
    result.printErrors()
  File "/usr/lib/python2.7/site-packages/nose/result.py", line 103, in printErrors
    _TextTestResult.printErrors(self)
  File "/usr/lib64/python2.7/unittest/runner.py", line 108, in printErrors
    self.printErrorList('ERROR', self.errors)
  File "/usr/lib64/python2.7/unittest/runner.py", line 114, in printErrorList
    self.stream.writeln("%s: %s" % (flavour,self.getDescription(test)))
  File "/usr/lib64/python2.7/unittest/runner.py", line 24, in writeln
    self.write(arg)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xf6' in position 8: ordinal not in range(128)

Comment 3 Igor Gnatenko 2017-11-19 14:48:59 UTC
try again, added export LC_ALL=C.UTF-8

Comment 4 Neal Gompa 2017-11-19 16:20:14 UTC
It's still failing the same way...

Comment 5 Igor Gnatenko 2017-11-19 17:41:37 UTC
Disabled tests for now..

Comment 6 Neal Gompa 2017-11-19 17:44:21 UTC
Review notes:

- Package is named properly
- Package follows Python packaging guidelines
- License is marked correctly
- No notable issues from fedora-review or rpmlint

PACKAGE APPROVED.

Comment 7 Gwyn Ciesla 2017-11-20 15:48:50 UTC
(fedrepo-req-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/python-parameterized


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