10796813334

Committed 10 Sep 2024 04:35PM UTC coverage: 14.192% (-0.09%) from 14.284%

Build # 10796813334

Build Type

push

github

Committed by

web-flow

Commit Message

Merge pull request #9035 from hicommonwealth/malik.8811.cleanup-ChainInfo-type

Update `app.chain.meta` and related types to `ExtendedCommunity` trpc response type + Removed `ChainInfo` type

Run Details

3101 of 20251 branches covered (15.31%)

Branch coverage included in aggregate %.

1 of 157 new or added lines in 85 files covered. (0.64%)

43 existing lines in 41 files now uncovered.

4029 of 29987 relevant lines covered (13.44%)

4.01 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/packages/commonwealth/client/scripts/controllers/chain/cosmos/account.ts

import BN from 'bn.js';
import CosmosChain from 'controllers/chain/cosmos/chain';
import type { CosmosToken } from 'controllers/chain/cosmos/types';
import _ from 'lodash';
import type { IApp } from 'state';
import Account from '../../../models/Account';
import type CosmosAccounts from './accounts';

export default class CosmosAccount extends Account {
  private _Chain: CosmosChain;
  private _Accounts: CosmosAccounts;

  // TODO: add delegations, validations
  private _balance: CosmosToken;
  // NOTE: this balance query will not work on Terra, as it uses a nonstandard Cosmos interface.
  //   We should either deprecate this query and replace it with TokenBalanceCache, or create a
  //   workaround specific for Terra.
  public get balance() {
    // @ts-expect-error StrictNullChecks
    return this.updateBalance().then(() => this._balance);
  }

  constructor(
    app: IApp,
    ChainInfo: CosmosChain,
    Accounts: CosmosAccounts,
    address: string,
  ) {
    super({
      community: {
        id: app.chain.meta.id || '',
        base: app.chain.meta.base,
        ss58Prefix: app.chain.meta.ss58_prefix || 0,
      },
      address,
    });
    if (!app.isModuleReady) {
      // defer chain initialization
      app.chainModuleReady.once('ready', () => {
        if (app.chain.chain instanceof CosmosChain)
          this._Chain = app.chain.chain;
        else
          console.error('Did not successfully initialize account with chain');
      });
    } else {
      this._Chain = ChainInfo;
    }
    this._Accounts = Accounts;
    this._Accounts.store.add(this);
  }

  public updateBalance = _.throttle(async () => {
    try {
      const bal = await this._Chain.api.bank.balance(
        this.address,
        this._Chain.denom,
      );
      this._balance = this._Chain.coins(new BN(bal.amount));
    } catch (e) {
      // if coins is null, they have a zero balance
      console.log(`no balance found: ${e.message}`);
      this._balance = this._Chain.coins(0);
    }
  });
}

1	import BN from 'bn.js';
2	import CosmosChain from 'controllers/chain/cosmos/chain';
3	import type { CosmosToken } from 'controllers/chain/cosmos/types';
4	import _ from 'lodash';
5	import type { IApp } from 'state';
6	import Account from '../../../models/Account';
7	import type CosmosAccounts from './accounts';
8
9	export default class CosmosAccount extends Account {
10	private _Chain: CosmosChain;
11	private _Accounts: CosmosAccounts;
12
13	// TODO: add delegations, validations
14	private _balance: CosmosToken;
15	// NOTE: this balance query will not work on Terra, as it uses a nonstandard Cosmos interface.
16	// We should either deprecate this query and replace it with TokenBalanceCache, or create a
17	// workaround specific for Terra.
18	public get balance() {
19	// @ts-expect-error StrictNullChecks
20	return this.updateBalance().then(() => this._balance);	×
21	}
22
23	constructor(
24	app: IApp,
25	ChainInfo: CosmosChain,
26	Accounts: CosmosAccounts,
27	address: string,
28	) {
NEW 29	super({	×
30	community: {
31	id: app.chain.meta.id \|\| '',	×
32	base: app.chain.meta.base,
33	ss58Prefix: app.chain.meta.ss58_prefix \|\| 0,	×
34	},
35	address,
36	});
UNCOV 37	if (!app.isModuleReady) {	×
38	// defer chain initialization
39	app.chainModuleReady.once('ready', () => {	×
40	if (app.chain.chain instanceof CosmosChain)	×
41	this._Chain = app.chain.chain;	×
42	else
43	console.error('Did not successfully initialize account with chain');	×
44	});
45	} else {
46	this._Chain = ChainInfo;	×
47	}
48	this._Accounts = Accounts;	×
49	this._Accounts.store.add(this);	×
50	}
51
52	public updateBalance = _.throttle(async () => {	×
53	try {	×
54	const bal = await this._Chain.api.bank.balance(	×
55	this.address,
56	this._Chain.denom,
57	);
58	this._balance = this._Chain.coins(new BN(bal.amount));	×
59	} catch (e) {
60	// if coins is null, they have a zero balance
61	console.log(`no balance found: ${e.message}`);	×
62	this._balance = this._Chain.coins(0);	×
63	}
64	});
65	}

hicommonwealth / commonwealth / 10796813334

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous