Skip to content

Init Testing setup #10954

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 65 commits into
base: master
Choose a base branch
from
Open

Init Testing setup #10954

wants to merge 65 commits into from

Conversation

@treeoflife2
Copy link
Member

@treeoflife2 treeoflife2 commented Nov 6, 2025

No description provided.

@treeoflife2 treeoflife2 marked this pull request as draft November 6, 2025 13:57
@treeoflife2
Copy link
Member Author

treeoflife2 commented Nov 6, 2025 

 PASS  api-tests/src/tvl/protocols.test.ts (7.287 s)
  TVL API - Protocols
    GET /protocols                                                                                               
      √ should return list of all protocols (6 ms)                                                               
      √ should return protocols with required fields (2757 ms)                                                   
      √ should return protocols with valid data types (8 ms)                                                     
      √ should return protocols with valid TVL values (2814 ms)                                                  
      √ should return protocols with valid chain TVLs structure (69 ms)                                          
      √ should handle protocols with borrowed amounts (25 ms)                                                    
      √ should return protocols ordered by TVL descending (11 ms)                                                
      √ should have consistent data structure (15 ms)                                                            
      √ should return protocols with categories (4 ms)                                                           
      √ should return protocols with valid percentage changes (9 ms)                                             
      √ should handle protocols with parent protocol slug (6 ms)                                                 
      √ should validate complete protocol structure (2 ms)                                                       
                                                                                                                 
Test Suites: 1 passed, 1 total                                                                                   
Tests:       12 passed, 12 total                                                                                 
Snapshots:   0 total
Time:        7.578 s, estimated 10 s
Ran all test suites matching /api-tests/i.

@treeoflife2 treeoflife2 requested a review from g1nt0ki November 6, 2025 13:57
g1nt0ki
g1nt0ki reviewed Nov 6, 2025
View reviewed changes
Copy link
Member

@g1nt0ki g1nt0ki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing pnpm lock file update?

g1nt0ki
g1nt0ki reviewed Nov 6, 2025
View reviewed changes
defi/jest.config.js
maxWorkers: 1,
ci: false,
cacheDirectory: '/tmp/jest-cache',
testEnvironment: "node",
Copy link
Member

@g1nt0ki g1nt0ki Nov 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hmm, unsure if this will break current github action flow that has a dependency on test passing before deploying to aws

g1nt0ki
g1nt0ki reviewed Nov 7, 2025
View reviewed changes
defi/api-tests/README.md
1. **Cache responses** - Use `beforeAll` to fetch once
2. **Group by purpose** - Basic → Data → Metadata → Edge Cases
3. **Test samples** - Don't iterate all items (use `.slice(0, 10)`)
Copy link
Member

@g1nt0ki g1nt0ki Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what I would do here is, take the top 10 (usually most valuable protocols) & and a random 10, maybe add a helper for this?

g1nt0ki
g1nt0ki reviewed Nov 7, 2025
View reviewed changes
defi/api-tests/jest.config.js
outputName: 'junit.xml',
}]
],
coverageDirectory: './coverage',
Copy link
Member

@g1nt0ki g1nt0ki Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we can skip the coverage as these are not unit tests but e2e?

Copy link
Member

@g1nt0ki g1nt0ki Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure if there is a config to disable it in the config

g1nt0ki
g1nt0ki reviewed Nov 7, 2025
View reviewed changes
defi/package.json Outdated
"test:api:coverage": "jest --testPathPattern=api-tests --maxWorkers=1 --coverage",
"test:api": "cd api-tests && jest",
"test:api:watch": "cd api-tests && jest --watch",
"test:api:coverage": "cd api-tests && jest --coverage",
Copy link
Member

@g1nt0ki g1nt0ki Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove coverage?

Copy link
Member Author

@treeoflife2 treeoflife2 Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

okay

g1nt0ki and others added 18 commits November 7, 2025 17:06
@g1nt0ki
minor fix
3aa844f
@g1nt0ki
remove cli support (use ui-tool)
f092c03
@g1nt0ki
report test failures to discord
75c74e9
@g1nt0ki
add json reporter
14b1b87
@Define101 @g1nt0ki
obol token data
3350861
@waynebruce0x @g1nt0ki
Coins decimals type (DefiLlama#10945)
67e6ffd 
* number

* check decimals before transform

* avnu address padding
@g1nt0ki
update osmosis twitter handle
90643fe
@FabienCoutant @g1nt0ki
feat: add erc4626 sUSDr address on Sei (DefiLlama#10947)
fa89a4a
@Silas-Zhu @g1nt0ki
asseto aoabt add Avalanche's token (DefiLlama#10939)
d66a215
@realdealshaman @g1nt0ki
add new listing: enable dimensions
5b80f46
@realdealshaman @g1nt0ki
move some listings to data5
604e7ac
@realdealshaman @g1nt0ki
add new listings
56c5015
@realdealshaman @g1nt0ki
add EthStrategy tvl
51bd029
@waynebruce0x @g1nt0ki
Mappings (DefiLlama#10950)
6f5bf34 
* token mappings

* makina name
@vrtnd @g1nt0ki
rely on slug form bridges-server data (DefiLlama#10936)
4ca269e 
Co-authored-by: vrtnd <[email protected]>
@waynebruce0x @g1nt0ki
ssui (DefiLlama#10955)
bf5ff2d
@realdealshaman @g1nt0ki
add audit link
ae80367
@swifth @everyding @g1nt0ki
add tvl、volume、fee three meters for dipcoin spot and dipcoin perps. a…
fe24dbe 
…nd combine dipcoin spot and dipcoin perps into one dipcoin group. (DefiLlama#10931)

* Add DipCoin Perp TVL Adapter for SUI

* add tvl、volume、fee three meters for dipcoin spot and dipcoin perps. and combine dipcoin spot and dipcoin perps into one dipcoin group.

* add tvl、volume、fee three meters for dipcoin spot and dipcoin perps. and combine dipcoin spot and dipcoin perps into one dipcoin group.

* add tvl、volume、fee three meters for dipcoin spot and dipcoin perps. and combine dipcoin spot and dipcoin perps into one dipcoin group.

---------

Co-authored-by: alan <[email protected]>
@treeoflife2 treeoflife2 marked this pull request as ready for review November 13, 2025 18:01
@treeoflife2
Copy link
Member Author

treeoflife2 commented Nov 15, 2025

Writing JSON test report to ./test-results/junit.json
Seed: 1888326014
Test Suites: 15 passed, 15 total
Tests: 203 passed, 203 total
Snapshots: 0 total
Time: 56.782 s
Ran all test suites.

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

Reviewers

@g1nt0ki g1nt0ki g1nt0ki left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.