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

[Newcomer] Seattlelib unit tests did not catch advertise NameError #113

Open
choksi81 opened this issue May 24, 2014 · 0 comments
Open

[Newcomer] Seattlelib unit tests did not catch advertise NameError #113

choksi81 opened this issue May 24, 2014 · 0 comments

Comments

@choksi81
Copy link
Contributor

My recent change to centralized v1 for #1257 introduced a NameError. However, it was not caught by the seattlelib unit tests. I only found out aobut this when I looked at a nodemanager log. The existing unit test should be modified so that exceptions that are raised by the different advertise libraries are picked up.

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

No branches or pull requests

2 participants