@voxgig/sdkgen 4.34.0 → 4.34.1

package/project/.sdk/src/cmp/csharp/ReadmeQuick_csharp.ts

@@ -68,8 +68,8 @@ var client = ${ctor};
     if (opnames.includes('list')) {
       Content(`### 2. List ${eName.toLowerCase()} records
 
-\`List(null)\` returns an aggregate list of records (as \`object?\`) and raises
-on error.
+\`List(null)\` returns a list of entities, one per record (as \`object?\`), and
+raises on error.
 
 \`\`\`csharp
 try
@@ -108,7 +108,8 @@ catch (Exception err)
       Content(`### 3. Load ${neArticle} ${neName.toLowerCase()}
 
 ${neName} is nested under ${parentName}, so provide the \`${parentParam}\`.
-\`Load()\` returns the bare record (as \`object?\`) and raises on error.
+\`Load()\` returns the entity (as \`object?\`) and raises on error; an entity's
+\`Data()\` reads its record.
 
 \`\`\`csharp
 try
@@ -138,7 +139,8 @@ catch (Exception err)
 
       Content(`### 3. Load ${article} ${eName.toLowerCase()}
 
-\`Load()\` returns the bare record (as \`object?\`) and raises on error.
+\`Load()\` returns the entity (as \`object?\`) and raises on error; an entity's
+\`Data()\` reads its record.
 
 \`\`\`csharp
 try
@@ -174,13 +176,14 @@ catch (Exception err)
       return it && it.type
     }
 
-    if (opnames.includes('create') || opnames.includes('update') || opnames.includes('remove')) {
+    if (opnames.includes('create') || opnames.includes('update') || opnames.includes('patch') ||
+      opnames.includes('remove')) {
       Content(`### 4. Create, update, and remove
 
 \`\`\`csharp
 `)
       if (opnames.includes('create')) {
-        Content(`// Create — returns the bare created record (as object?)
+        Content(`// Create — returns the created entity (as object?)
 var created = client.${eName}().Create(new Dictionary<string, object?> { ${examplePairs('create').join(', ')} });
 
 `)
@@ -191,6 +194,14 @@ var created = client.${eName}().Create(new Dictionary<string, object?> { ${examp
         Content(`// Update — supply the id in the match/data
 client.${eName}().Update(new Dictionary<string, object?> { ${updatePairs.join(', ')} });
 
+`)
+      }
+      if (opnames.includes('patch')) {
+        const patchPairs = (idF ? [`["${idF}"] = ${csLit(idParamType('patch'), 'example_id')}`] : [])
+          .concat(examplePairs('patch'))
+        Content(`// Patch — sends only the fields given
+client.${eName}().Patch(new Dictionary<string, object?> { ${patchPairs.join(', ')} });
+
 `)
       }
       if (opnames.includes('remove')) {

package/project/.sdk/src/cmp/csharp/ReadmeRef_csharp.ts

@@ -1,5 +1,5 @@
 
-import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape , targetFeatures } from '@voxgig/sdkgen'
+import { cmp, each, Content, canonToType, canonKey, canonScalarKey, File, isAuthActive, entityIdField, opRequestShape , targetFeatures, opNeedsAction, bodyNote } from '@voxgig/sdkgen'
 import { ReadmeRefFeatures } from '@voxgig/sdkgen'
 
 import {
@@ -27,28 +27,33 @@ function csLit(type: any, placeholder: string = 'example'): string {
 const OP_SIGNATURES: Record<string, { sig: string, returns: string, desc: string }> = {
   load: {
     sig: 'Load(reqmatch, ctrl = null) -> object?',
-    returns: 'the entity data',
-    desc: 'Load a single entity matching the given criteria. Returns the entity data and raises on error.',
+    returns: 'the entity',
+    desc: 'Load a single entity matching the given criteria. Returns the entity, whose record `Data()` reads, and raises on error.',
   },
   list: {
     sig: 'List(reqmatch, ctrl = null) -> object?',
-    returns: 'an aggregate list of entities',
-    desc: 'List entities matching the given criteria. The match is optional — call `List(null)` to list all records. Returns an aggregate list and raises on error.',
+    returns: 'a list of entities, one per record',
+    desc: 'List entities matching the given criteria. The match is optional — call `List(null)` to list all records. Returns a list of entities, one per record, and raises on error.',
   },
   create: {
     sig: 'Create(reqdata, ctrl = null) -> object?',
-    returns: 'the created entity data',
-    desc: 'Create a new entity with the given data. Returns the created entity data and raises on error.',
+    returns: 'the created entity',
+    desc: 'Create a new entity with the given data. Returns the created entity and raises on error.',
   },
   update: {
     sig: 'Update(reqdata, ctrl = null) -> object?',
-    returns: 'the updated entity data',
-    desc: 'Update an existing entity. The data must include the entity `id`. Returns the updated entity data and raises on error.',
+    returns: 'the updated entity',
+    desc: 'Update an existing entity. The data must include the entity `id`. Returns the updated entity and raises on error.',
+  },
+  patch: {
+    sig: 'Patch(reqdata, ctrl = null) -> object?',
+    returns: 'the patched entity data',
+    desc: 'Change part of an existing entity: only the fields given are sent. The data must include the entity `id`. Returns the patched entity data and raises on error.',
   },
   remove: {
     sig: 'Remove(reqmatch, ctrl = null) -> object?',
-    returns: 'the removed entity data',
-    desc: 'Remove the entity matching the given criteria. Raises on error.',
+    returns: 'the removed entity',
+    desc: 'Remove the entity matching the given criteria. Returns the entity, marked as deleted, and raises on error.',
   },
 }
 
@@ -217,7 +222,7 @@ var ${eVar} = client.${ent.Name}();
         // Field operations breakdown
         const hasFieldOps = fields.some((f: any) => f.op && Object.keys(f.op).length > 0)
         if (hasFieldOps) {
-          const opcols = ['load', 'list', 'create', 'update', 'remove']
+          const opcols = ['load', 'list', 'create', 'update', 'patch', 'remove']
             .filter((op: string) => opnames.includes(op) && ent.op[op]?.active !== false)
           Content(`### Field Usage by Operation
 
@@ -258,6 +263,10 @@ ${info.desc}
 
 `)
 
+          if (opNeedsAction(ent.op[opname])) {
+            return
+          }
+
           if ('load' === opname || 'remove' === opname) {
             const matchItems = opRequestShape(ent, opname).items
               .filter((it: any) => !it.optional || it.name === idF)
@@ -298,8 +307,8 @@ var result = client.${ent.Name}().Create(new Dictionary<string, object?>
 
 `)
           }
-          else if ('update' === opname) {
-            const updateItems = opRequestShape(ent, 'update').items
+          else if ('update' === opname || 'patch' === opname) {
+            const updateItems = opRequestShape(ent, opname).items
               .filter((it: any) => !it.optional || it.name === idF)
               .sort((a: any, b: any) =>
                 (a.name === idF ? 0 : 1) - (b.name === idF ? 0 : 1))
@@ -307,14 +316,22 @@ var result = client.${ent.Name}().Create(new Dictionary<string, object?>
               `    ["${it.name}"] = ${csLit(it.type,
                 it.name === idF ? ent.name + '_id' : it.name)},\n`).join('')
             Content(`\`\`\`csharp
-var result = client.${ent.Name}().Update(new Dictionary<string, object?>
+var result = client.${ent.Name}().${'patch' === opname ? 'Patch' : 'Update'}(new Dictionary<string, object?>
 {
-${updateLines}    // Fields to update
+${updateLines}    // ${'patch' === opname ? 'Only the fields to change' : 'Fields to update'}
 });
 \`\`\`
 
 `)
           }
+
+          if ('create' === opname || 'update' === opname || 'patch' === opname) {
+            const note = bodyNote(ent.op[opname], {
+              values: 'a `byte[]`, a `string` or a `Stream`',
+              once: 'a `Stream`',
+            })
+            if ('' !== note) Content(note)
+          }
         })
       }
 

package/project/.sdk/src/cmp/csharp/ReadmeTopQuick_csharp.ts

@@ -72,7 +72,7 @@ Console.WriteLine(${eVar}List);
             it.name === idF ? 'example_id' : 'example_' + it.name)}`).join(',')} }`
         : 'null'
       Content(`
-// Load a specific ${eName.toLowerCase()} (returns the record, raises on error)
+// Load a specific ${eName.toLowerCase()} (returns the entity, raises on error)
 var ${eVar} = client.${eName}().Load(${loadArg});
 Console.WriteLine(${eVar});
 `)

package/project/.sdk/src/cmp/csharp/ReadmeTopTest_csharp.ts

@@ -52,7 +52,7 @@ var client = ${model.const.Name}SDK.TestSDK(null, null);
         ? `new Dictionary<string, object?> {${items.map((it: any) =>
           ` ["${it.name}"] = ${it.name === idF ? '"test01"' : csLit(it.type)}`).join(',')} }`
         : 'null'
-    } else if ('create' === primaryOp || 'update' === primaryOp) {
+    } else if ('create' === primaryOp || 'update' === primaryOp || 'patch' === primaryOp) {
       const items = opRequestShape(exampleEntity, primaryOp).items
         .filter((it: any) => it.name !== idF && it.name !== 'id')
       const required = items.filter((it: any) => !it.optional)

package/project/.sdk/src/cmp/csharp/TestClean_csharp.ts

@@ -8,6 +8,7 @@ import {
   isHttpBasicAuth,
   resolveAuthIn,
   resolveAuthName,
+  targetFeatures,
 } from '@voxgig/sdkgen'
 
 
@@ -27,6 +28,10 @@ const TestClean = cmp(function TestClean(props: any) {
     basic: isHttpBasicAuth(model),
   }
 
+  // CostRecord is declared by the cost feature's source, which ships only
+  // when the model selects the feature.
+  const cost = null != targetFeatures(model, target).cost
+
   // Same order the ts sweep tries: list, then load, then the rest.
   const rank: Record<string, number> = { list: 0, load: 1 }
   const candidates = each(entityCollection(model))
@@ -39,7 +44,7 @@ const TestClean = cmp(function TestClean(props: any) {
     }))
     .filter((c: any) => 0 < c.ops.length)
 
-  File({ name: 'CleanTest.' + target.ext }, () => Content(render(model.const.Name, auth, candidates)))
+  File({ name: 'CleanTest.' + target.ext }, () => Content(render(model.const.Name, auth, candidates, cost)))
 })
 
 
@@ -47,6 +52,7 @@ function render(
   Name: string,
   auth: { suppressed: boolean, where: string, name: string, basic: boolean },
   candidates: { name: string, Name: string, ops: string[] }[],
+  cost: boolean,
 ): string {
   const candidateLines = candidates.map((c) =>
     `        new Candidate("${c.name}", sdk => sdk.${c.Name}(null), new[] { ${c.ops.map((o) => `"${o}"`).join(', ')} }),`)
@@ -370,8 +376,34 @@ public class CleanTest
         }),
     };
 
+    // Offline, as every generated suite is: the test OPTION resolves a
+    // required server variable to test-<name>, and installs no transport.
+    private static Dictionary<string, object?> Offline(Dictionary<string, object?> opts)
+    {
+        var copy = new Dictionary<string, object?>(opts);
+        copy["test"] = new Dictionary<string, object?> { ["active"] = true };
+        return copy;
+    }
+
+    // A client the sweep cannot build leaves nothing swept: a harness error,
+    // not a leak.
+    private static ${Name}SDK Construct(Dictionary<string, object?> opts)
+    {
+        try
+        {
+            return new ${Name}SDK(Offline(opts));
+        }
+        catch (Exception e)
+        {
+            throw new InvalidOperationException(
+                "clean harness: the client could not be constructed, so nothing was swept: " +
+                e.Message, e);
+        }
+    }
+
     private static ${Name}SDK MakeSdk(Scenario scenario, List<Sink> sinks,
-        Dictionary<string, object?>? cleanopts = null, BaseFeature? extra = null)
+        Dictionary<string, object?>? cleanopts = null, BaseFeature? extra = null,
+        Dictionary<string, object?>? auth = null)
     {
         Action<Dictionary<string, object?>> Capture(string name) =>
             rec => sinks.AddRange(FormsOf(name, rec));
@@ -407,7 +439,7 @@ public class CleanTest
                 ["active"] = true, ["exporter"] = Capture("telemetry"),
             };
         }
-        if (Fh.HasFeature("cost"))
+${cost ? `        if (Fh.HasFeature("cost"))
         {
             feature["cost"] = new Dictionary<string, object?>
             {
@@ -415,7 +447,7 @@ public class CleanTest
                 ["sink"] = (Action<CostRecord>)(rec => sinks.AddRange(FormsOf("cost", rec))),
             };
         }
-        if (Fh.HasFeature("metrics"))
+` : ''}        if (Fh.HasFeature("metrics"))
         {
             feature["metrics"] = new Dictionary<string, object?> { ["active"] = true };
         }
@@ -439,7 +471,7 @@ public class CleanTest
             extend.Add(extra);
         }
 
-        return new ${Name}SDK(new Dictionary<string, object?>
+        var opts = new Dictionary<string, object?>
         {
             ["apikey"] = CanaryApikey,
             ["secret"] = CanarySecret,
@@ -448,7 +480,12 @@ public class CleanTest
             ["feature"] = feature,
             ["extend"] = extend,
             ["utility"] = new Dictionary<string, object?> { ["fetcher"] = fetcher },
-        });
+        };
+        if (null != auth)
+        {
+            opts["auth"] = auth;
+        }
+        return Construct(opts);
     }
 
     // Emitted from the model: every active entity with the operations it
@@ -468,6 +505,7 @@ ${candidateLines}
             "load" => ent.Load(args, ctrl),
             "create" => ent.Create(args, ctrl),
             "update" => ent.Update(args, ctrl),
+            "patch" => ent.Patch(args, ctrl),
             "remove" => ent.Remove(args, ctrl),
             _ => throw new InvalidOperationException("unknown operation: " + op),
         };
@@ -499,7 +537,7 @@ ${candidateLines}
     {
         var fetcher = (Context _ctx, string _url, Dictionary<string, object?> _def) =>
             (object?)Response(200, new Dictionary<string, object?> { ["id"] = "i1" });
-        var plain = new ${Name}SDK(new Dictionary<string, object?>
+        var plain = Construct(new Dictionary<string, object?>
         {
             ["apikey"] = CanaryApikey,
             ["utility"] = new Dictionary<string, object?> { ["fetcher"] = fetcher },
@@ -532,11 +570,12 @@ ${candidateLines}
     {
         // A caller may keep the record it passed rather than read ctrl's entry.
         var held = ctrl.GetValueOrDefault("explain");
+        var entity = target.Candidate.Accessor(sdk);
         object? out_ = null;
         Exception? err = null;
         try
         {
-            out_ = Invoke(target.Candidate.Accessor(sdk), target.Op, target.Match, ctrl);
+            out_ = Invoke(entity, target.Op, target.Match, ctrl);
         }
         catch (Exception e)
         {
@@ -544,6 +583,8 @@ ${candidateLines}
         }
         if (null != err) sinks.AddRange(FormsOf("error", err));
         if (null != out_) sinks.AddRange(FormsOf("result", out_));
+        // Raw, as a caller copying the match into another query reads it.
+        sinks.AddRange(FormsOf("match", entity.Match()));
         if (ctrl.TryGetValue("explain", out var explain) && null != explain)
         {
             sinks.AddRange(FormsOf("explain", explain));
@@ -608,16 +649,21 @@ ${candidateLines}
             }
         }
 
+        // A name given at run time replaces the declared one: the match leaves
+        // out whichever name PrepareAuth placed.
+        Drive(MakeSdk(Scenarios[0], sinks, auth: new Dictionary<string, object?> { ["name"] = "zzcred" }),
+            target, new Dictionary<string, object?>(), sinks);
+
         // A credential mistyped as a map is rejected by validation, whose
         // message quotes the value it rejected.
         Exception? rejected = null;
         try
         {
-            new ${Name}SDK(new Dictionary<string, object?>
+            new ${Name}SDK(Offline(new Dictionary<string, object?>
             {
                 ["apikey"] = new Dictionary<string, object?> { ["value"] = CanaryApikey },
                 ["clean"] = new Dictionary<string, object?> { ["values"] = CanaryValue },
-            });
+            }));
         }
         catch (Exception e)
         {
@@ -776,11 +822,12 @@ ${candidateLines}
 
     // A feature's name is not a field name: only the sensitive names inside
     // its settings register. An entity block, of per-entity settings or
-    // seeded records keyed by entity name and id, is not read at all.
+    // seeded records keyed by entity name and id, is not read at all, and nor
+    // are rbac's rules, keyed by entity and operation names.
     [Fact]
     public void AFeatureNameDoesNotMakeItsSettingsSecret()
     {
-        var sdk = new ${Name}SDK(new Dictionary<string, object?>
+        var sdk = Construct(new Dictionary<string, object?>
         {
             ["apikey"] = CanaryApikey,
             ["feature"] = new Dictionary<string, object?>
@@ -793,6 +840,11 @@ ${candidateLines}
                 {
                     ["active"] = false, ["apitoken"] = "FEATTOKEN-z9y8x7w6",
                 },
+                ["rbac"] = new Dictionary<string, object?>
+                {
+                    ["active"] = false,
+                    ["rules"] = new Dictionary<string, object?> { ["zztoken.load"] = "PLAINRULE-k7j5h3g1" },
+                },
                 ["test"] = new Dictionary<string, object?>
                 {
                     ["active"] = false,
@@ -819,12 +871,13 @@ ${candidateLines}
         Assert.Equal("token " + Mask, Cleaned("token FEATTOKEN-z9y8x7w6"));
         Assert.Equal("record PLAINRECORD-t5r3e1w9", Cleaned("record PLAINRECORD-t5r3e1w9"));
         Assert.Equal("alias PLAINALIAS-m2n4b6v8", Cleaned("alias PLAINALIAS-m2n4b6v8"));
+        Assert.Equal("rule PLAINRULE-k7j5h3g1", Cleaned("rule PLAINRULE-k7j5h3g1"));
     }
 
     [Fact]
     public void TheGeneratedConfigsOwnCleanBlockIsHonoured()
     {
-        var utility = new ${Name}SDK(new Dictionary<string, object?>()).GetUtility();
+        var utility = Construct(new Dictionary<string, object?>()).GetUtility();
         var config = new Dictionary<string, object?>
         {
             ["options"] = new Dictionary<string, object?>
@@ -862,7 +915,7 @@ ${candidateLines}
     [Fact]
     public void WithNoCleanBlockTheSchemaDefaultsApply()
     {
-        var utility = new ${Name}SDK(new Dictionary<string, object?>()).GetUtility();
+        var utility = Construct(new Dictionary<string, object?>()).GetUtility();
         var ctx = utility.MakeContext(new Dictionary<string, object?>
         {
             ["utility"] = utility,

package/project/.sdk/src/cmp/csharp/TestDirect_csharp.ts

@@ -15,6 +15,8 @@ import {
   serverVarEnv,
   serverVariables,
   pointParts,
+  liveStrict,
+  liveStrictNote,
 } from '@voxgig/sdkgen'
 
 import { formatCsValue } from './utility_csharp'
@@ -162,6 +164,9 @@ namespace ${Name}Sdk.Test;
 
 public class ${entity.Name}DirectTest
 {
+${liveStrictNote(liveStrict(model, target.name), '//', '    ')}
+    private const bool LIVE_STRICT = ${liveStrict(model, target.name)};
+
 `)
 
     // Generate list test first (load needs list results in live mode)
@@ -184,7 +189,8 @@ public class ${entity.Name}DirectTest
             {
                 if (StructUtils.GetProp(setup.Idmap, _liveKey) == null)
                 {
-                    return; // live test needs *_ENTID env var (synthetic IDs only)
+                    TestRunner.LiveMiss(LIVE_STRICT, "Live test blocked: needs " + _liveKey + " via ${entidEnvVar}");
+                    return;
                 }
             }
         }
@@ -246,16 +252,14 @@ ${listSkipBlock}        var client = setup.Client;
 
       Content(`        if (setup.Live)
         {
-            // Live mode is lenient: synthetic IDs frequently 4xx and the
-            // list-response shape varies wildly across public APIs. Bail
-            // rather than fail when the call doesn't return a usable list.
-            if (!Equals(result["ok"], true))
+            if (!TestRunner.LiveOk(result))
             {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live list failed: " + TestRunner.LiveDescribe(result));
                 return;
             }
-            var status = Helpers.ToInt(result["status"]);
-            if (status < 200 || status >= 300)
+            if (TestRunner.LiveList(result["data"]) == null)
             {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live list returned no list: " + TestRunner.LiveDescribe(result));
                 return;
             }
         }
@@ -299,17 +303,11 @@ ${listSkipBlock}        var client = setup.Client;
       // Identify ancestor params (not 'id') for live mode
       const ancestorParams = loadParams.filter((p: any) => p.n !== 'id')
 
+      const liveKey = (p: any) => p.n === 'id' ? entity.name + '01' : p.n.replace(/_id$/, '') + '01'
       let loadLiveIdKeys: string[] = []
       if (loadParams.length > 0 && !loadAllHaveExamples) {
-        if (hasList) {
-          loadLiveIdKeys = listParams.map((p: any) => {
-            return p.n === 'id'
-              ? entity.name + '01'
-              : p.n.replace(/_id$/, '') + '01'
-          })
-        } else {
-          loadLiveIdKeys = loadParams.map((p: any) => p.n + '01')
-        }
+        loadLiveIdKeys = [...new Set((hasList ?
+          [...listParams, ...loadParams.filter((p: any) => p.n !== 'id')] : loadParams).map(liveKey))]
       }
       const loadSkipBlock = loadLiveIdKeys.length > 0
         ? `        if (setup.Live)
@@ -318,7 +316,8 @@ ${listSkipBlock}        var client = setup.Client;
             {
                 if (StructUtils.GetProp(setup.Idmap, _liveKey) == null)
                 {
-                    return; // live test needs *_ENTID env var (synthetic IDs only)
+                    TestRunner.LiveMiss(LIVE_STRICT, "Live test blocked: needs " + _liveKey + " via ${entidEnvVar}");
+                    return;
                 }
             }
         }
@@ -375,23 +374,38 @@ ${loadSkipBlock}        var client = setup.Client;
                 ["method"] = "GET",
                 ["params"] = listParams,
             });
-            if (!Equals(listResult["ok"], true))
+            if (!TestRunner.LiveOk(listResult))
             {
-                return; // list call not ok (likely synthetic IDs)
+                TestRunner.LiveMiss(LIVE_STRICT, "Live list discovery failed: " + TestRunner.LiveDescribe(listResult));
+                return;
             }
-
-            // Get first entity ID from list
-            var listData = listResult["data"] as List<object?>;
-            if (listData == null || listData.Count == 0)
+            var listData = TestRunner.LiveList(listResult["data"]);
+            if (listData == null)
+            {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live list discovery returned no list: " + TestRunner.LiveDescribe(listResult));
+                return;
+            }
+            if (listData.Count == 0)
             {
-                return; // no entities to load in live mode
+                TestRunner.LiveEmpty("The account has no ${entity.name} record to load");
+                return;
             }
             var firstEnt = Helpers.ToMapAny(listData[0]);
-            pathParams["id"] = firstEnt?["id"];
+            if (firstEnt == null || !firstEnt.TryGetValue("id", out var firstId) || firstId == null)
+            {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live load blocked: discovery returned no usable identity");
+                return;
+            }
+            pathParams["id"] = firstId;
 `)
           for (const p of ancestorParams) {
             const key = p.n.replace(/_id$/, '') + '01'
             Content(`            pathParams["${p.n}"] = setup.Idmap["${key}"];
+`)
+          }
+        } else if (loadParams.length > 0) {
+          for (const p of loadParams) {
+            Content(`            pathParams["${p.n}"] = StructUtils.GetProp(setup.Idmap, "${liveKey(p)}");
 `)
           }
         }
@@ -427,18 +441,16 @@ ${loadSkipBlock}        var client = setup.Client;
       Content(`        });
         if (setup.Live)
         {
-            // Live mode is lenient: synthetic IDs frequently 4xx. Bail
-            // rather than fail when the load endpoint isn't reachable.
-            if (!Equals(result["ok"], true))
+            if (!TestRunner.LiveOk(result))
             {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live load failed: " + TestRunner.LiveDescribe(result));
                 return;
             }
-            var status = Helpers.ToInt(result["status"]);
-            if (status < 200 || status >= 300)
+            if (result["data"] == null)
             {
+                TestRunner.LiveMiss(LIVE_STRICT, "Live load returned no data: " + TestRunner.LiveDescribe(result));
                 return;
             }
-            Assert.NotNull(result["data"]);
         }
         else
         {