Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error if you inadvertently add test to itself #5

Open
alecmce opened this issue Oct 20, 2009 · 3 comments
Open

Error if you inadvertently add test to itself #5

alecmce opened this issue Oct 20, 2009 · 3 comments

Comments

@alecmce
Copy link

alecmce commented Oct 20, 2009

I found that I had inadvertently added a test suite to itself today, but got an error that wasn't terribly helpful in working out what I had done. It would be good if ASUnit handled this without going to a recursion stack overflow.

(I'll do it myself if I remember, but wanted to add the issue here in case I forgot)

Cheers,

Alec

@robertpenner
Copy link
Collaborator

Do you mean a test suite?

@alecmce
Copy link
Author

alecmce commented Oct 21, 2009

Yes, sorry, I meant a test suite. I said so in the body, but not in the title. I haven't looked into it but imaigne it's a pretty trivial fix; if you're refactoring then I'll wait until your repository is pulled then I'll have a look at making the fix.

@robertpenner
Copy link
Collaborator

I haven't done much in TestCase or TestSuite. You can pull my changes from master yourself, make the change, and then Luke can pull from you. With Git, the order is more flexible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants