Make improvements to "Recorder Mode"

Author: mdmintz

help_docs/recorder_mode.md

@@ -1,4 +1,4 @@
-[<img src="https://seleniumbase.io/cdn/img/super_logo_sb.png" title="SeleniumBase" width="296">](https://github.com/seleniumbase/SeleniumBase/blob/master/README.md)
+[<img src="https://seleniumbase.io/cdn/img/sb_logo_10t.png" title="SeleniumBase" width="260">](https://github.com/seleniumbase/SeleniumBase/blob/master/README.md)
 
 <h2><img src="https://seleniumbase.io/img/logo6.png" title="SeleniumBase" width="32" /> Recorder Mode</h2>
 
@@ -47,6 +47,8 @@ class RecorderTest(BaseCase):
 
 <p>🔴 The launch of Recorder Mode has brought a new SeleniumBase method along with it: <code>self.open_if_not_url(URL)</code>. This method will open the URL given if the browser is not currently on that page. This is used as a method in recorded scripts when SeleniumBase detects that a click action has already brought the test to the given page. This method not only prevents an extra page load if not needed, but it also lets people know the current page of the browser at that point in the test.</p>
 
+<p>🔴 SeleniumBase <code>1.66.1</code> adds the ability to record changes to <i>"Choose File"</i> <code>input</code> fields. It also adds the <code>self.set_content_to_frame(frame)</code> method, which lets you record actions inside of iframes on pages. Sometimes the <i>"Choose File"</i> input field is hidden on some sites, so <code>self.show_file_choosers()</code> was also added to get around this edge case.
+
 --------
 
 <div>To learn more about SeleniumBase, check out the Docs Site:</div>

seleniumbase/extensions/recorder.zip

@@ -2955,6 +2955,20 @@ def __process_recorded_actions(self):
                 elif '"' in action[1] and '"' in action[2]:
                     sb_actions.append("self.set_value('%s', '%s')" % (
                         action[1], action[2]))
+            elif action[0] == "cho_f":
+                action[2] = action[2].replace("\\", "\\\\")
+                if '"' not in action[1] and '"' not in action[2]:
+                    sb_actions.append('self.choose_file("%s", "%s")' % (
+                        action[1], action[2]))
+                elif '"' not in action[1] and '"' in action[2]:
+                    sb_actions.append('self.choose_file("%s", \'%s\')' % (
+                        action[1], action[2]))
+                elif '"' in action[1] and '"' not in action[2]:
+                    sb_actions.append('self.choose_file(\'%s\', "%s")' % (
+                        action[1], action[2]))
+                elif '"' in action[1] and '"' in action[2]:
+                    sb_actions.append("self.choose_file('%s', '%s')" % (
+                        action[1], action[2]))
             elif action[0] == "c_box":
                 cb_method = "check_if_unchecked"
                 if action[2] == "no":

seleniumbase/fixtures/base_case.py

@@ -97,6 +97,9 @@
     non_id_attributes.push('data-test');
     non_id_attributes.push('data-test-id');
     non_id_attributes.push('data-test-selector');
+    non_id_attributes.push('data-nav');
+    non_id_attributes.push('data-action');
+    non_id_attributes.push('data-target');
     non_id_attributes.push('alt');
     non_id_attributes.push('title');
     non_id_attributes.push('heading');
@@ -315,6 +318,17 @@
         value = element.value;
         document.recorded_actions.push(['set_v', selector, value, d_now]);
     }
+    else if (tag_name === 'input' && element.type === 'file')
+    {
+        if (ra_len > 0 &&
+            document.recorded_actions[ra_len-1][1] === selector)
+        {
+            document.recorded_actions.pop();
+            ra_len = document.recorded_actions.length;
+        }
+        value = element.value;
+        document.recorded_actions.push(['cho_f', selector, value, d_now]);
+    }
     else if (ra_len > 0 &&
         document.recorded_actions[ra_len-1][1] === selector &&
         tag_name === 'input' && element.type === 'checkbox')