delete baseBalanceOf

[scott-silver]

No description provided.

[kevincheng96]

👍

[scott-silver]

This PR has a funny element to it; one of the flaky tests from #319 fails every time we run the Coverage job:

  1) accrue
       accrue initially succeeds and has the right parameters:
     AssertionError: expected 1650575142 to be close to 1650575224.854 +/- 80
      at Context.<anonymous> (test/accrue-test.ts:38:44)

The unit tests pass, but this flaky test always fails.

I'm fairly certain if I merge #319 and rebase this PR, it should pass the Coverage job. But I'm somewhat weirded out that a test could always pass when run with yarn test and always fail when run with yarn cover

[jflatow]

This PR has a funny element to it; one of the flaky tests from #319 fails every time we run the Coverage job:

  1) accrue
       accrue initially succeeds and has the right parameters:
     AssertionError: expected 1650575142 to be close to 1650575224.854 +/- 80
      at Context.<anonymous> (test/accrue-test.ts:38:44)

The unit tests pass, but this flaky test always fails.

I'm fairly certain if I merge #319 and rebase this PR, it should pass the Coverage job. But I'm somewhat weirded out that a test could always pass when run with yarn test and always fail when run with yarn cover

Knowing that coverage tests run slower does it still weird you out?

[scott-silver]

@jflatow nice, that makes sense