Skip to content

Commit

Permalink
test: increase no-errors integration timeout to 20s (#425)
Browse files Browse the repository at this point in the history
- this was still occassionally failing on CI at 15s, so bump up a bit
  - c.f. https://github.com/ezolenko/rollup-plugin-typescript2/actions/runs/3075482590/jobs/4968935300, https://github.com/ezolenko/rollup-plugin-typescript2/actions/runs/3038898669/jobs/4893183507, https://github.com/ezolenko/rollup-plugin-typescript2/actions/runs/3038898669/jobs/4893561438, and many more
  - only `no-errors` was timing out, which makes sense, since it does 3 builds for the first test to test against the cache
    - this potentially could be optimized, but it started getting complicated to do so (including some concurrency issues)
      - so punting on that for now and just increasing the timeout
  • Loading branch information
agilgur5 committed Sep 20, 2022
1 parent ffb562d commit a1ae42b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions __tests__/integration/no-errors.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ import { RPT2Options } from "../../src/index";
import { filesArr } from "./fixtures/no-errors";
import { findName, genBundle as genBundleH } from "./helpers";

// increase timeout to 15s for whole file since CI occassionally timed out -- these are integration and cache tests, so longer timeout is warranted
jest.setTimeout(15000);
// increase timeout to 20s for whole file since CI occassionally timed out -- these are integration and cache tests, so longer timeout is warranted
jest.setTimeout(20000);

const local = (x: string) => path.resolve(__dirname, x);
const testDir = local("__temp/no-errors");
Expand Down

0 comments on commit a1ae42b

Please sign in to comment.