get_user_id_ Tests and Fixes #68

Merged
8 commits merged into from Jan 1, 2011

Projects

None yet

3 participants

rfw commented Jan 1, 2011

Fixes for get_user_id as well as tests.

Contributor
jkeenan commented Jan 1, 2011

rofflwaffls:
I executed the commands described in Steps 1 and 2 below, i.e., I got your code but did not merge it into master. After Step 2, I called "perl Configure.pl --test; make; prove -v t/dynpmc/os.t. I got this error in the test file:
not ok 17 - Test get_user_id

Failed test 'Test get_user_id'

at t/dynpmc/os.t line 461.

got: '1000

'

expected: '0

'

ok 18 - Test can_execute

Looks like you failed 1 test of 18.

Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/18 subtests

Test Summary Report

t/dynpmc/os.t (Wstat: 256 Tests: 18 Failed: 1)
Failed test: 17
Non-zero exit status: 1

This was run on linux/i386. Can you diagnose the test failures? Thanks.

Contributor

Okay, I've already merged this branch (sorry, I don't think I saw Jim's last comment before merging). I'll take a look at the failures too.

Contributor
jkeenan commented Jan 1, 2011

I think rofflwaffls was fixing the code just about the same time I was testing. After the merge into trunk, my tests passed.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment