Bug 717067 - nspr builds should run the test suite
Summary: nspr builds should run the test suite
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: nspr
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Elio Maldonado Batiz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2011-06-27 20:46 UTC by Elio Maldonado Batiz
Modified: 2013-03-11 23:08 UTC (History)
2 users (show)

Fixed In Version: nspr-4.8.8-4.fc16
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-03-11 23:08:11 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
adds a check section to run the nspr tests (1.08 KB, patch)
2011-06-27 21:08 UTC, Elio Maldonado Batiz
kengert: review+
Details | Diff

Description Elio Maldonado Batiz 2011-06-27 20:46:41 UTC
Description of problem: No tests are run when nspr is built.

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

How reproducible: always

Steps to Reproduce:
1. Download the latest nspr srpm from Koji into ~/rpmbuild/SRPM
2. cd ~/rpmbuild/SRPM; rpm -ihv nspr-4.8.8-1.fc16.src.rpm
3. cd ~/rpmbuild/SPECS; rpmbuild --rebuild nspr-4.8.8-1.fc16.src.rpm' 
(or execute 'fedpkg local' from the nspr checkout directory)
  
Actual results:
No tests are run

Expected results:
the test suite is run

Additional info:

wtc informs me that one can run either the shell script mozilla/nsprpub/pr/tests/runtests.sh or its Perl script version mozilla/nsprpub/pr/tests/runtests.pl. The latter is reportedly better at aborting a timed-out test.

Comment 1 Elio Maldonado Batiz 2011-06-27 20:55:48 UTC
Like we do for nss.spec we could add to nspr.spec:

%check
# Run test suite.
perl ./mozilla/nsprpub/pr/tests/runtests.pl 2>&1 | tee output.log

TEST_PASSED=`grep -c Passed ./output.log`
TEST_FAILURES=`grep -c FAILED ./output.log` || :
if [ $TEST_FAILURES -ne 0 ]; then
  echo "error: test suite returned failure(s)"
  exit 1
fi
echo "test suite completed"

Comment 2 Elio Maldonado Batiz 2011-06-27 21:08:07 UTC
Created attachment 510169 [details]
adds a check section to run the nspr tests

Comment 3 Kai Engert (:kaie) (inactive account) 2011-07-18 16:21:30 UTC
Comment on attachment 510169 [details]
adds a check section to run the nspr tests

I think you don't need TEST_PASSED, you could remove that line.

r=kaie

Comment 4 Fedora Update System 2011-07-22 15:33:33 UTC
nspr-4.8.8-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/nspr-4.8.8-4.fc15

Comment 5 Fedora Update System 2011-07-23 02:07:47 UTC
Package nspr-4.8.8-4.fc15:
* should fix your issue,
* was pushed to the Fedora 15 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing nspr-4.8.8-4.fc15'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/nspr-4.8.8-4.fc15
then log in and leave karma (feedback).


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