Conversation
@@ -91,14 +92,14 @@ public class AuthorizeController : Controller | |||
data.SetCurrentProfile(result); | |||
data.SetProfiles(profiles); | |||
await this.botService.SetUserData(channelId, userId, data); | |||
|
|||
return this.View(new Authorize(pin)); | |||
} | |||
catch (Exception ex) |
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.
AFAIK there is no need to catch exception here and track it as it should be done automatically by AI request processing handler injected into MVC. It tracks exceptions in correlation with request processing,
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.
Good point. Will take care of it immediately.
@@ -64,46 +65,110 @@ public Task StartAsync(IDialogContext context) | |||
private async Task MessageReceivedAsync(IDialogContext context, IAwaitable<IMessageActivity> result) | |||
{ |
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.
Parameters verification is missing.
- Followed up on new rulesets.
} | ||
|
||
/// <summary> | ||
/// Gets the pin. |
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.
It is not clear what is the pin.
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.
Planning to do better documentation after we have some code in place. So we can build up a common way of writing comments.
But for now, when the user has authorized the bot to access their profile. The authorize page shows a pin that needs to be entered in the chat app.
This is kind of a re-entry point, so we can track where the user is with the flow.
Removed try..catch
Updated based on Vladimir his comments. |
Sorry - that time can't find the place where I can approve PR if I should. I'm fine wit completing it now. |
I think we can directly merge the PR's on the development branch. |
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.
Overall we seem to be missing a lot of tests. Perhaps something I could look at?
<IncludeAll Action="Warning" /> | ||
<Rules AnalyzerId="Microsoft.Analyzers.ManagedCodeAnalysis" RuleNamespace="Microsoft.Rules.Managed"> | ||
<Rule Id="CA1020" Action="None" /> | ||
<Rule Id="CA1305" Action="None" /> |
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 don't agree that we should suppress CA1305.
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.
You are right, my mistake. Then we need to use 'string result = FormattableString.Invariant($"{name}");" for interpolated strings.
@@ -44,18 +42,22 @@ public ConnectDialogTests() | |||
[TestMethod] |
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 recommend a TestCategory on every test method. I like to define some constants in a TestCategories class (can be a common file that is linked into the test projects). I know we can't use it in SpecFlow tests but we should use it elsewhere.
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.
Need to setup a guidance for the team.
toBot1.From.Id = fromId; | ||
toBot1.Text = "Hi"; | ||
|
||
var toBot2 = this.Fixture.CreateMessage(); |
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.
Probably one of those "religious" debates, but I dislike var at the best of times. I will accept var where the type is explicit on the right hand side, but I really think it should not be used if the type is not clear by inspection. You can't necessarily program to an interface if you use var as well.
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.
Let's get some guidance for the team, and refactor after.
Assert.IsNotNull(card, "Missing signin card."); | ||
|
||
var button = card.Buttons.FirstOrDefault(); | ||
Assert.IsNotNull(button, "Button is missing"); | ||
|
||
var expected = | ||
$"https://app.vssps.visualstudio.com/oauth2/authorize?client_id={appId}&response_type=Assertion&state={toBot.ChannelId};{toBot.From.Id}&scope={scope}&redirect_uri={authorizeUrl}/"; | ||
$"https://app.vssps.visualstudio.com/oauth2/authorize?client_id={appId}&response_type=Assertion&state={toBot2.ChannelId};{toBot2.From.Id}&scope={scope}&redirect_uri={authorizeUrl}/"; | ||
Assert.AreEqual(expected.ToLower(), button.Value.ToString().ToLower(), "OAuth url is invalid."); | ||
} |
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 wonder if this could be done as multiple unit tests rather than have a lot of asserts in a single test? Extracting the setup code into methods with intent-revealing names will also make the test much more readable.
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.
There should be one in the test and one in the api code. We can extract to a common.
@@ -44,18 +42,22 @@ public ConnectDialogTests() | |||
[TestMethod] | |||
public async Task FirstTimeConnectionTest() |
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.
The name of the test should be a little more descriptive so that if it fails I know from the name more about what has gone wrong. I often use a class called "GivenXYZ", with test methods inside called "WhenSomething_ThenThisThingHappens". I don't know what this test is going to test for from the name.
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, we need to setup guidance for the team.
@@ -44,7 +43,6 @@ public class AuthorizeController : Controller | |||
this.authenticationService = authenticationService; |
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.
We should check these for nulls.
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.
We should come up with a guards strategy, like you did before.
@@ -57,48 +55,41 @@ public class AuthorizeController : Controller | |||
/// <returns>A view</returns> | |||
public async Task<ActionResult> Index(string code, string error, string state) |
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.
Are there any tests that cover this code (acceptance, unit or anything else)?
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.
there is a unit test.
{ | ||
var activity = await result; | ||
var generator = new Random(); |
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 a security reviewer would not be very comfortable with using Random(), there are better ways to generate cryptographically random numbers. My friendly local security person tells me I should use System.Security.Cryptography.RNGCryptoServiceProvider.
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.
let's refactor as you describe.
if (!profiles.Any() || profile == null) | ||
{ | ||
await this.Login(context, activity, reply); | ||
return; |
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 another one that is somewhat about personal taste, but I think return in the middle of a method makes it hard to understand, you may be looking at code later in the method and fail to see that the code may never be reached.
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.
It needs a discussion. I rather have it like this, then have nested if statements. Which I find hard to read.
@@ -50,8 +73,30 @@ public Task StartAsync(IDialogContext context) | |||
} | |||
} | |||
|
|||
private async Task Welcome(IDialogContext context, IMessageActivity activity) | |||
{ | |||
var account = context.UserData.GetCurrentAccount(); |
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 little sequence seems to be common too. Should it be abstracted?
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.
We can introduce a base class.
So to recap, what Rob said:
@rjarrattmsft Do you want me to first refactor before we pull in the pull request, or afterwards? |
Generally I would prefer only to commit production quality code, but as I am sure we want to get this merged sooner rather than later I don’t mind if we make an exception this time around. But in my opinion we should address all quality issues before doing any more development.
Regards
Rob
Robert Jarratt
Senior Consultant
Microsoft Consulting Services
Microsoft Ltd.
Tel.: +44 (118) 909 4923
email: mailto:rjarratt@microsoft.com
This and subsequent emails on the same topic are for discussion and information purposes only. Only those matters set out in a fully executed agreement are legally binding. This email may contain confidential information and should not be shared with any third party without the prior written agreement of Microsoft. If you are not the intended recipient, take no action and contact the sender immediately.
Microsoft Limited (company number 01624297) is a company registered in England and Wales whose registered office is at Microsoft Campus, Thames Valley Park, Reading RG6 1WG
From: Jeffrey Opdam [mailto:notifications@github.com]
Sent: 08 May 2017 09:53
To: ALM-Rangers/Team-Services-Bot <Team-Services-Bot@noreply.github.com>
Cc: Rob Jarratt (MCS UK) <rjarratt@microsoft.com>; Mention <mention@noreply.github.com>
Subject: Re: [ALM-Rangers/Team-Services-Bot] Jo 13138 (#15)
So to recap, what Rob said:
* We need test strategy in place.
* Standarize on const string location.
* Guard clause strategy.
@rjarrattmsft<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frjarrattmsft&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=uhUBe%2F%2FIHagjudaSHOk9pNK928Z6k40f0ZzoiKUHb0k%3D&reserved=0> Do you want me to first refactor before we pull in the pull request, or afterwards?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FALM-Rangers%2FTeam-Services-Bot%2Fpull%2F15%23issuecomment-299810516&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=nrmra6pXIy1SlReXue5S7YwSIudRDusCeNV4dYEGpAE%3D&reserved=0>, or mute the thread<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZiI9E4oopsuLRuApzdZ9sNW1-AgOx0Mks5r3tf8gaJpZM4NTBqv&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=WrVcX8Fgdn1tVR3oxp6ieYzuGf%2FSYftwNszkXKnh4Y8%3D&reserved=0>.
|
Agree
~vg
From: rjarrattmsft [mailto:notifications@github.com]
Sent: Monday, May 8, 2017 12:00 PM
To: ALM-Rangers/Team-Services-Bot <Team-Services-Bot@noreply.github.com>
Cc: Vladimir Gusarov <vgusarov@hotmail.com>; Review requested <review_requested@noreply.github.com>
Subject: Re: [ALM-Rangers/Team-Services-Bot] Jo 13138 (#15)
Generally I would prefer only to commit production quality code, but as I am sure we want to get this merged sooner rather than later I don’t mind if we make an exception this time around. But in my opinion we should address all quality issues before doing any more development.
Regards
Rob
Robert Jarratt
Senior Consultant
Microsoft Consulting Services
Microsoft Ltd.
Tel.: +44 (118) 909 4923
email: mailto:rjarratt@microsoft.com
This and subsequent emails on the same topic are for discussion and information purposes only. Only those matters set out in a fully executed agreement are legally binding. This email may contain confidential information and should not be shared with any third party without the prior written agreement of Microsoft. If you are not the intended recipient, take no action and contact the sender immediately.
Microsoft Limited (company number 01624297) is a company registered in England and Wales whose registered office is at Microsoft Campus, Thames Valley Park, Reading RG6 1WG
From: Jeffrey Opdam [mailto:notifications@github.com]
Sent: 08 May 2017 09:53
To: ALM-Rangers/Team-Services-Bot <Team-Services-Bot@noreply.github.com<mailto:Team-Services-Bot@noreply.github.com>>
Cc: Rob Jarratt (MCS UK) <rjarratt@microsoft.com<mailto:rjarratt@microsoft.com>>; Mention <mention@noreply.github.com<mailto:mention@noreply.github.com>>
Subject: Re: [ALM-Rangers/Team-Services-Bot] Jo 13138 (#15)
So to recap, what Rob said:
* We need test strategy in place.
* Standarize on const string location.
* Guard clause strategy.
@rjarrattmsft<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frjarrattmsft&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=uhUBe%2F%2FIHagjudaSHOk9pNK928Z6k40f0ZzoiKUHb0k%3D&reserved=0> Do you want me to first refactor before we pull in the pull request, or afterwards?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FALM-Rangers%2FTeam-Services-Bot%2Fpull%2F15%23issuecomment-299810516&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=nrmra6pXIy1SlReXue5S7YwSIudRDusCeNV4dYEGpAE%3D&reserved=0>, or mute the thread<https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAZiI9E4oopsuLRuApzdZ9sNW1-AgOx0Mks5r3tf8gaJpZM4NTBqv&data=02%7C01%7Crjarratt%40microsoft.com%7C00726c66c85247f0c23b08d495efac1d%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636298303997745737&sdata=WrVcX8Fgdn1tVR3oxp6ieYzuGf%2FSYftwNszkXKnh4Y8%3D&reserved=0>.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub<#15 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAgZUfiMT-TB9twWmFyBeyOEZO1JuTIdks5r3tmGgaJpZM4NTBqv>.
|
Agreed. I will merge it this evening. @rjarrattmsft still interested in to look into the unittests? |
Me too. I thought to increase coverage for extension before adding new code.
Maybe it’s a good idea plan PBI(s) for this sort of debts?
~vg
From: Jeffrey Opdam [mailto:notifications@github.com]
Sent: Monday, May 8, 2017 12:36 PM
To: ALM-Rangers/Team-Services-Bot <Team-Services-Bot@noreply.github.com>
Cc: Vladimir Gusarov <vgusarov@hotmail.com>; Review requested <review_requested@noreply.github.com>
Subject: Re: [ALM-Rangers/Team-Services-Bot] Jo 13138 (#15)
Agreed. I will merge it this evening.
@rjarrattmsft<https://github.com/rjarrattmsft> still interested in to look into the unittests?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub<#15 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AAgZUfsH9VJ4SjznFHqWxKFLj_7ISIwuks5r3uHugaJpZM4NTBqv>.
|
Will add them. |
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot.
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings.
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet.
* Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file.
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file.
* Development (#71) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file. * fix
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file. * Feature/buildfix (#72) * Development (#71) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading o…
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file. * Feature/buildfix (#72) * Development (#71) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading o…
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file. * Feature/buildfix (#72) * Development (#71) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading o…
* Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Development (#65) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * fixed project file. * Feature/buildfix (#72) * Development (#71) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Added reading of build definitions. * Fixed continious login prompt. * Fixed GetAccountsTest * Added tests. * Added unit tests for VstsService. * Minor formatting changes. * Update the third party party notices file to reference all OSS libraries. (#37) * Updated NuGet packages * Added missing method description. * Fixed NuGet references. Added missing analyzers. * Implemented unit tests for VstsService.ChangeApprovalStatus() and VstsService.GetApprovals() * Removed unused reference. * Feature/jo acceptance tests (#38) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#40) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Feature/jo acceptance tests (#41) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Feature/jo acceptance tests (#42) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * Added re initialize of the refreshtoken * Check refreshtoken * debug info * Removed Console.WriteLine * Now using the id * Exception handling * Removed Fakes warnigns * added the authorizeurl * Another addition * more logging * More debug info * Debug info * Small refactor. * AGAIN * again * Debug * Update Team-Services-Bot.runsettings * Removed all the debug lines. * Fixed VSTS Tests * Removed Fakes warnings (#44) * Removed Fakes warnigns * Fixed VSTS Tests * Feature/jo pages (#45) * Adding acceptance tests. * Finished coding Approvals - Acceptance Tests. * Added connect acceptance test. * Updated ThirdPartyNotices.txt * Added basic pages. * Added pages. * Finished with the tests. * Added unit tests for default controller. * Added updated config files. * Added the Eula page. * Fixed some issues. * Ignore one test. * Second try. * Removed trailing space * - Moved EULA to it's own controller. - Added link to EULA in the bots welcome message. * Updated ChangeApprovalStatusTest to match changed method under test. * Corrected formatting. * Corrected formatting * Added unit tests for uncovered methods in VstsService class * Temporarily removed account Uri lookup. (#48) * Updated readme. * Added Vladimir as contributor. * Simplified connecting to VSTS. * Feature/13140 list build definitions (#49) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Feature/13140 list build definitions (#50) * Temporarily removed account Uri lookup. * Added more * Added basics & basic tests for BuildsDialog. * Added code for showing the BuildDefinitionCard * Added acceptance test. * Fixed message * Changed type to Index.cshtml of authorize to content * Added feature to clear user data. * Features/13141 jo (#51) * Added QueueBuildAsync method and test * Added logic for handling queing a build. * Added acceptance test. * Fixed some small bugs regarding Build Queue * Fixed a warning * Features/13142 jo (#52) * Added method to get release definitions * Added ReleasesDialog with unit tests for listing release definition. * Added acceptance test for listing releases * Updated CreateRelease so it picks up multiple latest builds for 1 or more artifacts. * Added Create method to releasesdialog, added unittests, added acceptance tests. * Renamed test name. * Added GetReleaseAsyncTest * Fixed error handling in AuthorizeController * added scope configuration value (#55) * fixed issue with unit tests * Restored back default name for AppSettings configuration file. * Update shims files * Joined null checks with assigments. * Added Guards to VSTSService * Feature/13290 jo (#56) * Updated guards in the extension method(s) * Added Guards on the Cards. * Added Guards to Bootstrap & Dialogs. * Feature/naming convention (#58) * Adhere to the naming * Converted to VSTS * Update README.md * Feature/jo refactorings (#59) * Removed the common tests project. * Removed ConfigurationProvider * Updated AppSettings Handling unknown commands. Updated connect message. * Added check if a team project from the list is selected. (#60) * Updated notice, so it uses the official name of the VSTS Bot. * Fixed issue with ignore change bat files. Cleaned AppSettings. * Using the display name of the profile to get the users approvals. * changed the profile scope. * Reverted * Feature/jo bug fixes (#64) * Fixed issue when there empty lists, which resulted in an empty reply. * Showing the connect button as help if not connected yet. * Updated NuGet packages. Fixed an issue with disappearing tests after app.config gets updated as part of NuGet package installation. * Removed few compilation warnings. * Added handling of exceptions during dialog conversations. * Fixed AppSettings file name * Added support for CommandName to UnknownCommandException and default error message. * Feature/13566 jo (#68) * Working on the homepage. * Updated title * using 1 layout * Updated pin page. * Added the text. * Feature/syncing branches (#70) * Development (#62) * Update README.md * Update README.md * Added a common folder for common assembly info and custom dictionary. Added code analysis to bot project and fixed errors and warnings. Work incomplete. * Added Code Analyzers. * fix CA * Refactored the tests to make it more simple. * Fixed wrong folder name. * Updated the source headers. * Added initial acceptance test. * Acceptance Tests. * updated ignore file, so it ignores the runsettings file. * Setup configuration. * Setup configuration. * Updated the readme.md file with guidance how to setup the user for Acceptance Tests. * Updated the headers * - Added the CommonAssemblyInfo to Acceptance Test. - Removed the unnecassary try catch from messagesController * Added AppSettings.config to repo * Trying to get it right * Added AppSettings.config again * Add runsettings * Added batch for to ignore changes. Updated the readme * Fixed the web.release.config * Web.Release.config * again * Change * And finally fixed. * - Adopted Autofac. - Updated Unit test to accomodate Autofac. * Added Telemetry * Enabled StyleCop for the api project. * Added stylecop analysis for the Unit Test project. * Added stylecop rules to Acceptance test project. * Initial code to popup a signin card and redirect to login screen. * Added first unit test for connecting to an account. * Tested with emulator, it is spinning up. * Mark the current specflow tests using the DirectLine client as Integration tests. * Start of controller feature tests. * Worked on the AuthorizeController, when a redirect is done to the controller with an authorization code. We get the profile and accounts for that user. * Able to do a round trip with the oauth. * Update AppSettings.config * Simplified. * Added initial steps for testing the controller, although not working yet. Also added a test code rule set for code analysis. * Fixed issue for Test Explorer not finding the tests. * Added acceptance tests for the controller, which needs dialogs to be invoked through an injected interfaces. Also tests situation where controller gets an exception. * Added test for non-message activities sent to the controller. * Finished the unit test for retrieving and processing a security code. * Refactored a bit. * Added a production code ruleset. Fixed all code analysis errors. Ensure all warnings are treated as errors. * Checks if a profile was selected before. * Small Updates. * Fixed issue with different version of assemblies Enabled the custom rulesets for release configuration. * Fixed issue with test not showing up * Added common files to solution for easy tracking. * Updated NuGet packages. * Revert "Updated NuGet packages." This reverts commit 141aeac7753c529ffa3de137ace4a70b8e82562d. * Jo 13138 (#15) * Moving forward to a using a flow for the user. * More changes to the flow. * Working on the flow. * Finished the flow for connecting to an account. * Fixed current unittests. * Small cleanup in the Authorize controller. * Some minor fixes. * - Updated Nuget packages. - Followed up on new rulesets. * Added guards in ConnectDialog Removed try..catch * Updated NuGet packages. * Added ReSharper settings file. * Removed unused parameter. * Corrected IDisposable patter implementation * Wrapped any iterpolated string with FormattableString.Invariant(), so… (#17) * Wrapped any iterpolated string with FormattableString.Invariant(), so it uses the Invariant cultureinfo. * Removed unused using. * Fixed failing controller tests. The builder needs to be set up in the controller tests. However, to make it work the builder needs configuration so added a configuration provider interface that can be mocked. Also had to remove some C# 7 syntax. * Updated the Random generator to RNGCryptoServiceProvider, so the gene… (#16) * Updated the Random generator to RNGCryptoServiceProvider, so the generation is more secure. * Using UInt32, and modules to get a 5 number pin * Added test categories and applied them. Renamed Acceptance tests to Behavioural. * Jo 13289 (#20) * Started on a work around to mock user data. * Got the current unit tests working. * Simplified the unit tests * Added unit tests. * Fixed minor inspection warnings. * Renamed IProfileService to IVstsService as it serves not only profile retrieval. It will be extended to get other VSTS objects like projects, etc. * Cleaned up documentation. (#24) * Cleaned up documentation. * Added suppress warning 1591 to all configurations for test projects. * Missed the change to the custom dictionary . This broke the build * Implemented reading VSTS projects. Added basic unit tests. * Reordered using * Acceptance tests welcome (#27) * Added Acceptance Tests for the welcome messages. * Update Team-Services-Bot.runsettings * Update Team-Services-Bot.runsettings * Added two acceptance tests for welcome, * Update Team-Services-Bot.runsettings * Added CreatedOn and ExpiredOn for easier expiration detection of the access token. * Undo ignore batch file. * Two suppress messages. * Renamed a method. * Moved TestCategories class into a separate common test project. Applied categories to uncategorised tests. Also pointed all assemblies to a common strong name key file. Removed a redundant warning suppression. * Getting some info from console. * showing bot id information. * Reverted logging stuff. * Updated test category to use common class rather then string. * Removed commented code. * issue with the authorization scope. Fixed. * Implemented project selection * Refactored usage of IVstsAccount in ConnectDialog. Fixed tests. * Feature/13280 jo (#31) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Feature/13280 jo (#32) * Added the acceptance tests. * Added approvals acceptance tests. * Start for the ApprovalsDialog. * Added Approvals list, Approve and Reject. * Finished Approve and reject tests. * Rewritten unit tests. * Added refresh token support. * Running some tests on the approvals flow. * Added email address to the profile. * Fixed button in Teams * Fixed issue with the refreshtoken. * - Updated according to the comments. - Using account as string again * Fixed warnings * Fixed warning * Fixed tests. * Reduced cognitive complexity * Improved method naming and disposed connection object. * Feature/jo refactoring root and connect dialogs (#33) * Refactored RootDialog Unit Tests. * Figured out how to mock the Wait method on DialogContext. * Added unit tests for connectdialog. * Finished unit tests for connect dialog. * Added unit tests for root dialog. * Finished Unit Tests for Approvals Dialog. * Added unit tests for the cards. * Excluded some classes from code coverage. * Added more unit tests. * Added more unit tests. * auth scope reset. * Added runsettings. * Changed to include. * fixing runsettings * Fixing runsettings * Final, * Fixed the issue * Fixed issue with tests. Changed the welcome message. * Feature/jo refactor msg controller tests (#35) * Moved the MessagesControllerTests to the unit test project. * Encapsulated ContainerBuilder into Bootstrap * Feature/jo more refactoring (#36) * Renamed a few method * Adjusted the code around the pin. so the profile is not used before the pin is entered. This way, it is harder to inject someones profile into another users session. * var. * Adde…
Thank you for your pull request!
By completing this pull request, you agree to the Contributing License Agreement.
Fixes # .
Changes proposed in this pull request:
@ALM-Rangers/Team-Services-Bot