search

saucelabs / sauce-cypress-plugin

Cypress Plugin to report your results directly to Sauce Labs
MIT License
7 stars 2 forks source link

refactor: Handle no credentials condidtion #83

Closed tianfeng92 closed 1 month ago

tianfeng92 commented 1 month ago

Description

Refactor and refine handing no credentials condition.

tianfeng92 commented 1 month ago

Since afterRunTestReport does not check account settings, there is no need for additional checks on the SAUCE_VM setting.

I think you do need to check it still. If the user is using this plugin while having a mixed use of environments where Cypress runs (some locally, in CI and Sauce Labs), any time they'd run their tests—even if partially—via Sauce Labs, they'd get this warning.

Good point!