-
Notifications
You must be signed in to change notification settings - Fork 1
add pressTile step #1
base: main
Are you sure you want to change the base?
Conversation
lib/index.js
Outdated
@@ -0,0 +1,6 @@ | |||
let { Given, When, Then } = require('@cucumber/cucumber'); |
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.
const
?
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.
essentially the same const
vs let
remark that I already left in the wdi5-fe-selectors
repo
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.
done
lib/index.js
Outdated
let { Given, When, Then } = require('@cucumber/cucumber'); | ||
let lib = require('wdi5-fe-selectors'); | ||
|
||
Given('we press tile {string}', async function (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.
any need for the this
-scoped function
as opposed to async (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.
done - no reason currently
No description provided.