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

Bump for 2.2.2 #49

Merged
merged 7 commits into from
Feb 23, 2021
Merged

Bump for 2.2.2 #49

merged 7 commits into from
Feb 23, 2021

Conversation

dylanjha
Copy link
Contributor

@dylanjha dylanjha commented Feb 23, 2021

Includes fix for #30 from PR #43

Diff: v2.2.1...master

This failed on a test run: https://github.com/muxinc/upchunk/pull/49/checks?check_run_id=1957508634

This commit doesn't fix the race condition, but it makes it less likely
to happen :/ not ideal, but we'll go with it

```
* FAIL src/upchunk.spec.ts
  ● Console

    console.error
      Error: Uncaught [Error: expect(received).toBeTruthy()

      Received: false]
          at reportException (/home/runner/work/upchunk/upchunk/node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:62:24)
          at Timeout.task [as _onTimeout] (/home/runner/work/upchunk/upchunk/node_modules/jsdom/lib/jsdom/browser/Window.js:396:9)
          at listOnTimeout (internal/timers.js:554:17)
          at processTimers (internal/timers.js:497:7) JestAssertionError: expect(received).toBeTruthy()

      Received: false
          at /home/runner/work/upchunk/upchunk/src/upchunk.spec.ts:315:30
          at Timeout.task [as _onTimeout] (/home/runner/work/upchunk/upchunk/node_modules/jsdom/lib/jsdom/browser/Window.js:391:19)
          at listOnTimeout (internal/timers.js:554:17)
          at processTimers (internal/timers.js:497:7) {
        matcherResult: { message: [Function: message], pass: false }
      }

      at VirtualConsole.<anonymous> (node_modules/jsdom/lib/jsdom/virtual-console.js:29:45)
      at reportException (node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:66:28)
      at Timeout.task [as _onTimeout] (node_modules/jsdom/lib/jsdom/browser/Window.js:396:9)
```
src/upchunk.spec.ts Outdated Show resolved Hide resolved
@dylanjha dylanjha requested a review from mmcc February 23, 2021 01:54
src/upchunk.spec.ts Outdated Show resolved Hide resolved
@dylanjha dylanjha merged commit e6475ab into master Feb 23, 2021
@dylanjha dylanjha deleted the dj/2-2-2 branch February 23, 2021 05:02
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.

None yet

2 participants