-
Notifications
You must be signed in to change notification settings - Fork 2
fnutils and errutils test cases, inline docs, listall fnc fix #82
Conversation
neo7337
commented
May 20, 2023
- added test cases for fnutils
- added test cases for errutils
- added inline docs for fnutils and errutils
// ExecuteAfter(nil, timeout) | ||
//}) | ||
// | ||
//t.Run("Execution does not occur when timeout is negative", func(t *testing.T) { |
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.
@nandagopalan have a look at this condition if we need to cater for this in our logic
// } | ||
//}) | ||
// | ||
//t.Run("No execution occurs if the function is nil", func(t *testing.T) { |
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.
@nandagopalan same here, if we need to handle this condition in our logic
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 think we may have to put a nil check in the code.
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 am approving this PR we will add that as a separate issue.