Skip to content

Commit

Permalink
Fix fill-human-multi (#684)
Browse files Browse the repository at this point in the history
* Fix fill-human-multi vector case

Incorrectly used `fill` rather than `fill-human` in the implementation
and didn't pass the options either.

* Update CHANGELOG.
  • Loading branch information
dgr authored Oct 8, 2024
1 parent b4f93eb commit eb0c0ad
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions CHANGELOG.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ A release with an intentional breaking changes is marked with:
* Changes
** {issue}676[#676]: Fix new driver creation so that it sidesteps some underlying Firefox race conditions and improves CI test stability. ({person}dgr[@dgr])
** {issue}679[#679]: Add `new-window` function that exposes WebDriver's New Window endpoint. ({person}dgr[@dgr])
** {issue}682[#682]: Fixed a bug in `fill-human-multi` to correctly call `fill-human` rather than `fill`. ({person}dgr[@dgr])

== v1.1.42 - 2024-09-27 [[v1.1.42]]

Expand Down
2 changes: 1 addition & 1 deletion src/etaoin/api.clj
Original file line number Diff line number Diff line change
Expand Up @@ -2867,7 +2867,7 @@
(vector? q-text)
(if (even? (count q-text))
(doseq [[q text] (partition 2 q-text)]
(fill driver q text))
(fill-human driver q text opts))
(throw+ {:type :etaoin/argument
:message "Vector q-text must have even length"
:q-text q-text
Expand Down

0 comments on commit eb0c0ad

Please sign in to comment.