Skip to content

Commit

Permalink
Merge pull request #390 from tmcgilchrist/core_fixes
Browse files Browse the repository at this point in the history
Update JaneStreet core and async.
  • Loading branch information
tmcgilchrist committed Jul 4, 2023
2 parents e13f8b9 + 167c6fb commit 7c997d1
Show file tree
Hide file tree
Showing 7 changed files with 29 additions and 16 deletions.
16 changes: 14 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,9 +18,8 @@ jobs:
runtest:
- true
ocaml-compiler:
- 4.12.x
- 4.13.x
- 4.14.x
- 5.0.x
include:
- os: macos-latest
ocaml-compiler: 4.14.x
Expand Down Expand Up @@ -56,6 +55,19 @@ jobs:
packages: 'alcotest alcotest-js alcotest-lwt alcotest-mirage'
opam-local-packages: 'alcotest.opam alcotest-js.opam alcotest-lwt.opam alcotest-mirage.opam'
runtest: false

- os: ubuntu-latest
ocaml-compiler: 4.12.x
packages: 'alcotest alcotest-js alcotest-lwt alcotest-mirage'
opam-local-packages: 'alcotest.opam alcotest-js.opam alcotest-lwt.opam alcotest-mirage.opam'
runtest: false

- os: ubuntu-latest
ocaml-compiler: 4.13.x
packages: 'alcotest alcotest-js alcotest-lwt alcotest-mirage'
opam-local-packages: 'alcotest.opam alcotest-js.opam alcotest-lwt.opam alcotest-mirage.opam'
runtest: false



runs-on: ${{ matrix.os }}
Expand Down
1 change: 1 addition & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

- Add `match_raises`, a generalized version of `check_raises`
(#88, #386, @JoanThibault)
- Update JaneStreet core and async to v0.16 (#390 @tmcgilchrist)

### 1.7.0 (2023-02-24)

Expand Down
10 changes: 5 additions & 5 deletions alcotest-async.opam
Original file line number Diff line number Diff line change
Expand Up @@ -13,14 +13,14 @@ depends: [
"re" {with-test}
"fmt" {with-test}
"cmdliner" {with-test & >= "1.2.0"}
"core" {>= "v0.15.0"}
"core_unix" {>= "v0.15.0"}
"core" {>= "v0.16.0"}
"core_unix" {>= "v0.16.0"}
"base"
"async_kernel"
"ocaml" {>= "4.11.0"}
"ocaml" {>= "4.14.0"}
"alcotest" {= version}
"async" {>= "v0.15.0"}
"async_unix" {>= "v0.15.0"}
"async" {>= "v0.16.0"}
"async_unix" {>= "v0.16.0"}
"odoc" {with-doc}
]
build: [
Expand Down
10 changes: 5 additions & 5 deletions dune-project
Original file line number Diff line number Diff line change
Expand Up @@ -45,14 +45,14 @@ tests to run.
(re :with-test)
(fmt :with-test)
(cmdliner (and :with-test (>= 1.2.0)))
(core (>= v0.15.0))
(core_unix (>= v0.15.0))
(core (>= v0.16.0))
(core_unix (>= v0.16.0))
base
async_kernel
(ocaml (>= 4.11.0))
(ocaml (>= 4.14.0))
(alcotest (= :version))
(async (>= v0.15.0))
(async_unix (>= v0.15.0))))
(async (>= v0.16.0))
(async_unix (>= v0.16.0))))

(package
(name alcotest-lwt)
Expand Down
4 changes: 2 additions & 2 deletions src/alcotest-async/alcotest_async.ml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ let run_test timeout name fn args =
| `Timeout ->
Alcotest.fail
(Printf.sprintf "%s timed out after %s" name
(Time_unix.Span.to_string_hum timeout))
(Time_float_unix.Span.to_string_hum timeout))

module Promise = struct
include Deferred
Expand All @@ -24,7 +24,7 @@ module V1 = struct

let test_case_sync n s f = test_case n s (fun x -> Deferred.return (f x))

let test_case ?(timeout = Time_unix.Span.of_sec 2.) name s f =
let test_case ?(timeout = Time_float_unix.Span.of_sec 2.) name s f =
test_case name s (run_test timeout name f)
end

Expand Down
2 changes: 1 addition & 1 deletion src/alcotest-async/alcotest_async_intf.ml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module type V1 = sig
include Alcotest_engine.V1.Cli.S with type return = unit Async.Deferred.t

val test_case :
?timeout:Time_unix.Span.t ->
?timeout:Time_float_unix.Span.t ->
string ->
Alcotest.speed_level ->
('a -> unit Async.Deferred.t) ->
Expand Down
2 changes: 1 addition & 1 deletion src/alcotest-async/dune
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@
async_unix
base
core
core_unix.time_unix))
core_unix.time_float_unix))

0 comments on commit 7c997d1

Please sign in to comment.