Skip to content

Add Eio_mock.Clock#328

Merged
talex5 merged 1 commit intoocaml-multicore:mainfrom
talex5:mock-clock
Sep 27, 2022
Merged

Add Eio_mock.Clock#328
talex5 merged 1 commit intoocaml-multicore:mainfrom
talex5:mock-clock

Conversation

@talex5
Copy link
Copy Markdown
Collaborator

@talex5 talex5 commented Sep 26, 2022

This adds a mock clock that can be used for testing time-related functions, such as the timeout handling in #302 (see https://github.com/talex5/eio/pull/new/with_connect for a draft updated version of that).

I also moved the signature for Time from eio.mli to time.mli, so that other Eio modules can refer to the type in future.

/cc @bikallem

@talex5 talex5 force-pushed the mock-clock branch 2 times, most recently from 49a7a32 to 5301f11 Compare September 26, 2022 12:59
@talex5 talex5 merged commit 3af12b9 into ocaml-multicore:main Sep 27, 2022
@talex5 talex5 deleted the mock-clock branch September 27, 2022 08:03
talex5 added a commit to talex5/opam-repository that referenced this pull request Oct 12, 2022
CHANGES:

Changes:

- Update to OCaml 5.0.0~beta1 (@anmonteiro @talex5 ocaml-multicore/eio#346).

- Add API for seekable read/writes (@nojb ocaml-multicore/eio#307).

- Add `Flow.write` (@haesbaert ocaml-multicore/eio#318).
  This provides an optimised alternative to `copy` in the case where you are writing from a buffer.

- Add `Net.with_tcp_connect` (@bikallem ocaml-multicore/eio#302).
  Convenience function for opening a TCP connection.

- Add `Eio.Time.Timeout` (@talex5 ocaml-multicore/eio#320).
  Makes it easier to pass timeouts around.

- Add `Eio_mock.Clock` (@talex5 ocaml-multicore/eio#328).
  Control time in tests.

- Add `Buf_read.take_while1` and `skip_while1` (@bikallem ocaml-multicore/eio#309).
  These fail if no characters match.

- Make the type parameter for `Promise.t` covariant (@anmonteiro ocaml-multicore/eio#300).

- Move list functions into a dedicated submodule (@raphael-proust ocaml-multicore/eio#315).

- Direct implementation of `Flow.source_string` (@c-cube ocaml-multicore/eio#317).
  Slightly faster.

Bug fixes:

- `Condition.broadcast` must interlock as well (@haesbaert ocaml-multicore/eio#324).

- Split the reads into no more than 2^32-1 for luv (@haesbaert @talex5 @EduardoRFS ocaml-multicore/eio#343).
  Luv uses a 32 bit int for buffer sizes and wraps if the value passed is too big.

- eio_luv: allow `Net.connect` to be cancelled (@talex5 @nojb ocaml-multicore/eio#311).

- eio_main: Use dedicated log source (@anmonteiro ocaml-multicore/eio#326).

- linux_eio: fix kernel version number in log message (@talex5 @nojb ocaml-multicore/eio#314).

- Account for stack differences in the socketpair test (issue ocaml-multicore/eio#312) (@haesbaert ocaml-multicore/eio#313).

Documentation:

- Add Best Practices section to README (@talex5 ocaml-multicore/eio#299).

- Documentation improvements (@talex5 ocaml-multicore/eio#295 ocaml-multicore/eio#337).
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.

1 participant