Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

ethcore/res: remove morden testnet #11392

Merged
merged 10 commits into from
Jan 28, 2020
Merged

ethcore/res: remove morden testnet #11392

merged 10 commits into from
Jan 28, 2020

Conversation

q9f
Copy link
Member

@q9f q9f commented Jan 20, 2020

ethereum foundation no longer uses the morden testnet. it was replaced by ropsten (ethash).

ethereum classic no longer uses the morden testnet. it was replaced by mordor (ethash) and kotti (clique).

the last two clients supporting morden are parity ethereum (this) and geth classic. geth classic reached end of life on january 15, 2020 leaving parity with the only client supporting the deprecated testnet.

this pull request removes morden testnet.

this pull request replaces morden in tests with either null_morden or ropsten whereever applicable.

  • You added a brief description of the PR, e.g.:
    • What does it do?
    • What important points reviewers should know?
    • Is there something left for follow-up PRs?
  • You labeled the PR with appropriate labels if you have permissions to do so.
  • You mentioned a related issue if this PR related to it, e.g. Fixes #228 or Related #1337.
  • You asked any particular reviewers to review. If you aren't sure, start with GH suggestions.
  • Your PR adheres the style guide
    • In particular, mind the maximal line length.
    • There is no commented code checked in unless necessary.
    • Any panickers have a proof or removed.
  • You updated any rustdocs which may have changed

@parity-cla-bot
Copy link

It looks like @q9f signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@dvdplm
Copy link
Collaborator

dvdplm commented Jan 20, 2020

Thanks for the PR!
The two test failures (on_close_block_with_uncle and on_close_block) seem legit; can you investigate what might be wrong there?

@dvdplm dvdplm added the A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. label Jan 20, 2020
@dvdplm dvdplm requested a review from sorpaas January 20, 2020 13:32
@q9f
Copy link
Member Author

q9f commented Jan 20, 2020

using ropsten instead of morden in tests broke some assumptions. I updated the tests but their logic should be reviewed carefully.

ethcore/src/tests/client.rs Outdated Show resolved Hide resolved
q9f and others added 2 commits January 27, 2020 12:15
@ordian
Copy link
Collaborator

ordian commented Jan 27, 2020

try this

diff --git a/ethcore/spec/src/chain.rs b/ethcore/spec/src/chain.rs
index 1112fb082..f714530f0 100644
--- a/ethcore/spec/src/chain.rs
+++ b/ethcore/spec/src/chain.rs
@@ -146,8 +146,8 @@ mod tests {
                assert_eq!(s.balance(&"0000000000000000000000000000000000000002".parse().unwrap()).unwrap(), 1u64.into());
                assert_eq!(s.balance(&"0000000000000000000000000000000000000003".parse().unwrap()).unwrap(), 1u64.into());
                assert_eq!(s.balance(&"0000000000000000000000000000000000000004".parse().unwrap()).unwrap(), 1u64.into());
-               assert_eq!(s.balance(&"102e61f5d8f9bc71d0ad4a084df4e65e05ce0e1c".parse().unwrap()).unwrap(), U256::from(1u64) << 200);
-               assert_eq!(s.balance(&"0000000000000000000000000000000000000000".parse().unwrap()).unwrap(), 0u64.into());
+               assert_eq!(s.balance(&"874b54a8bd152966d63f706bae1ffeb0411921e5".parse().unwrap()).unwrap(), U256::from(1000000000000000000000000000000u128));
+               assert_eq!(s.balance(&"0000000000000000000000000000000000000000".parse().unwrap()).unwrap(), 1u64.into());
        }
 
        #[test]
@@ -155,9 +155,9 @@ mod tests {
                let tempdir = TempDir::new("").unwrap();
                let ropsten = new_ropsten(&tempdir.path());
 
-               assert_eq!(ropsten.state_root, "f3f4696bbf3b3b07775128eb7a3763279a394e382130f27c21e70233e04946a9".parse().unwrap());
+               assert_eq!(ropsten.state_root, "217b0bbcfb72e2d57e28f33cb361b9983513177755dc3f33ce3e7022ed62b77b".parse().unwrap());
                let genesis = ropsten.genesis_block();
-               assert_eq!(view!(BlockView, &genesis).header_view().hash(), "0cd786a2425d16f152c658316c423e6ce1181e15c3295826d7c9904cba9ce303".parse().unwrap());
+               assert_eq!(view!(BlockView, &genesis).header_view().hash(), "41941023680923e0fe4d74a34bdac8141f2540e3ae90623718e47d66d1ca4a2d".parse().unwrap());
        }
 
        #[test]

@q9f
Copy link
Member Author

q9f commented Jan 27, 2020

tests pass. might need release notes if someone still uses morden.

@ordian ordian added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Jan 28, 2020
@ordian ordian merged commit c7c18d0 into openethereum:master Jan 28, 2020
@ordian ordian added the B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release. label Jan 28, 2020
@q9f q9f deleted the q9-remove-morden branch January 28, 2020 16:38
dvdplm added a commit that referenced this pull request Jan 29, 2020
…pstream

* master:
  Add POSDAO transition and malice report queue. (#11245)
  update master/nightly version: v2.8.0 (#11419)
  ethcore/res: remove morden testnet (#11392)
  fix: export hardcoded sync format (#11416)
  update hardcoded headers: mainnet and ropsten (#11414)
  AuthorityEngine: Minor cleanups. (#11408)
  Update POA bootnodes (#11411)
  Add EtherCore support (#11402)
  verification: fix race same block + misc (#11400)
  Update ProgPoW to 0.9.3 (#11407)
  update classic testnet bootnodes (#11398)
  dependencies: bump `derive_more v0.99` (#11405)
  engine error: remove faulty/unused `From` (#11404)
  Switching to stable-track (#11377)
  ethcore/res: fix ethereum classic chainspec blake2_f activation block num (#11391)
  Update copyright notice 2020 (#11386)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. B7-releasenotes 📜 Changes should be mentioned in the release notes of the next minor version release.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants