fix: move invariant function from tests folder to fix combuild error

Author: boblat

examples/precompiled/precompiled-typed.spec.ts

@@ -138,11 +138,6 @@ describe('pre compiled typed app calls', () => {
         itxnContext.setReturnValue(1)
       }
     })
-    spy.on.receivesAssetConfig((itxnContext) => {
-      if (itxnContext.appId === receivesTxnsApp) {
-        itxnContext.setReturnValue(ctx.any.bytes(64))
-      }
-    })
 
     ctx.addApplicationSpy(spy)
 

src/errors.ts

@@ -49,7 +49,7 @@ export class NotImplementedError extends Error {
   }
 }
 
-export function testInvariant(condition: unknown, message: string): asserts condition {
+export function invariant(condition: unknown, message: string): asserts condition {
   if (!condition) {
     throw new InternalError(message)
   }

src/impl/inner-transactions.ts

@@ -7,10 +7,9 @@ import type {
 } from '@algorandfoundation/algorand-typescript'
 import { TransactionType } from '@algorandfoundation/algorand-typescript'
 import type { BareCreateApplicationCallFields, TypedApplicationCallFields } from '@algorandfoundation/algorand-typescript/arc4'
-import { invariant } from '../../tests/util'
 import { ABI_RETURN_VALUE_LOG_PREFIX } from '../constants'
 import { lazyContext } from '../context-helpers/internal-context'
-import { InternalError } from '../errors'
+import { InternalError, invariant } from '../errors'
 import { extractArraysFromArgs } from '../subcontexts/contract-context'
 import type { DeliberateAny } from '../typescript-helpers'
 import { asBytes, asNumber, asUint8Array } from '../util'
@@ -297,7 +296,7 @@ export class ApplicationCallInnerTxnContext<TReturn = unknown> extends Applicati
   setReturnValue(value: TReturn) {
     // Ignore undefined (void) values
     if (value === undefined) return
-    super.appendLog(ABI_RETURN_VALUE_LOG_PREFIX.concat(encodeArc4Impl(undefined, value)))
+    this.appendLog(ABI_RETURN_VALUE_LOG_PREFIX.concat(encodeArc4Impl(undefined, value)))
     this.#returnValue = value
   }
   /* @internal */

src/impl/pure.ts

@@ -1,7 +1,7 @@
 import type { biguint, bytes, op, uint64 } from '@algorandfoundation/algorand-typescript'
 import { Base64 } from '@algorandfoundation/algorand-typescript'
 import { BITS_IN_BYTE, MAX_BYTES_SIZE, MAX_UINT64, MAX_UINT8, UINT64_SIZE } from '../constants'
-import { AvmError, CodeError, NotImplementedError, testInvariant } from '../errors'
+import { AvmError, CodeError, invariant, NotImplementedError } from '../errors'
 import { asBigUint, asBytes, asBytesCls, asMaybeBytesCls, asMaybeUint64Cls, asUint64Cls, binaryStringToBytes } from '../util'
 import type { StubBigUintCompat, StubBytesCompat, StubUint64Compat } from './primitives'
 import { BigUintCls, Bytes, BytesCls, checkBigUint, isUint64, Uint64, Uint64Cls } from './primitives'
@@ -222,7 +222,7 @@ export const setBit = ((a: StubUint64Compat | StubBytesCompat, b: StubUint64Comp
     return newBytes.toUint64().asAlgoTs()
   } else {
     const bytesCls = asMaybeBytesCls(a)
-    testInvariant(bytesCls, 'a must be uint64 or bytes')
+    invariant(bytesCls, 'a must be uint64 or bytes')
     const binaryString = toBinaryString(bytesCls.asAlgoTs())
     const newBytes = doSetBit(binaryString, indexParam, bit)
     return newBytes.asAlgoTs()

src/subcontexts/transaction-context.ts

@@ -6,7 +6,7 @@ import { checkRoutingConditions } from '../context-helpers/context-util'
 import { lazyContext } from '../context-helpers/internal-context'
 import type { DecodedLogs, LogDecoding } from '../decode-logs'
 import { decodeLogs } from '../decode-logs'
-import { InternalError, testInvariant } from '../errors'
+import { InternalError, invariant } from '../errors'
 import type {
   ApplicationCallInnerTxn,
   AssetConfigInnerTxn,
@@ -265,7 +265,7 @@ export class TransactionGroup {
     if (this.transactions.length === 0) {
       throw new InternalError('No transactions in the group')
     }
-    testInvariant(this.activeTransaction.type === TransactionType.ApplicationCall, 'No app_id found in the active transaction')
+    invariant(this.activeTransaction.type === TransactionType.ApplicationCall, 'No app_id found in the active transaction')
     return (this.activeTransaction as ApplicationCallTransaction).backingAppId.id
   }
 
@@ -322,7 +322,7 @@ export class TransactionGroup {
     if (this.constructingItxnGroup.length) {
       throw new InternalError('itxn begin without itxn submit')
     }
-    testInvariant(this.activeTransaction.type === TransactionType.ApplicationCall, 'No active application call transaction')
+    invariant(this.activeTransaction.type === TransactionType.ApplicationCall, 'No active application call transaction')
     if (this.activeTransaction.onCompletion === OnCompleteAction.ClearState) {
       throw new InternalError('Cannot begin inner transaction group in a clear state call')
     }
@@ -376,12 +376,12 @@ export class TransactionGroup {
   getItxnGroup(index?: StubUint64Compat): ItxnGroup {
     const i = index !== undefined ? asNumber(index) : undefined
 
-    testInvariant(this.itxnGroups.length > 0, 'no previous inner transactions')
+    invariant(this.itxnGroups.length > 0, 'no previous inner transactions')
     if (i !== undefined && i >= this.itxnGroups.length) {
       throw new InternalError('Invalid group index')
     }
     const group = i !== undefined ? this.itxnGroups[i] : this.itxnGroups.at(-1)!
-    testInvariant(group.itxns.length > 0, 'no previous inner transactions')
+    invariant(group.itxns.length > 0, 'no previous inner transactions')
 
     return group
   }
@@ -552,7 +552,7 @@ export class ItxnGroup {
   }
 
   private getInnerTxnImpl({ type, index }: { type?: TransactionType; index?: StubUint64Compat }) {
-    testInvariant(this.itxns.length > 0, 'no previous inner transactions')
+    invariant(this.itxns.length > 0, 'no previous inner transactions')
     const i = index !== undefined ? asNumber(index) : undefined
     if (i !== undefined && i >= this.itxns.length) {
       throw new InternalError('Invalid group index')

tests/state-op-codes.spec.ts

@@ -7,7 +7,7 @@ import { afterEach, beforeAll, describe, expect } from 'vitest'
 import { TestExecutionContext } from '../src'
 import { ABI_RETURN_VALUE_LOG_PREFIX, MIN_TXN_FEE, OnApplicationComplete, ZERO_ADDRESS } from '../src/constants'
 import { lazyContext } from '../src/context-helpers/internal-context'
-import { testInvariant } from '../src/errors'
+import { invariant } from '../src/errors'
 import { Block, gloadBytes, gloadUint64 } from '../src/impl'
 import type { InnerTxn } from '../src/impl/itxn'
 import { BytesCls, Uint64Cls } from '../src/impl/primitives'
@@ -93,7 +93,7 @@ describe('State op codes', async () => {
         methodName as string,
         asUint8Array(mockAccount.bytes),
       )
-      testInvariant(avmResult !== undefined, 'There must be an AVM result')
+      invariant(avmResult !== undefined, 'There must be an AVM result')
       const mockContract = ctx.contract.create(StateAcctParamsGetContract)
       const mockResult = mockContract[methodName as keyof StateAcctParamsGetContract](mockAccount)
 

tests/test-fixture.ts

@@ -15,8 +15,9 @@ import { OnApplicationComplete } from 'algosdk'
 import fs from 'fs'
 import type { ExpectStatic } from 'vitest'
 import { test } from 'vitest'
+import { invariant } from '../src/errors'
 import type { DeliberateAny } from '../src/typescript-helpers'
-import { generateTempDir, invariant } from './util'
+import { generateTempDir } from './util'
 
 const algorandTestFixture = (localnetFixture: AlgorandFixture) =>
   test.extend<{

tests/util.ts

@@ -8,12 +8,6 @@ import type { BytesCls, StubBigUintCompat, StubBytesCompat } from '../src/impl/p
 import { BigUintCls, Bytes } from '../src/impl/primitives'
 import { asUint8Array } from '../src/util'
 
-class InvariantError extends Error {}
-export function invariant(condition: unknown, message: string): asserts condition {
-  if (!condition) {
-    throw new InvariantError(message)
-  }
-}
 export const padUint8Array = (arr: Uint8Array, padSize: number): Uint8Array => {
   const paddedUint8Array = new Uint8Array(arr.length + padSize)
   arr.forEach((v, i) => (paddedUint8Array[padSize + i] = v))