test: minor login test cleanup

SmartThingsCommunity / smartthings-cli

Command-line Interface for the SmartThings APIs.

https://developer.smartthings.com/docs/sdks/cli/introduction

Apache License 2.0

230 stars 102 forks source link

test: minor login test cleanup #569

Closed rossiam closed 9 months ago

rossiam commented 9 months ago

I want to do at least a little more clean-up of these tests but have limited this PR because of the indentation changes due to the removal of the top-level describe.

remove superfluous top-level describe
mock delay function from util to wait 10ms

Checklist

[x] I have read the CONTRIBUTING document
[x] My code follows the code style of this project (npm run lint produces no warnings/errors)

changeset-bot[bot] commented 9 months ago

⚠️ No Changeset found

Latest commit: b0dd11e59ce119a0f0a4a7bbe474bf8c05a849f9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR