Bug 247268

Summary: Review Request: python-proctor - Proctor is a tool for running unit tests
Product: [Fedora] Fedora Reporter: Anderson Silva <ansilva>
Component: Package ReviewAssignee: Nobody's working on this, feel free to take it <nobody>
Status: CLOSED CANTFIX QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: fedora-package-review, notting
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-08-13 15:43:00 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Anderson Silva 2007-07-06 14:33:47 UTC
Spec URL: http://www.the-silvas.com/python-proctor.spec
SRPM URL: http://www.the-silvas.com/python-proctor-1.2-2.src.rpm
Description: Proctor is a tool for running unit tests. It enhances the standard unittest module to provide the ability to find all tests in a set of code, categorize them via tagging, and run some or all of them. Test output may be generated in a variety of formats to support parsing by another tool or simple, nicely formatted, reports for human review.

Comment 1 Anderson Silva 2007-07-16 17:40:08 UTC
Just to show the output of rpmlint on this package:

[ansilva@dhcp227-94 noarch]$ rpmlint python-proctor-1.2-2.noarch.rpm 
[ansilva@dhcp227-94 noarch]$ 


Comment 2 Steve Milner 2007-08-02 01:21:22 UTC
This person is in need of a sponsor.

Comment 3 Steve Milner 2007-08-06 23:54:05 UTC
* Package passes rpmlint
[steve@psycho Desktop]$ rpmlint
/home/steve/rpmbuild/RPMS/noarch/python-proctor-1.2-2.fc7.noarch.rpm
[steve@psycho Desktop]$ 
* Package passes Package Naming Guidelines
* %{name} is the same as spec file
* Meets the Packaging Guidelines
* Licensed under BSD. NOTE: license is not in upsteam source. Please notify the
owner and request it be added.
* The License field in the package spec file matches the actual license.
* spec file is written in American English.
* spec is legible
* Upstream matches whats in srpm

[steve@psycho Desktop]$ md5sum Proctor-1.2.tar.gz*
cee63679980ea25816511bb1b8be0d7f  Proctor-1.2.tar.gz
cee63679980ea25816511bb1b8be0d7f  Proctor-1.2.tar.gz-orig
[steve@psycho Desktop]$ 

* Package compiles in noarch
* BuildRequires looks sane. 
* Build doesn't have duplicate files listed
* %files look sane 
* %clean is sane
* macros used throughout spec
* package is python libraries which is code
* Package does not require docs to run
X Unclear if gui.py should be available or if it is in progress. If it should
work then a %{name}.desktop file is needed and that file must be properly
installed with desktop-file-install in the %install section. Please talk with
the developer.
* Does not own files or directories owned by other packages
* %install has rm -rf $RPM_BUILD_ROOT
* Files are all valid UTF-8
X Package seems to require Pmw (http://pmw.sourceforge.net/ I believe). Without
Pmw warnings are thrown on running tests and gui does not work.

[steve@psycho rpmbuild]$ python /usr/lib/python2.5/site-packages/proctorlib/gui.py
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/proctorlib/gui.py", line 38, in <module>
    import Pmw
ImportError: No module named Pmw
[steve@psycho rpmbuild]$ 

- Note: A later version has been released -- 1.3 Mon Jul 30 07:43:36 2007
(http://www.doughellmann.com/projects/Proctor/)
* Package seems to work as expected with the noted exception of the missing GUI
lib: 

[steve@psycho proctorlib]$ proctorbatch --list
WARNING: Could not import icon (No module named PmwContribD)
test: tests.ExampleTestCase.test00 
test: tests.ExampleTestCase.test01 
test: tests.ExampleTestCase.test02 
test: tests.ExampleTestCase.test03Failure 
test: tests.ExampleTestCase.test04Error 
test: tests.ExampleTestCase.test05 
test: tests.ExampleTestCase.test06 
test: tests.ExampleTestCase.test07 
test: tests.ExampleTestCase.test08Failure 
test: tests.ExampleTestCase.test09Error 
test: tests.ExampleTestCase.test10 
test: tests.ExampleTestCase.test11 
test: tests.ExampleTestCase.test12 
....

It also is importable as expected ....
[steve@psycho proctorlib]$ python
Python 2.5 (r25:51908, Apr 10 2007, 10:29:13) 
[GCC 4.1.2 20070403 (Red Hat 4.1.2-8)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import proctorlib
>>> print proctorlib.__doc__
Library for Proctor.

Comment 4 Anderson Silva 2007-08-13 15:43:00 UTC
I've resolved to move along to a new package, I will try to get someone else to
pick this package up.