Extract time inputs to seperate test file

The test inputs load a data attribute for an SVG icon. This is changing the count of network requests in unrelated tests.

Author: Antsiscool

spec/features/session_spec.rb

@@ -162,110 +162,121 @@
     end
 
     describe "Node#set" do
-      before do
-        @session.visit("/cuprite/with_js")
-        @session.find(:css, "#change_me").set("Hello!")
-      end
+      context "with_js" do
+        before do
+          @session.visit("/cuprite/with_js")
+          @session.find(:css, "#change_me").set("Hello!")
+        end
 
-      it "fires the change event" do
-        expect(@session.find(:css, "#changes").text).to eq("Hello!")
-      end
+        it "fires the change event" do
+          expect(@session.find(:css, "#changes").text).to eq("Hello!")
+        end
 
-      it "fires the input event" do
-        expect(@session.find(:css, "#changes_on_input").text).to eq("Hello!")
-      end
+        it "fires the input event" do
+          expect(@session.find(:css, "#changes_on_input").text).to eq("Hello!")
+        end
 
-      it "accepts numbers in a maxlength field" do
-        element = @session.find(:css, "#change_me_maxlength")
-        element.set 100
-        expect(element.value).to eq("100")
-      end
+        it "accepts numbers in a maxlength field" do
+          element = @session.find(:css, "#change_me_maxlength")
+          element.set 100
+          expect(element.value).to eq("100")
+        end
 
-      it "accepts negatives in a number field" do
-        element = @session.find(:css, "#change_me_number")
-        element.set(-100)
-        expect(element.value).to eq("-100")
-      end
+        it "accepts negatives in a number field" do
+          element = @session.find(:css, "#change_me_number")
+          element.set(-100)
+          expect(element.value).to eq("-100")
+        end
 
-      it "fires the keydown event" do
-        expect(@session.find(:css, "#changes_on_keydown").text).to eq("6")
-      end
+        it "fires the keydown event" do
+          expect(@session.find(:css, "#changes_on_keydown").text).to eq("6")
+        end
 
-      it "fires the keyup event" do
-        expect(@session.find(:css, "#changes_on_keyup").text).to eq("6")
-      end
+        it "fires the keyup event" do
+          expect(@session.find(:css, "#changes_on_keyup").text).to eq("6")
+        end
 
-      it "fires the keypress event" do
-        expect(@session.find(:css, "#changes_on_keypress").text).to eq("6")
-      end
+        it "fires the keypress event" do
+          expect(@session.find(:css, "#changes_on_keypress").text).to eq("6")
+        end
 
-      it "fires the focus event" do
-        expect(@session.find(:css, "#changes_on_focus").text).to eq("Focus")
-      end
+        it "fires the focus event" do
+          expect(@session.find(:css, "#changes_on_focus").text).to eq("Focus")
+        end
 
-      it "fires the blur event" do
-        expect(@session.find(:css, "#changes_on_blur").text).to eq("Blur")
-      end
+        it "fires the blur event" do
+          expect(@session.find(:css, "#changes_on_blur").text).to eq("Blur")
+        end
 
-      it "fires the keydown event before the value is updated" do
-        expect(@session.find(:css, "#value_on_keydown").text).to eq("Hello")
-      end
+        it "fires the keydown event before the value is updated" do
+          expect(@session.find(:css, "#value_on_keydown").text).to eq("Hello")
+        end
 
-      it "fires the keyup event after the value is updated" do
-        expect(@session.find(:css, "#value_on_keyup").text).to eq("Hello!")
-      end
+        it "fires the keyup event after the value is updated" do
+          expect(@session.find(:css, "#value_on_keyup").text).to eq("Hello!")
+        end
 
-      it "clears the input before setting the new value" do
-        element = @session.find(:css, "#change_me")
-        element.set ""
-        expect(element.value).to eq("")
-      end
+        it "clears the input before setting the new value" do
+          element = @session.find(:css, "#change_me")
+          element.set ""
+          expect(element.value).to eq("")
+        end
 
-      it "supports special characters" do
-        element = @session.find(:css, "#change_me")
-        element.set "$52.00"
-        expect(element.value).to eq("$52.00")
-      end
+        it "supports special characters" do
+          element = @session.find(:css, "#change_me")
+          element.set "$52.00"
+          expect(element.value).to eq("$52.00")
+        end
 
-      it "attaches a file when passed a Pathname" do
-        filename = Pathname.new("spec/tmp/a_test_pathname").expand_path
-        File.write(filename, "text")
+        it "attaches a file when passed a Pathname" do
+          filename = Pathname.new("spec/tmp/a_test_pathname").expand_path
+          File.write(filename, "text")
 
-        element = @session.find(:css, "#change_me_file")
-        element.set(filename)
-        expect(element.value).to eq("C:\\fakepath\\a_test_pathname")
-      ensure
-        FileUtils.rm_f(filename)
-      end
+          element = @session.find(:css, "#change_me_file")
+          element.set(filename)
+          expect(element.value).to eq("C:\\fakepath\\a_test_pathname")
+        ensure
+          FileUtils.rm_f(filename)
+        end
 
-      it "sets a value for a color input" do
-        element = @session.find(:css, "#change_me_color")
-        element.set("#ddeeff")
-        expect(element.value).to eq("#ddeeff")
+        it "sets a value for a color input" do
+          element = @session.find(:css, "#change_me_color")
+          element.set("#ddeeff")
+          expect(element.value).to eq("#ddeeff")
+        end
       end
 
-      it "sets a value for a time input" do
-        element = @session.find(:css, "#change_me_time")
-        element.set("17:21")
-        expect(element.value).to eq("17:21")
-      end
+      # The time inputs are loading SVG icons as data: urls.
+      # This is causing other unrelated tests to break.
+      # Keep the time inputs (And others with SVG icons) in their own file
+      context "time_inputs" do
+        before(:each) do
+          @session.visit("/cuprite/time_inputs")
+        end
 
-      it "sets a value for a time input with a time object" do
-        element = @session.find(:css, "#change_me_time")
-        element.set(Time.new(2023, 9, 26, 17, 21))
-        expect(element.value).to eq("17:21")
-      end
+        it "sets a value for a time input" do
+          element = @session.find(:css, "#change_me_time")
+          element.set("17:21")
+          expect(element.value).to eq("17:21")
+        end
 
-      it "sets a value for a date input" do
-        element = @session.find(:css, "#change_me_date")
-        element.set("2023-09-26")
-        expect(element.value).to eq("2023-09-26")
-      end
+        it "sets a value for a time input with a time object" do
+          element = @session.find(:css, "#change_me_time")
+          element.set(Time.new(2023, 9, 26, 17, 21))
+          expect(element.value).to eq("17:21")
+        end
 
-      it "sets a value for a date input with a date object" do
-        element = @session.find(:css, "#change_me_date")
-        element.set(Date.new(2023, 9, 26))
-        expect(element.value).to eq("2023-09-26")
+        it "sets a value for a date input" do
+          element = @session.find(:css, "#change_me_date")
+          element.set("2023-09-26")
+          expect(element.value).to eq("2023-09-26")
+        end
+
+        it "sets a value for a date input with a date object" do
+          element = @session.find(:css, "#change_me_date")
+          element.set(Date.new(2023, 9, 26))
+          expect(element.value).to eq("2023-09-26")
+        end
       end
     end
 

spec/support/views/time_inputs.erb

@@ -0,0 +1,13 @@
+<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
+  <head>
+    <meta http-equiv="Content-type" content="text/html; charset=utf-8"/>
+    <title>cuprite with_js</title>
+    <script src="/cuprite/jquery.min.js" type="text/javascript" charset="utf-8"></script>
+    <script src="/cuprite/jquery-ui.min.js" type="text/javascript" charset="utf-8"></script>
+    <script src="/cuprite/test.js" type="text/javascript" charset="utf-8"></script>
+  </head>
+  <body>
+    <p><input type="time" name="change_me_time" id="change_me_time"></p>
+    <p><input type="date" name="change_me_date" id="change_me_date"></p>
+  </body>
+</html>

spec/support/views/with_js.erb

@@ -39,8 +39,6 @@
     </p>
     <p><input type="file" name="change_me_file" id="change_me_file"></p>
     <p><input type="color" name="change_me_color" id="change_me_color"></p>
-    <p><input type="time" name="change_me_time" id="change_me_time"></p>
-    <p><input type="date" name="change_me_date" id="change_me_date"></p>
     <p id="changes"></p>
     <p id="changes_on_input"></p>
     <p id="changes_on_keydown"></p>