[FIX] issue 999: error in generate.ts is masked and hence command lin… #312
Conversation
…e is returning status 0
|
@dennisseah Tests are failing |
Yes, I need help from Nate. |
we have figured out the problem and the problem is in the unit test and our implementation is masking the issue. Hence fixing the implementation causes problem because unit test is incorrect. I have moved the git clone tests to the beginning of the test run and hence |
NathanielRose
left a comment
NathanielRose
left a comment
There was a problem hiding this comment.
Integration tests and unit tests passing locally.
| jest.restoreAllMocks(); | ||
| }); | ||
|
|
||
| ////////////////////////////////////////////////////////////////////////////// |
There was a problem hiding this comment.
Did you mean to keep this?
There was a problem hiding this comment.
to clearly separate git related tests from other tests
…e is returning status 0
Closes microsoft/bedrock#999
modified code to not catch the exception and return it. Instead allow the exception to propagate to the calling function