fix(validity): accept a validity query param that returns interactio …

…id and if they were valid or not

This is actually a miss when implementing `syncState` functionality. There is a related bug in the warp-sdk that does not default this value when not set - which is how we discovered this was an issue.

src/middleware/query.ts

@@ -23,12 +23,14 @@ const WARP_SORT_KEY_REGEX = /^[0-9]{12},[0-9]{13},[0-9a-f]{64}$/;
 const MAX_PAGE_LIMIT = 1000;
 const DEFAULT_PAGE_SIZE = 100;
 const DEFAULT_PAGE = 1;
+const DEFAULT_VALIDITY = false;
 export const queryMiddleware = async (ctx: KoaContext, next: Next) => {
   const {
     blockHeight,
     sortKey,
     page = DEFAULT_PAGE,
     pageSize = DEFAULT_PAGE_SIZE,
+    validity = DEFAULT_VALIDITY,
   } = ctx.query;
 
   logger.debug('Query params provided', {
@@ -82,5 +84,10 @@ export const queryMiddleware = async (ctx: KoaContext, next: Next) => {
     ctx.state.pageSize = +pageSize;
   }
 
+  if (validity) {
+    logger.debug('Validity provided', { validity });
+    ctx.state.validity = validity === 'true' || validity === '1';
+  }
+
   return next();
 };

src/routes/contract.ts

@@ -33,6 +33,7 @@ export async function contractHandler(ctx: KoaContext) {
     warp,
     sortKey: requestedSortKey,
     blockHeight: requestedBlockHeight,
+    validity: requestedValidity,
   } = ctx.state;
   const { contractTxId } = ctx.params;
   logger.debug('Fetching contract state', {
@@ -42,6 +43,7 @@ export async function contractHandler(ctx: KoaContext) {
   });
   const {
     state,
+    validity,
     evaluationOptions,
     sortKey: evaluatedSortKey,
   } = await getContractState({
@@ -57,6 +59,7 @@ export async function contractHandler(ctx: KoaContext) {
     state,
     sortKey: evaluatedSortKey,
     evaluationOptions,
+    ...(requestedValidity && { validity }),
   };
 }
 

tests/integration/routes.test.ts

@@ -139,7 +139,7 @@ describe('Integration tests', () => {
           expect(data).to.equal('Contract is blocklisted.');
           expect(statusText).to.equal('Contract is blocklisted.');
         });
-        it('should return the contract state and id and default evaluation options', async () => {
+        it('should return the contract state and id and default evaluation options without validity', async () => {
           const { status, data } = await axios.get(`/v1/contract/${id}`);
           expect(status).to.equal(200);
           expect(data).to.not.be.undefined;
@@ -158,6 +158,27 @@ describe('Integration tests', () => {
           ]);
         });
 
+        it('should return the contract state and id and default evaluation options when validity is provided', async () => {
+          const { status, data } = await axios.get(`/v1/contract/${id}`);
+          expect(status).to.equal(200);
+          expect(data).to.not.be.undefined;
+          const { contractTxId, state, evaluationOptions, sortKey, validity } =
+            data;
+          expect(contractTxId).to.equal(id);
+          expect(evaluationOptions).not.to.be.undefined;
+          expect(sortKey).not.be.undefined;
+          expect(validity).not.to.be.undefined;
+          expect(state).to.include.keys([
+            'balances',
+            'owner',
+            'name',
+            'records',
+            'ticker',
+            'owner',
+            'controller',
+          ]);
+        });
+
         it('should return a 404 for an invalid id', async () => {
           const { status } = await axios.get(`/v1/contract/non-matching-regex`);
           expect(status).to.equal(404);