929 Commits

Author SHA1 Message Date
Eric
817c2bc82d
remove unneeded exception from start raises 2025-06-05 15:58:09 +10:00
Eric
e1cc23d4fd
Add "starting test" log to multinodes logging output for context 2025-06-05 15:58:08 +10:00
Eric
a8c573e8aa
unittest2 > unittest1 2025-06-05 15:58:08 +10:00
Eric
a3db38d779
remove noCancel from node startup in multinodes 2025-06-05 15:58:08 +10:00
Eric
41eca535e4
Remove noCancel from captureProcessOutput call 2025-06-05 15:58:08 +10:00
Eric
114a46afa2
Continuously read from stdout/stderr to prevent buffers from filling 2025-06-05 15:58:08 +10:00
Eric
a9b094ae86
reinstate logging after it also failed on windows and ci 2025-06-05 15:58:08 +10:00
Eric
a52a53ba52
disable all logging to see if windows passes 2025-06-05 15:58:07 +10:00
Eric
9084dd7b14
Move test assertions at end of start so that test.status can be set after timeout 2025-06-05 15:58:07 +10:00
Eric
4f2f000a93
fix rebase on top of async http client change 2025-06-05 15:58:07 +10:00
Eric
3ec35dc788
Remove debug from twonodesconfig 2025-06-05 15:58:07 +10:00
Eric
6bef231297
Add wait to stdout/stderr stream future 2025-06-05 15:58:07 +10:00
Eric
daa23f4ffb
fix windows issue where reading stdout/stderr hangs forever 2025-06-05 15:58:07 +10:00
Eric
9d8dbee9aa
ensure runTests doesn't raise CancelledError
TrackedFutures now only accepts Futures that do not raise any exceptions
2025-06-05 15:58:06 +10:00
Eric
f93112dc28
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-06-05 15:58:06 +10:00
Eric
4da5c8c372
do not re-raise CancelledError as convertError not used in async procs 2025-06-05 15:58:06 +10:00
Eric
fa772f7ac6
re-raise CancelledError 2025-06-05 15:58:06 +10:00
Eric
139aa6eef8
readd prevention of hanging when closing process streams on windows 2025-06-05 15:58:06 +10:00
Eric
e7a9adab21
Add debugging and assertions after test run
Mostly to debug in windows
2025-06-05 15:58:06 +10:00
Eric
f4b78f048f
CI yml fix for gcc-14 2025-06-05 15:58:05 +10:00
Eric
f3901d13b1
Update NBS action.yml in CI 2025-06-05 15:58:05 +10:00
Eric
67719ee395
formatting 2025-06-05 15:58:05 +10:00
Eric
e648954eea
formatting 2025-06-05 15:58:05 +10:00
Eric
357840db4d
Fix process hanging after restart 2025-06-05 15:58:05 +10:00
Eric
eb77643405
refactor: debug and logging
- Move test manager values to config object
- Increase codex port separation (between api and disc ports) in an attempt to prevent overlap across tests (ie Test1: api=8000(tcp), disc=9000(udp), and Test2: api=9000(tcp), disc=10000(udp))
- print stderr when exitcode == 1 if there's < 3 lines of stdout
- Logging:
  - Always write test manager harness chronicles logs to file, ie testmanager.chronicles.log in the root of the `integration/logs/<run name>` dir
  - Always write individual test stdout to file, ie `<test file name>.stdout.log` in the root of the `integration/logs/<run name>/<test file name>` dir
  - On error, print stderr to screen and write stderr to file. Or on failure, if stdout is sufficiently short, write stderr to screen and file in `integration/logs/<run name>/<test file name>/<test file name>.stderr.log`
  - When debugging, ie DebugCodexNodes == true
- Removes DebugTestHarness from controlling anything other than printing chronicles output from the testmanager to the terminal
- Now, if DebugCodexNodes is set to true:
  - Codex node (chronicles) output for multinodesuite tests is logged to file, eg `integration/logs/<run name>/<test file name>/<test name>/<role>_<idx>.log`
  - Codex chronicles output is logged to stdout, which also written to file (see above)
2025-06-05 15:58:05 +10:00
Eric
0a4369d0cd
enable more logging to file
- test harness chronicles logs to log file
- test process stdout and stderr to log file for each integration test file

Note: does not work when DEBUG=1 is not present in make command
2025-06-05 15:58:04 +10:00
Eric
595e1d3fdb
Ensure started tests are torn down properly on timeout 2025-06-05 15:58:04 +10:00
Eric
b9e3e07dc1
filter out log topics that are making too much noise
- set disabled topics during codex compilation due to lack of support for disabling chronicles topics at runtime in codex
- ensure that TRACE logs are set for nodes running in multinodesuite when DebugCodexNodes is set
- add TRACE loglevel to testcli
2025-06-05 15:58:04 +10:00
Slava
494a3b9244
ci: check runner for parallel integration tests and use value from vars 2025-06-05 15:58:04 +10:00
Eric
40c243a080
Colorise stdout from test runs
Adds terminal colour to [OK], [FAILED], and [Suite]

# Conflicts:
#	tests/integration/testmanager.nim
2025-06-05 15:58:04 +10:00
Eric
72d2931183
Refactor process launch
Refactor to keep a process handle so that the processes can be stopped on timeout.
2025-06-05 15:58:04 +10:00
Eric
bc5fdd49be
run tests async from start so that stop can cancel all tracked futures 2025-06-05 15:58:03 +10:00
Eric
fb2e3436f6
Add proper async exception tracking to multinodesuite 2025-06-05 15:58:03 +10:00
Eric
592e7c1e27
bump chronos to 4.0.4
potentially fixes an issue on macos where process exit codes were reported -1
2025-06-05 15:58:03 +10:00
Eric
c3558e1e04
formatting 2025-06-05 15:58:03 +10:00
Eric
8257ec29b8
CI: enable artefact upload for parallel tests 2025-06-05 15:58:03 +10:00
Eric
23875b875d
run npm install for parallel tests in ci 2025-06-05 15:58:03 +10:00
Eric
22cdacb0fe
Allow debug runs from the github UI 2025-06-05 15:58:03 +10:00
Eric
cddb407ddc
clean up 2025-06-05 15:58:02 +10:00
Eric
c59fa9f1e5
increase port gap by 1000 to avoid clashes in windows 2025-06-05 15:58:02 +10:00
Eric
aa6678b8ff
lock codex port selection in multinodesuites
Codex api/disc ports were clashing in windows because `lastUsedCodexApiPort` and `lastUsedCodeexDiscPort` were not locked when read and written, while client and provider nodes were started concurrently (async).
2025-06-05 15:58:02 +10:00
Eric
5333856083
add test name to datadir path
avoids reusing datadirs between tests
2025-06-05 15:58:02 +10:00
Eric
25ce97e81e
update Make flag to simply debug
DEBUG -> enables DebugCodexNodes, DebugTestHarness, and ShowContinuousStatusUpdates
DEBUG_HARDHAT -> enables DebugHardhat
PARALLEL -> enables EnableParallelTests

Additionally, when DEBUG is enabled, all integration tests debug configs are enabled for Codex nodes, the Codex node output is printed with the test output (not interleaved), and the Codex node output is logged to file in `tests/integrations/logs/<starttime>__IntegrationTests/<integration_test_name>/<suite_name>/<testname>/<role>_<idx>.log`.
When DEBUG_HARDHAT is enabled, all hardhat output is printed with the test output (not interleaved), and the output is also written to a log file in `tests/integrations/logs/<starttime>__IntegrationTests/<integration_test_name>/hardhat.log
2025-06-05 15:58:02 +10:00
Eric
29df100e93
cancel processes soon
Windows really does not like to terminate processes, so if we try to cancel the process, we will hang in Windows.
2025-06-05 15:58:01 +10:00
Eric
92ca6de95b
fix: hardhot not being stopped when test times out 2025-06-05 15:58:01 +10:00
Eric
7ea21795c9
Allow parallel integration tests to be disabled
Disable with `make ENABLE_PARALLEL_TESTS=0 testIntegration`
2025-06-05 15:58:01 +10:00
Eric
44f99f5d39
fix: use the last used port instead of the globally defined port 2025-06-05 15:58:01 +10:00
Eric
42701ee54e
Increase hardhat, codex api, and codex discovery ports for each subsequent test
Previously, each test would start with a port specified by a global const and overwritable using `define` cli flag. This seems to be causing some port clashes that `nextFreePort` is not picking up. Instead of relying on `nextFreePort`, do not reuse port numbers for any of the tests.
2025-06-05 15:58:01 +10:00
Eric
fb39f12fe4
enable debug logs for testpurchasing since the failure in ci 2025-06-05 15:58:01 +10:00
Eric
680430ef81
swap linux large runner 2025-06-05 15:58:00 +10:00