207 - Integrate foundry tests #214

github-actions · 2023-11-26T20:38:56Z

Solves #207

0xSwapFeeder

few cleaning left to do but lgtm 👍

toolchains/solidity/extension/src/tests/test-manager.ts

toolchains/solidity/core/crates/tests-positions-server/src/main.rs

toolchains/solidity/extension/src/tests/test-manager.ts

…retrieve contracts and tests positions

…structure to add contract and tests names

…code testing API

…tract tests were relaunching all children tests

… based on foundry naming convention

0xtekgrinder

LGTM

0xmemorygrinder force-pushed the feature/207-integrate-foundry-tests-staging branch 2 times, most recently from 20aa5a2 to 50f7a1f Compare December 6, 2023 05:38

0xmemorygrinder force-pushed the feature/207-integrate-foundry-tests-staging branch 2 times, most recently from 7d3318f to 51bee47 Compare December 15, 2023 18:26

Leon-azerty requested review from 0xSwapFeeder and 0xtekgrinder January 4, 2024 20:45

Leon-azerty marked this pull request as ready for review January 4, 2024 20:45

0xSwapFeeder approved these changes Jan 6, 2024

View reviewed changes

toolchains/solidity/extension/src/tests/test-manager.ts Outdated Show resolved Hide resolved

toolchains/solidity/extension/src/tests/test-manager.ts Show resolved Hide resolved

toolchains/solidity/extension/src/tests/test-manager.ts Show resolved Hide resolved

0xSwapFeeder assigned 0xtekgrinder Jan 6, 2024

0xtekgrinder suggested changes Jan 9, 2024

View reviewed changes

toolchains/solidity/core/crates/tests-positions-server/src/main.rs Outdated Show resolved Hide resolved

toolchains/solidity/extension/src/tests/test-manager.ts Show resolved Hide resolved

0xmemorygrinder requested a review from 0xtekgrinder January 9, 2024 23:57

Astrodevs CI and others added 17 commits January 9, 2024 19:36

chore: create branch feature/207-integrate-foundry-tests-staging

ff32f7a

feat(solidity/core/tests-positions-server): added language server to …

42f91b2

…retrieve contracts and tests positions

feat(solidity/extension): launch tests-positions language server

384c186

chore: deleted remove-me

17ff06d

chore(solidity/core/tests-positions): ran lint and format

86f13f2

feat(solidity/extension): functions to run foundry tests

24d6731

refactor(solidity./core/tests-positions-server): refactored response …

971eaa1

…structure to add contract and tests names

feat(solidity/extension): added TestManager class to interact with vs…

522bbd5

…code testing API

fix(solidity/extension): compilation errors after rebase

896503a

fix(solidity/extension): tests were never failing and launching a con…

ae50d0f

…tract tests were relaunching all children tests

fix(solidity/extension): this was not callable

a34aca0

feat: show logs when test failed

0e0076e

fix: bad cargo.lock merge

567487a

chore: ran cargo fmt

dd2ae99

refactor(solidity/extension/tests): removed useless comment

8d70c30

refactor(solidity/core/tests-positions-server): filter test functions…

27cb9ab

… based on foundry naming convention

chore: fixed lint issue

ff9213d

0xmemorygrinder force-pushed the feature/207-integrate-foundry-tests-staging branch from 8de3b03 to ff9213d Compare January 10, 2024 00:39

0xtekgrinder approved these changes Jan 10, 2024

View reviewed changes

0xmemorygrinder merged commit 5de0680 into dev Jan 10, 2024
46 checks passed

0xmemorygrinder deleted the feature/207-integrate-foundry-tests-staging branch January 10, 2024 16:02

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

207 - Integrate foundry tests #214

207 - Integrate foundry tests #214

github-actions bot commented Nov 26, 2023

0xSwapFeeder left a comment

0xtekgrinder left a comment

207 - Integrate foundry tests #214

207 - Integrate foundry tests #214

Conversation

github-actions bot commented Nov 26, 2023

0xSwapFeeder left a comment

Choose a reason for hiding this comment

0xtekgrinder left a comment

Choose a reason for hiding this comment