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
root: tests #454
root: tests #454
Conversation
}; | ||
} | ||
|
||
S.prototype = Object.create(Emitter.prototype); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've always wondered how to assign the Emitter properties to a single object instance!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use a 🔨 !
@rwaldron thanks for putting this together. I got started on tests but didn't get far so this was very useful. |
|
||
var opts = {}; | ||
|
||
controller.root(opts) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm a little surprised root
doesn't reject
because standardTesselCommand
doesn't return a Tessel.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see - because standardTesselCommand
returns Promise.resolve (instead of an unfulfilled promise), the whole root
procedure finishes up before it gets to the shell
part.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Looks great! |
No description provided.