Skip to content

Commit c70f8c5

Browse files
authored
Merge pull request #12629 from ethereum/reenable-bleeps-ext-test-without-governor-test
Re-enable Bleeps external test without the failing governor test
2 parents 93d5b79 + b52032a commit c70f8c5

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

.circleci/config.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1468,8 +1468,7 @@ workflows:
14681468
- t_ems_ext: *job_native_test_ext_trident
14691469
- t_ems_ext: *job_native_test_ext_euler
14701470
- t_ems_ext: *job_native_test_ext_yield_liquidator
1471-
# Disabled until we have a fix for https://github.com/wighawag/bleeps/issues/2
1472-
#-t_ems_ext: *job_native_test_ext_bleeps
1471+
- t_ems_ext: *job_native_test_ext_bleeps
14731472
- t_ems_ext: *job_native_test_ext_pool_together
14741473
- t_ems_ext: *job_native_test_ext_perpetual_pools
14751474
- t_ems_ext: *job_native_test_ext_uniswap

test/externalTests/bleeps.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,10 @@ function bleeps_test
7272
sed -i 's/msg\.sender\.transfer(/payable(msg.sender).transfer(/g' src/externals/WETH9.sol
7373
sed -i 's/^\s*\(Deposit\|Withdrawal\|Approval\|Transfer\)(/emit \1(/g' src/externals/WETH9.sol
7474

75+
# This test does not currently pass due to an upstream problem.
76+
# TODO: Remove this line when https://github.com/wighawag/bleeps/issues/2 is fixed
77+
rm test/BleepsDAO.governor.test.ts
78+
7579
neutralize_package_lock
7680
neutralize_package_json_hooks
7781
force_hardhat_compiler_binary "$config_file" "$BINARY_TYPE" "$BINARY_PATH"

0 commit comments

Comments
 (0)