@suffolklitlab/alkiln 5.10.0 → 5.10.2

package/CHANGELOG.md

@@ -45,6 +45,18 @@ Format:
 
 <!-- ## [Unreleased] -->
 
+## [5.10.2] - 2024-02-29
+
+### Fixed
+
+- Fixed dates only being part-way filled.
+
+## [5.10.1] - 2024-02-23
+
+### Changed
+
+- Fixes the issue that typing is too fast for some fields, like Ajax comboboxes. See [#857](https://github.com/SuffolkLITLab/ALKiln/issues/857).
+
 ## [5.10.0] - 2024-02-22
 
 ### Added
@@ -62,7 +74,7 @@ Format:
 
 - Handle non-existence of `elem` more robustly when checking for `elem.disabled`
 - Add a pause between pull request tests and push tests when the PR is coming from our own repo
-- Make getting page field values more robust by doing more to ensure that undefined values don't cause errors
+- Make getting page field values more robust by doing more to ensure that undefined option `value`s don't cause errors. For example, ajax combobox `select` options don't always have `value`s
 
 ## [5.9.0] - 2024-02-11
 

package/lib/scope.js

@@ -1796,6 +1796,7 @@ module.exports = {
         await scope.setText( scope, { handle, answer: answer_date });
         await handle.press(`Tab`);
         await scope.afterStep(scope, { waitForShowIf: true });
+        
       } else if (type == `time`) {
         // Authors should enter the time like 12:34 PM, but we shouldn't type ":", space, or M.
         let answer_time = answer.replace(":", "").replace(" ", "").replace("M", "");
@@ -1864,7 +1865,6 @@ module.exports = {
                      valid_date.getDate().toString().padStart(2, 0) + '/' +
                      valid_date.getFullYear().toString());
     }
-
     return valid_date;
   },  // Ends scope.today_to_date()
 
@@ -1975,12 +1975,34 @@ module.exports = {
   },  // Ends scope.setCheckbox()
 
   setText: async function ( scope, { handle, answer }) {
-    // Set text in some kind of field (input text, input date, textarea, etc.)
-    await handle.evaluate( el => { el.value = '' });
+    /** Set text in some kind of field (input text, input date, textarea, etc.) */
+    const typing_cutoff = 300;
+    const slice_value = -3;
+
+    let is_ajax = await handle.evaluate(( elem, text, typing_cutoff, slice_value ) => {
+      // For long strings or ajax comboboxes, set the value of the field to
+      // most of the text. This will be faster in those cases.
+      is_ajax = elem.className.includes(`da-ajax-combobox`);
+
+      if ( is_ajax || text.length > typing_cutoff ) {
+        elem.value = answer.slice(0, slice_value);  // .slice(0, -3)
+      }
+
+      return is_ajax;
+    }, answer, typing_cutoff, slice_value );
+
+    // Interact like a "human" in all cases.
     await handle.focus();
-    let delay = 0;
-    if ( session_vars.get_debug() ) { delay = 25; }
-    await handle.type( answer, { delay });
+
+    // For long strings or ajax comboboxes, where we have set most of the text
+    // already, add a good pause for the field to detect the interaction.
+    if ( is_ajax || answer.length > typing_cutoff ) {
+      const to_type = answer.slice( slice_value );  // .slice(-3)
+      await handle.type( to_type, { delay: 100 });
+    } else {
+      // Otherwise type the whole string with 0 delay
+      await handle.type( answer );
+    }
   },
 
   draw_signature: async function ( scope, name ) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@suffolklitlab/alkiln",
-  "version": "5.10.0",
+  "version": "5.10.2",
   "description": "Integrated automated end-to-end testing with docassemble, puppeteer, and cucumber.",
   "main": "lib/index.js",
   "scripts": {