celestia-app

celestia-app copied to clipboard

Consolidate network testing code

1

With the introduction of our own network tests, we should replace our usage of the old cosmos-sdk network tests for our integration tests. This will involve: - [ ] Adding...

evan-forbes

QGB 2.0 implementation proposal

2

Currently, the QGB orchestrator queries the state for attestations, confirms and unbonding periods. After [ADR-005](https://github.com/celestiaorg/celestia-app/blob/22a48e47721acb57030b5c520994ef9d0bd4d82a/docs/architecture/adr-005-qgb-reduce-state-usage.md) which reduces the QGB state usage, the confirms are no longer part of the state....

rach-id

Where is this actually called? I found it being used in `func Merge(eds...)`, but I don't see this latter being used anywhere _Originally posted by @SweeXordious in https://github.com/celestiaorg/celestia-app/pull/837#discussion_r987355330_

rootulp

Consolidate utility testing functions where possible

4

As we've discussed synchronously many times, there's quite a bit of redundant testing utility functions. We should consolidate where possible. If its not possible due to cyclic imports, then we...

evan-forbes

### Summary Currently, commitments in Celestia are dependent on the size of the block. It would be nice to have commitments that are independent of size. ### Problem Definition See...

nashqueue

We should use the existing https://github.com/celestiaorg/celestia-app/blob/e18de882c3de4ee92e4632af21d4007c1b642adb/testutil/test_app.go#L71 for the QGB tests instead of: ```golang func SetupTestChain(t *testing.T, weights []uint64) (TestInput, sdk.Context) { ``` _Originally posted by @evan-forbes in https://github.com/celestiaorg/celestia-app/pull/742#discussion_r975808339_

rach-id

## Context - https://github.com/celestiaorg/celestia-app/issues/507#issue-1284883031 - https://github.com/celestiaorg/celestia-app/pull/554 ## Problem We want to remove the temporary import hack in https://github.com/celestiaorg/celestia-app/pull/742/files#diff-63113d53e02a4e334ffa1a1ba0c308329c7acd431153571a2c02ce9268af5816R28-R32

rootulp

Limit validators' set for the PoA network

2

As per discussions with the leadership team, we want to limit the validator set to 4 This can be done in the genesis.json file and I assume that we are...

Bidon15

Potentially this can be done by following what gai does (I did some testing with Alessio back in the days but I don't remember the details and the process likely...

liamsi

As pointed out in this [comment](https://github.com/celestiaorg/celestia-app/pull/692#discussion_r967369561) and this [comment](https://github.com/celestiaorg/celestia-app/pull/692#discussion_r967387967), it will be easier to maintain and modify share encoding code if we also have a canonical API for calculating the...

evan-forbes