Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Update get name function of NFT module #8988

Merged
merged 2 commits into from
Sep 14, 2023
Merged

Update get name function of NFT module #8988

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
b90a783
Use constant
Incede Sep 14, 2023
709c938
Merge branch 'release/6.1.0' into 8983-branch
shuse2 Sep 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion framework/src/modules/nft/module.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ import {
ALL_SUPPORTED_NFTS_KEY,
LENGTH_ADDRESS,
LENGTH_CHAIN_ID,
MODULE_NAME_NFT,
NFT_NOT_LOCKED,
} from './constants';

Expand Down Expand Up @@ -122,7 +123,7 @@ export class NFTModule extends BaseInteroperableModule {
}

public get name(): string {
return 'nft';
return MODULE_NAME_NFT;
}

public addDependencies(
Expand Down
3 changes: 2 additions & 1 deletion framework/test/unit/modules/nft/module.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ import {
LENGTH_CHAIN_ID,
LENGTH_COLLECTION_ID,
LENGTH_NFT_ID,
MODULE_NAME_NFT,
NFT_NOT_LOCKED,
} from '../../../../src/modules/nft/constants';
import { NFTStore } from '../../../../src/modules/nft/stores/nft';
Expand All @@ -55,7 +56,7 @@ describe('nft module', () => {
};

it('should have the name "nft"', () => {
expect(module.name).toBe('nft');
expect(module.name).toBe(MODULE_NAME_NFT);
});

describe('initGenesisState', () => {
Expand Down