Skip to content

integration-tests: verifier#2068

Closed
tynes wants to merge 1 commit intodevelopfrom
feat/verifier-itests
Closed

integration-tests: verifier#2068
tynes wants to merge 1 commit intodevelopfrom
feat/verifier-itests

Conversation

@tynes
Copy link
Contributor

@tynes tynes commented Jan 25, 2022

Description

Add verifier integration tests behind the env var
RUN_VERIFIER_TESTS. Note that this depends on the
batch submitter correctly submitting batches because
the verifier syncs from the batches submitted by the
batch submitter.

The verifier is not enabled by default with the
docker-compose.yml file. To enable it, the replicas
field must be updated from 0 to 1.

Fixes ENG-1911

Add verifier integration tests behind the env var
`RUN_VERIFIER_TESTS`. Note that this depends on the
batch submitter correctly submitting batches because
the verifier syncs from the batches submitted by the
batch submitter.

The verifier is not enabled by default with the
`docker-compose.yml` file. To enable it, the replicas
field must be updated from `0` to `1`.
@changeset-bot
Copy link

changeset-bot bot commented Jan 25, 2022

🦋 Changeset detected

Latest commit: 2aa274e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@eth-optimism/integration-tests Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added 2-reviewers A-integration Area: integration tests A-ops Area: ops labels Jan 25, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 25, 2022

Codecov Report

Merging #2068 (2aa274e) into develop (4d7c545) will increase coverage by 0.14%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2068      +/-   ##
===========================================
+ Coverage    74.58%   74.73%   +0.14%     
===========================================
  Files           79       80       +1     
  Lines         2554     2561       +7     
  Branches       401      403       +2     
===========================================
+ Hits          1905     1914       +9     
+ Misses         649      647       -2     
Flag Coverage Δ
batch-submitter 62.50% <ø> (ø)
contracts 90.48% <ø> (ø)
core-utils 57.73% <ø> (+0.23%) ⬆️
data-transport-layer 38.64% <ø> (ø)
message-relayer 70.86% <ø> (ø)
sdk 89.43% <ø> (+1.04%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/core-utils/src/index.ts 100.00% <0.00%> (ø)
packages/core-utils/src/coders/index.ts 100.00% <0.00%> (ø)
packages/core-utils/src/batches.ts
packages/core-utils/src/types/rollup.ts 100.00% <0.00%> (ø)
packages/core-utils/src/types/index.ts 100.00% <0.00%> (ø)
packages/sdk/src/cross-chain-provider.ts 84.84% <0.00%> (+2.17%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d7c545...2aa274e. Read the comment docs.

@tynes
Copy link
Contributor Author

tynes commented Jan 31, 2022

Replaced by #2090

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-integration Area: integration tests A-ops Area: ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants