Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Missing error check for NewRequestedPlugin in tests results in panic if plugin not found #698

Closed
1 of 3 tasks
tiffanyfay opened this issue Feb 4, 2016 · 0 comments
Closed
1 of 3 tasks

Missing error check for NewRequestedPlugin in tests results in panic if plugin not found #698

tiffanyfay opened this issue Feb 4, 2016 · 0 comments
Assignees
@IRCody
Labels
tracked type/bug

Comments

@tiffanyfay
Copy link
Contributor

Tests panic if plugin isn't found because there isn't error checking.
See #695

Some tests with this issue:
@lynxbat lynxbat added the tracked label Feb 4, 2016
@IRCody IRCody self-assigned this Feb 4, 2016
IRCody added a commit to IRCody/snap that referenced this issue Feb 4, 2016
@IRCody
Partially fixes intelsdi-x#698 - Adds error checking for NewRequested…
835bc58 
…Plugin calls in plugin_test.go
IRCody added a commit to IRCody/snap that referenced this issue Feb 10, 2016
@IRCody
Partially fixes intelsdi-x#698 - Adds error checking for NewRequested…
6478220 
…Plugin calls in plugin_test.go
tiffanyfay added a commit that referenced this issue Feb 12, 2016
@tiffanyfay
Merge pull request #699 from IRCody/Fixes_698_Missing_Error_Checks
62d84ad 
Partially fixes #698 - Adds error checking for NewRequestedPlugin
candysmurf pushed a commit to candysmurf/snap that referenced this issue Feb 12, 2016
@IRCody @candysmurf
Partially fixes intelsdi-x#698 - Adds error checking for NewRequested…
d122442 
…Plugin calls in plugin_test.go
@IRCody IRCody mentioned this issue Feb 29, 2016
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@IRCody IRCody

Labels
tracked type/bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lynxbat @IRCody @tiffanyfay