Add checks for ddo chain id and did. (#1868)

oceanprotocol · Nov 4, 2024 · 6e0d0e8 · 6e0d0e8
1 parent dc64981
commit 6e0d0e8
Show file tree

Hide file tree

Showing 8 changed files with 19 additions and 12 deletions.
diff --git a/CodeExamples.md b/CodeExamples.md
@@ -167,7 +167,7 @@ Next, we define the metadata that will describe our data asset. This is what we
     '@context': ['https://w3id.org/did/v1'],
     id: '',
     version: '4.1.0',
-    chainId: 4,
+    chainId: 8996,
     nftAddress: '0x0',
     metadata: {
       created: '2021-12-20T14:35:20Z',

diff --git a/ComputeExamples.md b/ComputeExamples.md
@@ -183,7 +183,7 @@ const DATASET_DDO: DDO = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'id:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 5,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',
@@ -219,7 +219,7 @@ const ALGORITHM_DDO: DDO = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 5,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',

diff --git a/src/utils/Assets.ts b/src/utils/Assets.ts
@@ -153,6 +153,13 @@ export async function createAsset(
   }
   const chainID = (await owner.provider.getNetwork()).chainId
 
+  if (ddo.chainId !== chainID) {
+    throw new Error('Chain ID from DDO is different than the configured network.')
+  }
+  if (ddo.id) {
+    throw new Error('DID already filled in.')
+  }
+
   const config = new ConfigHelper().getConfig(parseInt(String(chainID)))
 
   let templateIndex = await calculateActiveTemplateIndex(

diff --git a/test/integration/CodeExamples.test.ts b/test/integration/CodeExamples.test.ts
@@ -167,7 +167,7 @@ describe('Marketplace flow tests', async () => {
     '@context': ['https://w3id.org/did/v1'],
     id: '',
     version: '4.1.0',
-    chainId: 4,
+    chainId: 8996,
     nftAddress: '0x0',
     metadata: {
       created: '2021-12-20T14:35:20Z',

diff --git a/test/integration/ComputeExamples.test.ts b/test/integration/ComputeExamples.test.ts
@@ -183,7 +183,7 @@ const DATASET_DDO: DDO = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'id:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 5,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',
@@ -219,7 +219,7 @@ const ALGORITHM_DDO: DDO = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 5,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',

diff --git a/test/integration/ComputeFlow.test.ts b/test/integration/ComputeFlow.test.ts
@@ -56,7 +56,7 @@ const ddoWithNoTimeout = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 4,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',
@@ -93,7 +93,7 @@ const ddoWith5mTimeout = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 4,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',
@@ -140,7 +140,7 @@ const algoDdoWithNoTimeout = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 4,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',
@@ -182,7 +182,7 @@ const algoDdoWith5mTimeout = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 4,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',

diff --git a/test/integration/PublishEditConsume.test.ts b/test/integration/PublishEditConsume.test.ts
@@ -102,7 +102,7 @@ const assetDdo = {
   '@context': ['https://w3id.org/did/v1'],
   id: 'did:op:efba17455c127a885ec7830d687a8f6e64f5ba559f8506f8723c1f10f05c049c',
   version: '4.1.0',
-  chainId: 4,
+  chainId: 8996,
   nftAddress: '0x0',
   metadata: {
     created: '2021-12-20T14:35:20Z',

diff --git a/test/integration/PublishFlows.test.ts b/test/integration/PublishFlows.test.ts
@@ -56,7 +56,7 @@ describe('Publish tests', async () => {
     '@context': ['https://w3id.org/did/v1'],
     id: '',
     version: '4.1.0',
-    chainId: 4,
+    chainId: 8996,
     nftAddress: '0x0',
     metadata: {
       created: '2021-12-20T14:35:20Z',