message: Rejecting a promise with a meaningful error object is just as drastic and much more informative than doing an assert. Assert throws, which should never be done in an async method.
diff: @@ -363,7 +363,6 @@ export class IModelDbElements {
  if (elementId instanceof Id64) return this._doGetElement({ id: elementId });
  if (elementId instanceof Guid) return this._doGetElement({ federationGuid: elementId.toString() });
  if (elementId instanceof Code) return this._doGetElement({ code: elementId });
  - assert(false);
  return Promise.reject(new IModelError(IModelStatus.BadArg));
  }
  