@suffolklitlab/alkiln 5.9.0-feat-url2 → 5.9.0-feat-url3

package/lib/scope.js

@@ -1498,6 +1498,8 @@ module.exports = {
     }
 
     let handle = await scope.get_handle_from_field( scope, { field: field });
+    // TODO: expect elem to exist
+    expect(handle).to.exist;  // I don't think this is sufficient. Maybe in function called above instead
 
     // After each previous input, figure out if this input is (now/still) disabled
     // TODO: Consider testing for disabled this before we get the matching row
@@ -1506,7 +1508,17 @@ module.exports = {
       // sometimes it blocks nodes from clicks. Scroll item to the bottom instead.
       // Note: I'm not sure how we'd get in here if `elem` didn't exist, but
       // I once got an error here that I couldn't read `.disabled` of `null`.
-      if ( elem && !elem.disabled ) { elem.scrollIntoView(false); }
+
+      // if ( elem && !elem.disabled ) {
+      //   elem.scrollIntoView(false);
+      //   return elem.disabled;
+      // } else {
+      //   return false;
+      // }
+
+      if ( elem && !elem.disabled ) {
+        elem.scrollIntoView(false);
+      }
       return elem.disabled;
     });
 
@@ -1539,7 +1551,9 @@ module.exports = {
 
   get_handle_from_field: async function ( scope, { field }) {
     /** Get the element's puppeteer handle using the field's selector. */
-
+    if ( session_vars.get_debug() ) {
+      console.log(`scope.get_handle_from_field() field.selector:`, field.selector );
+    }
     let handle = await scope.page.evaluateHandle(( selector ) => {
       let elem = document.querySelector( selector );
       // If the element has a sibling that's a label, we need to

package/package.json

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