Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add IE11 partitions to travis config #776

Merged
merged 4 commits into from
Dec 16, 2022
Merged

Conversation

abachman
Copy link
Contributor

Problem

We're seeing errors on BrowserStack during the IE11 portion of the unit test suite.

image

It's not clear why it's happening, but it seems to be consistent. Maybe out of memory?

If it's a memory error, maybe splitting the suite into more partitions will prevent the error?

Solution

Added (slightly) more flexible partitioning to test/unit/support/init.js and utilized the new partitions in .travis.yml.

@coveralls
Copy link

coveralls commented Nov 15, 2022

Coverage Status

Coverage remained the same at 92.635% when pulling 040bc37 on splitting-ie-tests into 3920bfe on master.

@chrissrogers chrissrogers merged commit a1e6ba2 into master Dec 16, 2022
@chrissrogers chrissrogers deleted the splitting-ie-tests branch December 16, 2022 21:15
recurly-integrations pushed a commit that referenced this pull request Jan 12, 2023
[Full Changelog](v4.22.7...v4.22.8)

**Merged Pull Requests**

- Include tiered addon math [#781](#781) ([ELepolt](https://github.com/ELepolt))
- Add IE11 partitions to travis config [#776](#776) ([abachman](https://github.com/abachman))

**Closed Issues**

- Apple Pay via Braintree causes CSP violation [#767](#767)

##### Minified MD5 Checksum
```
17c13a7eef17b6ed9b1d0137561cf092 ./build/recurly.min.js
```
##### [SRI Hash](https://www.srihash.org/)
```
4nlU1rUnX2BNywbz37DXl4Dw4umOsJ1tsUluT5iHfQpI/FZdLJxlU+2G65wXnRq4
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants