993660 - Test infer_arguments()

Status: RESOLVED FIXED

(Developer Services :: Mercurial: bzexport, defect)

Description

[Steve Fink [:sfink] [:s:]]

bzexport really really needs tests.

Comment 1

[Steve Fink [:sfink] [:s:]]

Attachment: Test infer_arguments()

But I'm too much of a Python newbie to know the currently recommended way to do tests. So I'll put this up for review to force you to tell me. ;-)

I ran this from within the bzexport directory with

  PYTHONPATH=.. python test/test_infer_arguments.py

which probably is NOT the recommended way.

Comment 2

[(not currently active) Ted Mielczarek]

If you just put your test.py in the hgext directory it should be able to import bzexport directly. If you're not keen on having it up that far then you'll have to munge sys.path, something like:
import sys
sys.path.append(os.path.join(os.path.dirname(__file__), '../..'))

Comment 3

[Ed Morley [:emorley]]

Comment on attachment 8403538 [details] [diff] [review]
Test infer_arguments()

gps has just added support for Mercurial style tests in https://hg.mozilla.org/hgcustom/version-control-tools/rev/7085790ff3af - I'll switch the review to him since it would be good to have a coordinated approach if possible :-)

Comment 4

[Gregory Szorc [:gps]]

Comment on attachment 8403538 [details] [diff] [review]
Test infer_arguments()

Review of attachment 8403538 [details] [diff] [review]:
-----------------------------------------------------------------

I'm out of commission for a while. Redirecting review.

::: hgext/bzexport/test/test_infer_arguments.py
@@ +1,1 @@
> +import unittest

Add license.

Comment 5

[(not currently active) Ted Mielczarek]

Comment on attachment 8403538 [details] [diff] [review]
Test infer_arguments()

Review of attachment 8403538 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, good to have tests for this stuff.

Comment 6

[Steve Fink [:sfink] [:s:]]

http://hg.mozilla.org/hgcustom/version-control-tools/rev/d371a48ca617