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.

Sign up for GitHub

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

Add an integration test for canton-repl #12772

Merged
merged 14 commits into from
Feb 8, 2022
Merged

Add an integration test for canton-repl #12772

Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
b87b263
Add an integration test for canton-repl
sofiafaro-da Feb 4, 2022
d815aeb
Use script to run canton-repl
sofiafaro-da Feb 7, 2022
2784e0f
buildifier, add comment
sofiafaro-da Feb 7, 2022
162ba1c
remove script_dev_env
sofiafaro-da Feb 7, 2022
5138a26
.
sofiafaro-da Feb 7, 2022
0ab8c39
.
sofiafaro-da Feb 7, 2022
e24a63d
apparently unixtools is bad so don’t use all of it
cocreature Feb 7, 2022
26c2ca1
fix test
sofiafaro-da Feb 7, 2022
bcf56db
the -- is necessary
sofiafaro-da Feb 7, 2022
f32e317
.
sofiafaro-da Feb 7, 2022
d579941
fix invocation of script on linux
sofiafaro-da Feb 8, 2022
06821c5
need exit on linux
sofiafaro-da Feb 8, 2022
5a9416c
remove debugs and document the jank
sofiafaro-da Feb 8, 2022
3c360a3
.
sofiafaro-da Feb 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
.
  • Loading branch information
@sofiafaro-da
sofiafaro-da committed Feb 7, 2022
commit 0ab8c398e92170494d2d664e4a0ccdf365fcda8f
8 changes: 6 additions & 2 deletions daml-assistant/integration-tests/src/DA/Daml/Assistant/IntegrationTests.hs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -810,17 +810,21 @@ cantonTests = testGroup "daml sandbox"
let cmd = unwords
-- NOTE (Sofia): We need `script` on Linux and Mac because of this ammonite issue:
-- https://github.com/com-lihaoyi/Ammonite/issues/276
[ (if isWindows then "" else "script -q -- tty.txt ") <> "daml canton-repl"
[ "daml canton-repl"
, "--port", show ledgerApiPort
, "--admin-api-port", show adminApiPort
, "--domain-public-port", show domainPublicApiPort
, "--domain-admin-port", show domainAdminApiPort
]
wrappedCmd
| isWindows = cmd
| isMac = concat ["script -q tty.txt ", cmd]
| otherwise = concat ["script --quiet --command '", cmd, "'"]
input = unlines
[ "sandbox.health.running"
, "local.health.running"
]
proc' = (shell cmd) { cwd = Just dir }
proc' = (shell wrappedCmd) { cwd = Just dir }
output <- readCreateProcess proc' input
hPutStrLn stderr "canton-repl output:"
hPutStrLn stderr output
Expand Down