-
Notifications
You must be signed in to change notification settings - Fork 269
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
Set mock tl function #161
Set mock tl function #161
Conversation
@@ -18,17 +18,28 @@ export class TaskMockRunner { | |||
this._answers = answers; | |||
} | |||
|
|||
public setFunction(name: string, val: any) { |
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.
This is untested. Submitting PR for discussion only at this point.
The goal of this function is to still leverage mock-task but additionally allow custom mocking for specific functions only.
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.
registerMockExport
if (this._answers) { | ||
tlm.setAnswers(this._answers); | ||
|
||
public run(noMockTask: boolean) { |
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.
Also I added a bool here whether to register mock-task. What I'm thinking is, the runner is still useful for L1 scenarios where the tl would not be mocked.
This makes sense |
6df7975
to
da68cea
Compare
this is ready for review now |
da68cea
to
174b596
Compare
No description provided.