#1 Fix NPE while processing exception in new node creator

Merged
jonathan merged 1 commits from kris/fix_npe into master 4 years ago
kris commented 4 years ago

@jonathan please review

Passing Jenkins build: https://jenkins.bubblev.org/job/bubble-dev-kris/1/

I couldn’t find a way to actually cover this problematic line with a test. Do you know any any test, I guess using Mocked Compute Service, that is actually passing through this method I could update to go once through this catch block?

@jonathan please review Passing Jenkins build: https://jenkins.bubblev.org/job/bubble-dev-kris/1/ I couldn't find a way to actually cover this problematic line with a test. Do you know any any test, I guess using Mocked Compute Service, that is actually passing through this method I could update to go once through this catch block?
jonathan was assigned by kris 4 years ago
jonathan approved these changes 4 years ago
jonathan closed this pull request 4 years ago
kris deleted branch kris/fix_npe 4 years ago

Reviewers

jonathan approved these changes 4 years ago
The pull request has been merged as c22bda6037.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.