-
Notifications
You must be signed in to change notification settings - Fork 479
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
Unit Test for "No Answer Found in KB" case #1914
Conversation
This causes an exception in the case where personal information is being logged. The question property is already created. See this issue: microsoft/botframework-solutions#1358
… into mekinney-patch-1
Pull Request Test Coverage Report for Build 64230
💛 - Coveralls |
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Configuration.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Connector.dll✔️ No Binary Compatibility issues for Microsoft.Bot.Schema.dll |
@cleemullins this is the PR that complements the other PR you merged in today (unit tests) |
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.
Hi Ashley, I requested one small change for the file name. The coveralls issue should be fixed, if you just fix that, it should trigger another build and we can merge it.
Gabo
✔️ No Binary Compatibility issues for Microsoft.Bot.Builder.dll |
Requested change is fixed. Checking in.
Wrote test for Mike Kinney's (mekinney) PR #1899
Note: I approved his PR, however coveralls/cla is "stuck" atm. I'm aware that this is re-writing his 1-line change on LN255