Skip to content

Commit

Permalink
chore(fix-test-helpers): fix Symbol.observable refernce for Safari 8 (#…
Browse files Browse the repository at this point in the history
…2109)

This fix is related to saucelabs tests failign in Safari 8.
The suspicion is that it is a problem with the fact a line of code was
using `Symbol.observable` directly instead of `56526symbolObservable`
since we're no longer polyfilling `Symbol.observable` unless `Symbol`
exists properly. Might also be related to `symbol-observable` dependency.
  • Loading branch information
benlesh authored and jayphelps committed Nov 5, 2016
1 parent a047e7a commit 65ebf18
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions spec/helpers/test-helper.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
declare const global: any;
declare const Symbol: any;

import * as Rx from '../../dist/cjs/Rx';
import {ObservableInput} from '../../dist/cjs/Observable';
Expand All @@ -19,7 +18,7 @@ export function lowerCaseO<T>(...args): Rx.Observable<T> {
}
};

o[Symbol.observable] = function () {
o[$$symbolObservable] = function () {
return this;
};

Expand Down

0 comments on commit 65ebf18

Please sign in to comment.