test: rebase integration tests to main #167
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
# What ❔ Deprecate warning for Goerli network: <img width="1279" alt="image" src="https://github.com/matter-labs/block-explorer/assets/6553665/6f1e0117-3561-4d85-a2a4-dd8fc53b695a"> ## Why ❔ To warn users that we are ending support of Goerli testnet. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs).
# What ❔ Improve transaction indexing status: <img width="723" alt="image" src="https://github.com/matter-labs/block-explorer/assets/6553665/3d86294a-8d74-4c93-8acf-1f0f5a6c3c72"> ## Why ❔ For better UX we want to show that _Indexing_ transaction status means that transaction has been already included. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
# What ❔ Fix header height when warning message is shown.
# What ❔ Remove warning message for mainnet since we don't need it anymore.
…134) # What ❔ Do not try to save tokens whose symbols contain only special characters. ## Why ❔ Because it violates the token DB constraint. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
# What ❔ Remove prefetch reuest when search by tx hash. ## Why ❔ To be able to see search for tx that are in blockchain but hasn't been indexed yet. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs).
# What ❔ Add mainnet warning - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
…ets for RPC calls (#140)
# What ❔ - Updates dev discussions link <!-- What are the changes this PR brings about? --> <!-- Example: This PR adds a PR template to the repo. --> <!-- (For bigger PRs adding more context is appreciated) --> ## Why ❔ - We updated the link <!-- Why are these changes done? What goal do they contribute to? What are the principles behind them? --> <!-- Example: PR templates ensure PR reviewers, observers, and future iterators are in context about the evolution of repos. --> ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [ ] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [ ] Tests for the changes have been added / updated. - [ ] Documentation comments have been added / updated. Co-authored-by: Vasyl Ivanchuk <[email protected]>
# What ❔ updated workflow to make public allure reports ## Why ❔ current reports provide a link to a private url ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [ + ] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [ n/a ] Tests for the changes have been added / updated. - [ n/a ] Documentation comments have been added / updated. --------- Co-authored-by: Vasyl Ivanchuk <[email protected]>
# What ❔ Fix links to documentation. ## Why ❔ The documentation has been updated so we should use new links. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
# What ❔ Add data fetcher service that can be vertically scaled. ## Why ❔ Having data fetching vertically scaled we will be able to keep up with higher TPS. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
# What ❔ - Remove Portal link from the top menu - Do not show an error reason for the failed transaction if it's empty - Docker compose starts services in production and not in development mode - Add zkVM filter option for solc compilers on the verification page ## Why ❔ To fix bug reports / implement feature requests. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated. --------- Co-authored-by: Oleh Bairak <[email protected]>
# What ❔ Add more Data Fetcher debug logs. ## Why ❔ For easier troubleshooting. ## Checklist <!-- Check your PR fulfills the following items. --> <!-- For draft PRs check the boxes as you complete them. --> - [X] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [X] Tests for the changes have been added / updated.
# What ❔
Added test for TC id1803 /tokens/{address}/transfers endpoint
## Why ❔
Expand test coverage - id1803 wasn't covered by autotest
## Checklist
- [ +] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
# What ❔ Fix a typo for deprecated docker command ## Why ❔ to run command without an error ## Checklist - [ +] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [ -] Tests for the changes have been added / updated. - [ +] Documentation comments have been added / updated.
covering Batches API
# What ❔
1506 [Transactions] /transactions response returns elements (wasn't
covered fully)
1513 [Batches] /batches response returns elements (wasn't covered fully)
1510 [Transactions] /address/{address}/logs response returns elements
(wasn't covered fully)
1656 [Transactions] Verify failed tx (new test)
1655 [Transactions] Verify deployed the own ERC20 token contract in BE
(new test)
## Why ❔
Why are these changes done? - expanding test coverage
## Checklist
- [ +] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ +] Tests for the changes have been added / updated.
- [ n/a] Documentation comments have been added / updated.
# What ❔ - new API integration tests to cover account API functionality **_List of test cases for automation:_** - [x] [[Account API] /api?module=account&action=txlistinternal&address= response returns elements](https://allure.matterlabs.dev/project/11/test-cases/1852?treeId=22) - [x] [[Account API] /api?module=account&action=getminedblocks](https://allure.matterlabs.dev/project/11/test-cases/1807?treeId=22) - [x] [[Account API] /api?module=account&action=tokentx response returns elements](https://allure.matterlabs.dev/project/11/test-cases/1805?treeId=22) - [x] [[Account API] /api?module=account&action=txlistinternal response returns elements](https://allure.matterlabs.dev/project/11/test-cases/1804?treeId=22) - [x] [[Account API] /api?module=account&action=tokennfttx response returns elements](https://allure.matterlabs.dev/project/11/test-cases/1806?treeId=22) ## Why ❔ - to increase coverage of the automation test suite and cover accounts API part ## Checklist - [ +] PR title corresponds to the body of PR (we generate changelog entries from PRs). - [ +] Tests for the changes have been added / updated. - [ -] Documentation comments have been added / updated.
# What ❔
Fixed disabled (xdescribe and xit)
/transactions/{transactionHash}/transfers tests and turned them on back
## Why ❔
/transactions/{transactionHash}/transfers tests were disabled since they
were outdated
## Checklist
- [ +] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ +] Tests for the changes have been added / updated.
- [ -] Documentation comments have been added / updated.
Unit Test Results 4 files 263 suites 13m 24s ⏱️ Results for commit bf16f2a. ♻️ This comment has been updated with latest results. |
API E2E Test Results206 tests 206 ✅ 27s ⏱️ Results for commit bf16f2a. ♻️ This comment has been updated with latest results. |
|
Visit the preview URL for this PR (updated for commit bf16f2a):
(expires Fri, 16 Feb 2024 20:01:12 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: e508f9012944951194447cb8885950b451a24403 |
vasyl-ivanchuk
approved these changes
Feb 12, 2024
pcheremu
added a commit
that referenced
this pull request
Feb 15, 2024
pcheremu
added a commit
that referenced
this pull request
Jul 5, 2024
pcheremu
added a commit
that referenced
this pull request
Aug 22, 2024
pcheremu
added a commit
that referenced
this pull request
Aug 22, 2024
pcheremu
added a commit
that referenced
this pull request
Aug 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What ❔
This is git rebase of integration-tests to main.
$ git merge-base main integration-tests4ceb487
changes:
$ git log --first-parent --graph --oneline --decorate 4ceb4878c3a4850b4d081a871fdf347bbd65c560..HEADWhy ❔
Keep integration tests up to date
Please, double check the docker-compose.yaml
Checklist