Skip to content

Commit 61306a6

Browse files
disabling more zeppline tests that expect reverts with specific errors and they're broken via IR due to Hardhat heuristics.
Updated zeppelin.sh Added cautionary comment for two sed commands that depend on the order of occurrence of the patterns in the files. L122, and L123 both are referencing the 3rd occurrence of the patterns from their respective files. This could result in an error in the future when the code is further modified.
1 parent 035f6ab commit 61306a6

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

test/externalTests/zeppelin.sh

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,11 @@ function zeppelin_test
106106
sed -i "s|it(\('other accounts cannot pause'\)|it.skip(\1|g" test/token/ERC721/presets/ERC721PresetMinterPauserAutoId.test.js
107107
sed -i "s|it(\('other accounts cannot unpause'\)|it.skip(\1|g" test/token/ERC721/presets/ERC721PresetMinterPauserAutoId.test.js
108108
sed -i "s|it(\('prevents initialization'\)|it.skip(\1|g" test/proxy/utils/Initializable.test.js
109+
sed -i "s|it(\('divide by 0'\)|it.skip(\1|g" test/utils/math/Math.test.js
110+
# CAUTION:: The following two sed commands depend on the order of occurrence of the relevant patterns in the mentioned files.
111+
# Could result in an error in the future.
112+
sed -zi "s|it(\('deposit'\)|it.skip(\1|3" test/token/ERC20/extensions/ERC20TokenizedVault.test.js
113+
sed -zi "s|it(\('withdraw'\)|it.skip(\1|3" test/token/ERC20/extensions/ERC20TokenizedVault.test.js
109114

110115
# TODO: Remove this when https://github.com/NomicFoundation/hardhat/issues/2115 gets fixed.
111116
sed -i "s|describe\(('Polygon-Child'\)|describe.skip\1|g" test/crosschain/CrossChainEnabled.test.js

0 commit comments

Comments
 (0)