address comment

Author: timotheecour

testament/lib/stdtest/testutils.nim

@@ -36,8 +36,9 @@ proc greedyOrderedSubsetLines*(lhs, rhs: string): bool =
       return false
   return true
 
-const enableRemoteNetworking* = getEnv("NIM_TESTAMENT_REMOTE_NETWORKING") == "1"
+template enableRemoteNetworking*: bool =
   ## Allows contolling whether to run some test at a statement-level granularity.
   ## Using environment variables simplifies propagating this all the way across
   ## process calls, e.g. `testament all` calls itself, which in turns invokes
   ## a `nim` invocation (possibly via additional intermediate processes).
+  getEnv("NIM_TESTAMENT_REMOTE_NETWORKING") == "1"

testament/testament.nim

@@ -54,7 +54,7 @@ On Azure Pipelines, testament will also publish test results via Azure Pipelines
 provided that System.AccessToken is made available via the environment variable SYSTEM_ACCESSTOKEN.
 
 Experimental: using environment variable `NIM_TESTAMENT_REMOTE_NETWORKING=1` enables
-tests with remote networking (as in CI). It is accessed when tests are compiled.
+tests with remote networking (as in CI).
 """ % resultsFile
 
 type