max/nix-super
1
0
Fork 0
mirror of https://github.com/privatevoid-net/nix-super.git synced 2024-11-26 07:46:21 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #9098 from obsidiansystems/test-proto

Browse source 
Enable most of the third `BuildResult` worker protocol test
This commit is contained in:
John Ericson 2023-10-04 22:44:22 -04:00 committed by GitHub
commit d344c112f7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/libstore/tests/worker-protocol.cc
View file

@ -187,7 +187,7 @@ CHARACTERIZATION_TEST(
"build-result",
({
using namespace std::literals::chrono_literals;
std::tuple<BuildResult, BuildResult/*, BuildResult*/> t {
std::tuple<BuildResult, BuildResult, BuildResult> t {
BuildResult {
.status = BuildResult::OutputRejected,
.errorMsg = "no idea why",
@ -200,9 +200,6 @@ CHARACTERIZATION_TEST(
.startTime = 30,
.stopTime = 50,
},
#if 0
// This is commented because this test would fail!
// FIXME uncomment this and fix the underlying bug.
BuildResult {
.status = BuildResult::Built,
.timesBuilt = 1,
@ -230,10 +227,14 @@ CHARACTERIZATION_TEST(
},
.startTime = 30,
.stopTime = 50,
#if 0
// These fields are not yet serialized.
// FIXME Include in next version of protocol or document
// why they are skipped.
.cpuUser = std::chrono::milliseconds(500s),
.cpuSystem = std::chrono::milliseconds(604s),
},
#endif
},
};
t;
}))

BIN
unit-test-data/libstore/worker-protocol/build-result.bin
View file

Binary file not shown.