nix-output-monitor icon indicating copy to clipboard operation
nix-output-monitor copied to clipboard

Fix tests on pure aarch64-linux systems

Open SuperSandro2000 opened this issue 1 year ago • 6 comments

Tested by deploying the config on an aarch64-linux without an x86_64-linux remote builders (in fact without any remote builders at all).

SuperSandro2000 avatar Dec 17 '23 00:12 SuperSandro2000

That was only evaluation though, not sure why the checks fail:

Running phase: checkPhase
Running 2 test suites...
Test suite golden-tests: RUNNING...
### Failure in: 0:golden test standard for old-style messages with log from file
test/Golden.hs:124
No running builds
### Failure in: 1:golden test standard for json messages with log from file
test/Golden.hs:124
No running builds
Cases: 4  Tried: 4  Errors: 0  Failures: 2
Test suite golden-tests: FAIL
Test suite logged to: dist/test/nix-output-monitor-2.1.1-golden-tests.log
Test suite unit-tests: RUNNING...
Cases: 6  Tried: 6  Errors: 0  Failures: 0
Test suite unit-tests: PASS
Test suite logged to: dist/test/nix-output-monitor-2.1.1-unit-tests.log
1 of 2 test suites (1 of 2 test cases) passed.

SuperSandro2000 avatar Dec 17 '23 03:12 SuperSandro2000

I don’t think this makes sense. The golden tests depend on the hashes of the derivations in questions. So those tests can’t work on other platforms.

maralorn avatar Jan 24 '24 21:01 maralorn

I suggest disabling these checks on all systems other than x86_64-linux.

maralorn avatar Jan 24 '24 21:01 maralorn

I suggest disabling these checks on all systems other than x86_64-linux.

Changed that. Not sure how to convince cabal to not run the tests though.

SuperSandro2000 avatar Jan 27 '24 19:01 SuperSandro2000

I suggest

doCheck = system == "x86_64-linux";

directly in the line above preCheck.

maralorn avatar Jan 28 '24 18:01 maralorn

changed that

SuperSandro2000 avatar Feb 10 '24 17:02 SuperSandro2000

Great, thanks!

maralorn avatar Feb 26 '24 10:02 maralorn