Skip to content

Commit efd18e0

Browse files
committed
fix: update maxAllowedRewardShare, maxRts, maxMintingFee, and maxRevenueShare values in group and IP asset tests for improved accuracy and consistency
1 parent e633fd3 commit efd18e0

File tree

3 files changed

+9
-19
lines changed

3 files changed

+9
-19
lines changed

packages/core-sdk/test/integration/group.test.ts

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ describe("Group Functions", () => {
151151
},
152152
},
153153
],
154-
maxAllowedRewardShare: 5,
154+
maxAllowedRewardShare: 100,
155155
});
156156

157157
expect(result.txHash).to.be.a("string");
@@ -172,7 +172,6 @@ describe("Group Functions", () => {
172172
groupId,
173173
nftContract: spgNftContract,
174174
tokenId,
175-
maxAllowedRewardShare: 5,
176175
licenseData: [
177176
{
178177
licenseTermsId,
@@ -197,7 +196,7 @@ describe("Group Functions", () => {
197196
it("should successfully register group with license and add multiple IPs", async () => {
198197
const result = await client.groupClient.registerGroupAndAttachLicenseAndAddIps({
199198
groupPool: groupPoolAddress,
200-
maxAllowedRewardShare: 5,
199+
maxAllowedRewardShare: 55.5,
201200
ipIds: [ipId],
202201
licenseData: {
203202
licenseTermsId,
@@ -264,7 +263,7 @@ describe("Group Functions", () => {
264263
const result = await client.groupClient.addIpsToGroup({
265264
groupIpId: groupId,
266265
ipIds: ipIds,
267-
maxAllowedRewardSharePercentage: 5,
266+
maxAllowedRewardSharePercentage: 55.55555555,
268267
});
269268
expect(result.txHash).to.be.a("string");
270269
});

packages/core-sdk/test/integration/ipAsset.test.ts

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,6 @@ describe("IP Asset Functions", () => {
234234
const response = await client.ipAsset.registerDerivativeWithLicenseTokens({
235235
childIpId: ipId,
236236
licenseTokenIds: [mintLicenseTokensResult.licenseTokenIds![0]],
237-
maxRts: 5 * 10 ** 6,
238237
});
239238
expect(response.txHash).to.be.a("string");
240239
});
@@ -537,7 +536,6 @@ describe("IP Asset Functions", () => {
537536
const result = await client.ipAsset.mintAndRegisterIpAndMakeDerivativeWithLicenseTokens({
538537
spgNftContract: nftContract,
539538
licenseTokenIds: [mintLicenseTokensResult.licenseTokenIds![0]],
540-
maxRts: 5 * 10 ** 6,
541539
ipMetadata: {
542540
ipMetadataURI: "test-uri",
543541
ipMetadataHash: toHex("test-metadata-hash", { size: 32 }),
@@ -1116,7 +1114,7 @@ describe("IP Asset Functions", () => {
11161114
await client.ipAsset.mintAndRegisterIpAndMakeDerivativeWithLicenseTokens({
11171115
spgNftContract: nftContractWithMintingFee,
11181116
licenseTokenIds: licenseTokenIds!,
1119-
maxRts: MAX_ROYALTY_TOKEN,
1117+
maxRts: 0,
11201118
ipMetadata: {
11211119
ipMetadataURI: "test",
11221120
ipMetadataHash: zeroHash,
@@ -1603,15 +1601,12 @@ describe("IP Asset Functions", () => {
16031601
childIpId: childIpId,
16041602
parentIpIds: [parentIpId],
16051603
licenseTermsIds: [noCommercialLicenseTermsId],
1606-
maxMintingFee: 0,
1607-
maxRts: 5 * 10 ** 6,
1608-
maxRevenueShare: 0,
16091604
},
16101605
{
16111606
childIpId: childIpId2,
16121607
parentIpIds: [parentIpId],
16131608
licenseTermsIds: [noCommercialLicenseTermsId],
1614-
maxMintingFee: 0,
1609+
maxMintingFee: 10000000,
16151610
maxRts: 5 * 10 ** 6,
16161611
maxRevenueShare: 0,
16171612
},
@@ -1875,7 +1870,7 @@ describe("IP Asset Functions", () => {
18751870
})
18761871
).ipId!,
18771872
licenseTokenIds: [mintLicenseTokensResult.licenseTokenIds![0]],
1878-
maxRts: 5 * 10 ** 6,
1873+
maxRts: 0,
18791874
});
18801875

18811876
const tokenId2 = await getTokenId();
@@ -3568,9 +3563,6 @@ describe("IP Asset Functions", () => {
35683563
derivData: {
35693564
parentIpIds: [parentIpId!],
35703565
licenseTermsIds: [commercialRemixLicenseTermsId],
3571-
maxMintingFee: 10000n,
3572-
maxRts: 100,
3573-
maxRevenueShare: 100,
35743566
},
35753567
royaltyShares: [
35763568
{
@@ -3620,6 +3612,7 @@ describe("IP Asset Functions", () => {
36203612
nft: { type: "minted", nftContract: mockERC721, tokenId: tokenId! },
36213613
licenseTokenIds: licenseTokenIds!,
36223614
maxRts: 100,
3615+
maxMintingFee: 10000n,
36233616
});
36243617

36253618
expect(result.ipId).to.be.a("string");

packages/core-sdk/test/integration/license.test.ts

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,8 +109,6 @@ describe("License Functions", () => {
109109
const result = await client.license.mintLicenseTokens({
110110
licenseTermsId: licenseId,
111111
licensorIpId: ipId,
112-
maxMintingFee: 1,
113-
maxRevenueShare: 100,
114112
});
115113
expect(result.txHash).to.be.a("string");
116114
expect(result.licenseTokenIds).to.be.a("array");
@@ -134,8 +132,8 @@ describe("License Functions", () => {
134132
const result = await client.license.mintLicenseTokens({
135133
licenseTermsId: licenseId,
136134
licensorIpId: ipIdB,
137-
maxMintingFee: 1,
138-
maxRevenueShare: 100,
135+
maxMintingFee: 10000000,
136+
maxRevenueShare: 55.55555555,
139137
});
140138
expect(result.txHash).to.be.a("string");
141139
expect(result.licenseTokenIds).to.be.a("array");

0 commit comments

Comments
 (0)