Enable test again (#280)

* Enable london fork in allEnabledConfig

* Move up the make-all in Makefile

* Add Makefile entry to download tests

* Fix rawdb tests

* Fix core tests

* Remove old testdata

* Enable full test

* Fix makefile

* Remove duplicate test for ci

* Fix small format issue

* Filter a couple of folders

* Fix go test

Makefile

@@ -50,8 +50,8 @@ ios:
 
 test: all
  # $(GORUN) build/ci.go test
- go test github.com/ethereum/go-ethereum/consensus/bor -v
- go test github.com/ethereum/go-ethereum/tests/bor -v
+ # Skip mobile and cmd tests since they are being deprecated
+ go test -v $(go list ./... | grep -v go-ethereum/cmd/ | grep -v go-ethereum/mobile/)
 
 lint: ## Run linters.
  $(GORUN) build/ci.go lint

core/rawdb/freezer.go

@@ -151,8 +151,10 @@ func newFreezer(datadir string, namespace string, readonly bool, maxTableSize ui
  // This way they don't have to sync again from block 0 and still be compatible
  // for block logs for future blocks. Note that already synced nodes
  // won't have past block logs. Newly synced node will have all the data.
- if err := freezer.tables[freezerBorReceiptTable].Fill(freezer.tables[freezerHeaderTable].items); err != nil {
- return nil, err
+ if _, ok := freezer.tables[freezerBorReceiptTable]; ok {
+ if err := freezer.tables[freezerBorReceiptTable].Fill(freezer.tables[freezerHeaderTable].items); err != nil {
+ return nil, err
+ }
  }
 
  // Truncate all tables to common length.

internal/ethapi/api.go

@@ -634,7 +634,7 @@ func (s *PublicBlockChainAPI) GetTransactionReceiptsByBlock(ctx context.Context,
  }
 
  if len(txs) != len(receipts) {
- return nil, fmt.Errorf("txs length doesn't equal to receipts' length", len(txs), len(receipts))
+ return nil, fmt.Errorf("txs length %d doesn't equal to receipts' length %d", len(txs), len(receipts))
  }
 
  txReceipts := make([]map[string]interface{}, 0, len(txs))

params/config.go

@@ -360,7 +360,7 @@ var (
  //
  // This configuration is intentionally not using keyed fields to force anyone
  // adding flags to the config to also have to set these fields.
- AllEthashProtocolChanges = &ChainConfig{big.NewInt(1337), big.NewInt(0), nil, false, big.NewInt(0), common.Hash{}, big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), nil, nil, new(EthashConfig), nil, nil}
+ AllEthashProtocolChanges = &ChainConfig{big.NewInt(1337), big.NewInt(0), nil, false, big.NewInt(0), common.Hash{}, big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), nil, new(EthashConfig), nil, nil}
 
  // AllCliqueProtocolChanges contains every protocol change (EIPs) introduced
  // and accepted by the Ethereum core developers into the Clique consensus.
@@ -369,7 +369,7 @@ var (
  // adding flags to the config to also have to set these fields.
  AllCliqueProtocolChanges = &ChainConfig{big.NewInt(1337), big.NewInt(0), nil, false, big.NewInt(0), common.Hash{}, big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), nil, nil, nil, &CliqueConfig{Period: 0, Epoch: 30000}, nil}
 
- TestChainConfig = &ChainConfig{big.NewInt(1), big.NewInt(0), nil, false, big.NewInt(0), common.Hash{}, big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), nil, nil, new(EthashConfig), nil, nil}
+ TestChainConfig = &ChainConfig{big.NewInt(1), big.NewInt(0), nil, false, big.NewInt(0), common.Hash{}, big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), big.NewInt(0), nil, new(EthashConfig), nil, nil}
  TestRules = TestChainConfig.Rules(new(big.Int))
 )