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

Conversation

sitetester
Copy link
Contributor

@sitetester sitetester commented Mar 10, 2023

What was the problem?

This PR resolves #8151
Mainchain PR: #8239

How was it solved?

This PR includes:

  • All changes from mainchain (WITH & WITHOUT terminatedStateAccounts and terminatedOutboxAccounts)
  • Sidechain verification checks
  • Common fixtures from mainchain.spec.ts moved to interopFixtures.ts
  • Common tests moved to base_interoperability_module.spec.ts

How was it tested?

module.spec.ts added for sidechain

…OutboxAccounts' of https://github.com/LiskHQ/lisk-sdk into 8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts
…eAccounts & terminatedOutboxAccounts)

# Conflicts:
#	framework/src/modules/interoperability/mainchain/module.ts
- Include ALL changes from mainchain
- Apply sidechain verification (this also includes common checks with mainchain)
@sitetester sitetester changed the base branch from 8151-update-genesis-asset-schema-interoperability to 8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts March 10, 2023 11:12
@sitetester sitetester changed the title 8151 update genesis asset schema interoperability sidechain Update genesis asset schema interoperability sidechain Mar 10, 2023
@sitetester sitetester marked this pull request as ready for review March 10, 2023 11:17
@codecov
Copy link

codecov bot commented Mar 10, 2023

Codecov Report

Merging #8256 (85c4626) into 8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts (0122cb2) will increase coverage by 0.04%.
The diff coverage is 96.92%.

❗ Current head 85c4626 differs from pull request most recent head 1cf6ef9. Consider uploading reports for the commit 1cf6ef9 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@                                            Coverage Diff                                            @@
##           8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts    #8256      +/-   ##
=========================================================================================================
+ Coverage                                                                  83.10%   83.14%   +0.04%     
=========================================================================================================
  Files                                                                        588      588              
  Lines                                                                      21722    21804      +82     
  Branches                                                                    3164     3192      +28     
=========================================================================================================
+ Hits                                                                       18051    18129      +78     
- Misses                                                                      3671     3675       +4     
Impacted Files Coverage Δ
framework/src/modules/interoperability/schemas.ts 100.00% <ø> (ø)
...k/src/modules/interoperability/sidechain/module.ts 94.95% <95.94%> (+1.34%) ⬆️
...s/interoperability/base_interoperability_module.ts 97.26% <97.77%> (+0.70%) ⬆️
...k/src/modules/interoperability/mainchain/module.ts 96.35% <100.00%> (-0.25%) ⬇️

... and 3 files with indirect coverage changes

…tedOutboxAccounts' of github.com:LiskHQ/lisk-sdk into 8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts
@ishantiw ishantiw requested a review from gkoumout March 21, 2023 14:08
…OutboxAccounts' of github.com:LiskHQ/lisk-sdk into 8151-update-genesis-asset-schema-interoperability-sidechain
ishantiw added a commit that referenced this pull request Mar 24, 2023
…ccounts and terminatedOutboxAccounts (#8238)

### What was the problem?

This PR resolves #8151 (ONLY mainchain)
Sidechain PR: #8256

### How was it solved?
Verification checks applied per
[LIP](https://github.com/LiskHQ/lips/blob/main/proposals/lip-0045.md#mainchain)
Relevant PR: #8239

**Note:** 
- To reduce size of PR, this task is split into multiple PRs
- This PR only covers mainchain part

### How was it tested?

- Test cases added to `module.spec.ts`
- To check/verify schema at example path: 
    - cd lisk-sdk/examples/pos-mainchain
- ./bin/run genesis-block:create --output=config/default
--assets-file=./config/default/genesis_assets.json
- To run example app
    -  ./bin/run start --api-ipc --api-http --port 9001
- if it fails with `Error: Genesis block does not match.`, run `rm -rf
~/.lisk/pos-mainchain` (**Note:**, it's not `rm -rf
~/.lisk/dpos-mainchain`)
@ishantiw ishantiw changed the base branch from 8151-update-with-terminatedStateAccounts-and-terminatedOutboxAccounts to development March 24, 2023 09:45
@sitetester sitetester marked this pull request as draft March 24, 2023 11:30
@sitetester sitetester changed the base branch from development to 8151-update-genesis-asset-schema-interoperability March 24, 2023 11:30
@ishantiw ishantiw merged commit 1cf6ef9 into 8151-update-genesis-asset-schema-interoperability Mar 24, 2023
@ishantiw ishantiw deleted the 8151-update-genesis-asset-schema-interoperability-sidechain branch March 24, 2023 13:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants