Inumedia / SlackAPI

.NET Implementation of the Slack team communication platform API.
MIT License
452 stars 243 forks source link

Rewrite GetAccessToken test using Selenium and remove SlackAPI.Console app #151

Closed gpailler closed 6 years ago

gpailler commented 6 years ago

It's only cleanup in tests. I noticed the GetAccessToken test was disabled and I reimplemented it using Selenium. I'm using Chrome driver so you need Chrome installed to be able to run this test.

@Inumedia I removed SlackAPI.Console app because it was doing something similar to the new test but maybe this app had another purpose. Do you agree to remove it?

Inumedia commented 6 years ago

That's fine with me. Selenium is more commonly known to use anyways.