From 7e966819b59d968366a9df50bfaa7c7abd36d61e Mon Sep 17 00:00:00 2001
From: eldritch horrors <pennae@lix.systems>
Date: Mon, 4 Mar 2024 03:49:09 +0100
Subject: Merge pull request #9098 from obsidiansystems/test-proto

Enable most of the third `BuildResult` worker protocol test

(cherry picked from commit d344c112f772282bacacd4c66a75df4022d16e12)
Change-Id: I7b2b72aa84c19a6069f9c12128d901262db6f91c
---
 tests/unit/libstore/worker-protocol.cc | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'tests/unit/libstore/worker-protocol.cc')

diff --git a/tests/unit/libstore/worker-protocol.cc b/tests/unit/libstore/worker-protocol.cc
index 24a06503c..fa7cbe121 100644
--- a/tests/unit/libstore/worker-protocol.cc
+++ b/tests/unit/libstore/worker-protocol.cc
@@ -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;
     }))
-- 
cgit v1.2.3