unsupported types should result in error tests failing on Safari and on Edge / Windows 10

[aninde]

Description

After running the tests runner on does not pass 1 test on

• Safari v.10.15 / macOs Catalina 10.15.4 (latest)
• QQ Browser Lite v.1.2.9 / macOs Catalina 10.15.4
• Safari / iPad osX 13.3.1.
• Edge v.44.17763.831.0 / Windows 10
• Sogou v.8.6.3.32395 / Windows 10

Failed test:
unsupported types should result in error > should give parsing error # 11
Expected function to throw an exception with message 'Unable to parse value: "BigInt(9007199254740991)"', but it threw an exception with message 'Can't find variable: BigInt'.

the test also fails if it is released independently.

I have a problem reproducing it on iPhone 6s, probably by releasing a new version of iPhone SE. I will try to reproduce this using BrowserStack

[aninde]

On newer ✅Edge Chromium (Edge Devs) all tests pass on Windows 10.

[wojciechczerniak]

BigInt is not always implemented. We should make the support optional, and check if it exists.

Maybe it's in the test only but we need to verify the production code @izulin

[aninde]

The same bug appears in the browser Sogou v. 8.6.3.32395 / Windows 10. Sogou can only be installed on Windows. No tests were done on MacOS.

[aninde]

I have updated the list in the main description because the error also occurs on QQ browser / MacOs.

[aninde]

unsupported types should result in error > should give parsing error # 11

hyperformula/test/unsupported-types.spec.ts

Line 63 in c9d7d87

it('should give parsing error #11', () => {

[izulin]

unsupported types should result in error > should give parsing error # 11

hyperformula/test/unsupported-types.spec.ts

Line 63 in c9d7d87

it('should give parsing error #11', () => {

Thanks!

[aninde]

BigInt is not supported by Internet Explorer and Safari :( the rest of the browsers listed above can be based on the IE-engine, so I wouldn't count on having support for this, so this is not our bug.

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/BigInt

[wojciechczerniak]

@aninde Should be fixed in #338. Can you re-test develop for this issue?

[aninde]

I checked on all browsers mentioned above. Is fixed.