Add test coverage for setAttribute update.

facebook · Aug 2, 2016 · 1b097ff · 1b097ff
1 parent b9ccffa
commit 1b097ff
Showing 1 changed file with 12 additions and 0 deletions.
diff --git a/src/renderers/dom/shared/__tests__/DOMPropertyOperations-test.js b/src/renderers/dom/shared/__tests__/DOMPropertyOperations-test.js
@@ -346,6 +346,18 @@ describe('DOMPropertyOperations', function() {
       expect(stubNode.className).toBe('');
     });
 
+    it('should not update numeric values when the input.value is loosely the same', function() {
+      stubNode.setAttribute('type', 'number');
+      stubNode.setAttribute('value', '3e1');
+      // Keep in sync. The comparison occurs between setAttribute and value.
+      stubNode.value = '3e1';
+
+      spyOn(stubNode, 'setAttribute');
+
+      DOMPropertyOperations.setValueForProperty(stubNode, 'value', 30);
+
+      expect(stubNode.setAttribute.calls.count()).toBe(0);
+    });
   });
 
   describe('deleteValueForProperty', function() {