test(ngOptions): fix flaky test on Firefox 54+

Author: gkalpak

test/ng/directive/ngOptionsSpec.js

@@ -2931,15 +2931,54 @@ describe('ngOptions', function() {
       var _selected = [];
 
       // Set up spies
+      var optionProto = Object.getPrototypeOf(options[0]);
+      var originalSelectedDescriptor = isFunction(Object.getOwnPropertyDescriptor) &&
+                                       Object.getOwnPropertyDescriptor(optionProto, 'selected');
+      var addSpiesOnProto = originalSelectedDescriptor && originalSelectedDescriptor.configurable;
+
       forEach(options, function(option, i) {
-        optionsSetSelected[i] = jasmine.createSpy('optionSetSelected' + i);
-        _selected[i] = option.selected;
-        Object.defineProperty(option, 'selected', {
-          get: function() { return _selected[i]; },
-          set: optionsSetSelected[i].and.callFake(function(value) { _selected[i] = value; })
-        });
+        var setSelected = function(value) { _selected[i] = value; };
+        optionsSetSelected[i] = jasmine.createSpy('optionSetSelected' + i).and.callFake(setSelected);
+        setSelected(option.selected);
       });
 
+      if (!addSpiesOnProto) {
+        forEach(options, function(option, i) {
+          Object.defineProperty(option, 'selected', {
+            get: function() { return _selected[i]; },
+            set: optionsSetSelected[i]
+          });
+        });
+      } else {
+        // Support: Firefox 54+
+        // We cannot use the above (simpler) method on all browsers because of Firefox 54+, which
+        // is very flaky when the getter/setter property is defined on the element itself and not
+        // the prototype. (Possibly the result of some (buggy?) optimization.)
+        var getSelected = function(index) { return _selected[index]; };
+        var setSelected = function(index, value) { optionsSetSelected[index](value); };
+        var getSelectedOriginal = function(option) {
+          return originalSelectedDescriptor.get.call(option);
+        };
+        var setSelectedOriginal = function(option, value) {
+          originalSelectedDescriptor.set.call(option, value);
+        };
+        var getIndexAndCall = function(option, foundFn, notFoundFn, value) {
+          for (var i = 0, ii = options.length; i < ii; ++i) {
+            if (options[i] === option) return foundFn(i, value);
+          }
+          return notFoundFn(option, value);
+        };
+
+        Object.defineProperty(optionProto, 'selected', {
+          get: function() {
+            return getIndexAndCall(this, getSelected, getSelectedOriginal);
+          },
+          set: function(value) {
+            return getIndexAndCall(this, setSelected, setSelectedOriginal, value);
+          }
+        });
+      }
+
       // Select `optionA`
       scope.$apply('selected = [values[0]]');
 
@@ -2999,6 +3038,12 @@ describe('ngOptions', function() {
       expect(options[1].selected).toBe(false);
       optionsSetSelected[0].calls.reset();
       optionsSetSelected[1].calls.reset();
+
+      // Support: Firefox 54+
+      // Restore `originalSelectedDescriptor`
+      if (addSpiesOnProto) {
+        Object.defineProperty(optionProto, 'selected', originalSelectedDescriptor);
+      }
     });
 
     if (window.MutationObserver) {