saucelabs / sauce_whisk

ActiveRecord style client for the Sauce Labs RESTful API
https://opensource.saucelabs.com/sauce_whisk/
MIT License
20 stars 17 forks source link

Updates SauceWhisk.load_first_found #58

Open rringler opened 7 years ago

rringler commented 7 years ago

Description

Updates SauceWhisk.load_first_found to check for the Sauce::Config constant, instead of Sauce. This will help prevent against accidental collisions with Sauce namespaces not provided by the (deprecated) Sauce gem.

I ran into this while grouping some of my other Sauce utility classes under the Sauce namespace, which makes the current version of SauceWhisk think a Sauce::Config class exists.

Risks

Low: Increases the specificity of assumptions (that a Sauce::Config class exists.)