SuffolkLITLab / ALKiln

Integrated automated end-to-end testing with docassemble, puppeteer, and cucumber.
https://assemblyline.suffolklitlab.org/docs/alkiln/intro
MIT License
14 stars 4 forks source link

Fixing one of my frequent typos #758

Closed michaelhofrichter closed 1 year ago

michaelhofrichter commented 1 year ago

In this PR, I have:

Reason for this PR

Docassemble's Continue button has a capital C. When I write the tests, I consistently write it out to match that format. This clones the I tap to continue step so that it works with lower or upper case.

It also appears that my markdown linter is a little aggressive in reformatting the Changelog.

Links to any solved or related issues

None.

Any manual testing I have done to ensure my PR is working

None (sorry). One of the tests in Reports.feature is updated to use a capital I tap to Continue so if that still passes, all should be ok.

plocket commented 1 year ago

This is a great improvement and something that totally hadn't occurred to us! Thanks for this PR! If it's ok, I might tweak the implementation of the new step to avoid the duplicate code.