Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: integration tests setup #46

Draft
wants to merge 146 commits into
base: main
Choose a base branch
from
Draft

feat: integration tests setup #46

wants to merge 146 commits into from

Conversation

Romsters
Copy link
Collaborator

@Romsters Romsters commented Oct 10, 2023

What ❔

Integration tests setup

@github-actions
Copy link

github-actions bot commented Oct 10, 2023

API E2E Test Results

206 tests   206 ✅  28s ⏱️
 14 suites    0 💤
  1 files      0 ❌

Results for commit 946b0f6.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Oct 10, 2023

Visit the preview URL for this PR (updated for commit 946b0f6):

(expires Wed, 21 Feb 2024 14:41:27 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e508f9012944951194447cb8885950b451a24403

@github-actions
Copy link

github-actions bot commented Oct 10, 2023

Unit Test Results

    4 files    263 suites   13m 42s ⏱️
2 079 tests 2 078 ✅ 1 💤 0 ❌
2 284 runs  2 283 ✅ 1 💤 0 ❌

Results for commit 946b0f6.

♻️ This comment has been updated with latest results.

abilevych and others added 17 commits January 18, 2024 16:46
…I-requests

chore: retry wrapper over api requests
# 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.
@pcheremu pcheremu added the automation writing/fixing/maintaining auto-tests/e2e-tests label Feb 13, 2024
@pcheremu pcheremu self-assigned this Feb 13, 2024
docker-compose.yaml Outdated Show resolved Hide resolved
.github/workflows/app-e2e.yml Show resolved Hide resolved
# What ❔

Remove docker version

## Why ❔

Because it's not necessary to specify it.
@pcheremu pcheremu marked this pull request as draft August 21, 2024 12:22
@pcheremu pcheremu mentioned this pull request Aug 28, 2024
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation writing/fixing/maintaining auto-tests/e2e-tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants