fix: improve code

src/api/procedures/__tests__/reserveTicker.ts

@@ -173,7 +173,7 @@ describe('reserveTicker procedure', () => {
     );
   });
 
-  test('should throw an error if extendPeriod property is set to true and the ticker has not reserved or the reservation has expired', async () => {
+  test('should throw an error if extendPeriod property is set to true and the ticker has not been reserved or the reservation has expired', async () => {
     const expiryDate = new Date(2019, 1, 1);
     mockTickerReservation.mock('details', {
       ownerDid: 'someDid',
@@ -214,7 +214,7 @@ describe('reserveTicker procedure', () => {
     proc.context = mockContext;
 
     return expect(prepareReserveTicker.call(proc, { ...args, extendPeriod: true })).rejects.toThrow(
-      'You must be the owner of the ticker to extend its registration period'
+      'You must be the owner of the ticker to extend its reservation period'
     );
   });
 

src/api/procedures/reserveTicker.ts

@@ -78,7 +78,7 @@ export async function prepareReserveTicker(
     } else if (ownerDid !== context.currentIdentity?.did) {
       throw new PolymeshError({
         code: ErrorCode.ValidationError,
-        message: 'You must be the owner of the ticker to extend its registration period',
+        message: 'You must be the owner of the ticker to extend its reservation period',
       });
     }
   }

src/testUtils/mocks/PolkadotMockFactory.ts

@@ -38,6 +38,7 @@ interface TxMockData {
 }
 
 interface ContextOptions {
+  did?: string;
   withSeed?: boolean;
   balance?: BigNumber;
 }
@@ -233,7 +234,7 @@ export class PolkadotMockFactory {
    */
   private initContext(opts: ContextOptions): void {
     const currentIdentity = opts.withSeed
-      ? { getIdentityBalance: sinon.stub().resolves(opts.balance), did: 'someDid' }
+      ? { getIdentityBalance: sinon.stub().resolves(opts.balance), did: opts.did || 'someDid' }
       : undefined;
     const currentPair = opts.withSeed
       ? ({