988 Commits

Author SHA1 Message Date
Eric
16f882dacb
cache bust in an attempt to prevent illegal instruction issues 2025-11-07 14:30:17 +11:00
Eric
3400e17012
re-add new line at EOF 2025-11-06 20:18:26 +11:00
Eric
a12dc6ea64
remove testmanager
Not needed for now since parallel integration tests aren't enabled
2025-11-05 18:05:35 +11:00
Eric
5ff0886840
Updates after rebase 2025-11-05 18:00:43 +11:00
Slava
72c8222d73
ci: use macos-14-xlarge for macos integration tests
Signed-off-by: Slava <20563034+veaceslavdoina@users.noreply.github.com>
2025-11-04 21:13:22 +11:00
Eric
83aa74ed13
Update ci.yml to be in line with master 2025-11-04 21:05:17 +11:00
Eric
562da50dda
disable ccache symlink
Permission denied creating the symlink in CI
2025-11-04 21:05:17 +11:00
Eric
0a6921b9e6
update to websockets addresses for hardhat connection
Also, catch exceptions on JsonRpcProvider.close, which should not need to be done, as this routine should not raise any exceptions.
2025-11-04 21:05:17 +11:00
Eric
e4e3830745
revert posix-only verbose unittest output
It didn't fix the error with invalid proofs in windows in ci
2025-11-04 21:05:16 +11:00
Eric
52c0482295
fix previous commit compilation 2025-11-04 21:05:16 +11:00
Eric
889d4ae67f
try removing verbose output to see if windows ci passes 2025-11-04 21:05:16 +11:00
Eric
e5182cb3c4
logging: show test setup/teardown in logs 2025-11-04 21:05:16 +11:00
Eric
331350c441
try increasing polling interval to see if this has an effect on windows in ci 2025-11-04 21:05:16 +11:00
Eric
e732efa61a
verbose unittest2 output to stdout 2025-11-04 21:05:16 +11:00
Eric
9ab59ab075
update after rebase 2025-11-04 21:05:15 +11:00
Eric
c5c3a04f67
re-enable all CI tests
Also adds a condition such that a hardhat node is not started for parallel integration tests
2025-11-04 21:05:15 +11:00
Eric
6b091767a9
formatting 2025-11-04 21:04:56 +11:00
Eric
603110b990
fix compilation errors in windows 2025-11-04 21:04:56 +11:00
Eric
59e9196edc
fix eventually symbol resolution
Because `eventuallySafe` calls the symbol `eventually`, it should be declared before `proc eventually` is declared to avoid ambiguous symbol lookups.
2025-11-04 21:04:55 +11:00
Eric
9e0d151c6b
force kill windows test processes
In situations like timeouts, windows will hang when attempting to close the test process streams. In this case, we have to force kill the test process externally. This is the same process as force killing hardhat nodes after they are already terminated, but windows refuses hangs when closing their process streams. This commit creates a forceKillProcess utility that allows a process to be killed by its process name and matching commandline criteria, like TestId (for test process) or --port (for hardhat)
2025-11-04 20:59:57 +11:00
Eric
e970989205
Reinstate unittest > unittest2, move eventually to ./asynctest 2025-11-04 20:59:57 +11:00
Eric
50bbf8ac31
Don't store hardhat logs in memory and do not print to term even if DebugHardhat=true
Hardhat output is logged to file in hardhat.log for each test, and printing to screen is not necessarily needed as it is already logged to file and can create clutter in the output, so stop writing logging output to memory and writing to screen.
2025-11-04 20:59:57 +11:00
Eric
3d66c01ed6
re-enable debug logging in codex nodes
Since the HttpClient now supports async, re-enable debug logging in the Codex nodes
2025-11-04 20:59:56 +11:00
Eric
a652c8dedb
fix makefile: DEBUG enables DebugHardhat 2025-11-04 20:59:56 +11:00
Eric
c4459f2322
Remove new implementation of eventuallySafe by overidding the default params of eventually
With this fix in, there is no need to use the asynctest update that sets longer defaults for eventually, so downgrade asynctest
2025-11-04 20:59:56 +11:00
Eric
866d754de6
safeEventually > eventuallySafe 2025-11-04 20:59:48 +11:00
Eric
93832a6ef1
fix: forcefully kill windows hardhat processes after termination
On windows, termination of hardhat processes would not actually kill the process, and then closing the process' streams would then hang the calling nim process. To get around this, the process is now killed externally using a script, winkillhardhat.sh. This script first queries open processes by inspecting the command line value of all "node.exe" processes, searching for "vendor/codex-contracts-eth" and for the port parameter it was started with. After querying, the process is killed using the `Stop-Process` powershell command (passing the pid of the windows process).
2025-11-04 20:18:44 +11:00
Arnaud
1663bd224f
Reduce the number of requests for asserts in tests 2025-11-04 20:18:44 +11:00
Eric
98b42f4319
formatting 2025-11-04 20:16:05 +11:00
Eric
498c0c42cb
attempt to kill windows hardhat process
Windows hangs when attempting to hardhat's process streams, so try to kill the process externally.
TODO: This doesn't actually kill the process, as the pid given by chronos is an msys2 pid, and the command is used to kill a windows process. `ps -ef` in msys2 also doesn't show hardhat running as a process, so the only way to kill the process is to kill it with the windows pid. So we need to figure out a way to get a windows pid from the msys2 pid.
2025-11-04 20:16:05 +11:00
Eric
98f3ff76e0
add new line to stdout/stderr logs when logging lines 2025-11-04 20:16:05 +11:00
Eric
05fae04db6
fix compilation error 2025-11-04 20:16:04 +11:00
Eric
c93694ac36
write to stdout/stderr as the stream is read, rearrange node stop logging 2025-11-04 20:16:04 +11:00
Eric
58b70a5ee3
remove unneeded exception from start raises 2025-11-04 20:16:04 +11:00
Eric
2659913c2d
Add "starting test" log to multinodes logging output for context 2025-11-04 20:16:04 +11:00
Eric
dc3ea9b16d
unittest2 > unittest1 2025-11-04 20:16:04 +11:00
Eric
4e3673edd2
remove noCancel from node startup in multinodes 2025-11-04 20:16:04 +11:00
Eric
389ce830d9
Remove noCancel from captureProcessOutput call 2025-11-04 20:16:03 +11:00
Eric
c1b36e7f21
Continuously read from stdout/stderr to prevent buffers from filling 2025-11-04 20:16:03 +11:00
Eric
e86f596133
reinstate logging after it also failed on windows and ci 2025-11-04 20:16:03 +11:00
Eric
2d8783f49e
disable all logging to see if windows passes 2025-11-04 20:16:03 +11:00
Eric
c22cc10d48
Move test assertions at end of start so that test.status can be set after timeout 2025-11-04 20:16:03 +11:00
Eric
f3997dd95e
fix rebase on top of async http client change 2025-11-04 20:16:03 +11:00
Eric
7acb05d5b1
Remove debug from twonodesconfig 2025-11-04 20:16:03 +11:00
Eric
828fcf6edb
Add wait to stdout/stderr stream future 2025-11-04 20:16:02 +11:00
Eric
8eb5013b5a
fix windows issue where reading stdout/stderr hangs forever 2025-11-04 20:16:02 +11:00
Eric
6d1b924c41
ensure runTests doesn't raise CancelledError
TrackedFutures now only accepts Futures that do not raise any exceptions
2025-11-04 20:16:02 +11:00
Eric
338ee181d3
Remove DebugCodexNodes
- Codex nodes will log to file by default, so this parameter could be removed.
- Add NoCodexLogFilters define, that unsets chronicles' topic filters when logging codex nodes chronicles logs to file
2025-11-04 20:16:02 +11:00
Eric
6057d5ac45
do not re-raise CancelledError as convertError not used in async procs 2025-11-04 20:16:02 +11:00
Eric
2695b46dc4
re-raise CancelledError 2025-11-04 20:16:02 +11:00