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

Unskip stack-too-deep unit test #848

Closed

cgewecke opened this issue Feb 4, 2024 · 1 comment

Labels

viaIR

Member

cgewecke commented Feb 4, 2024

A self-reminder to do this as part of supporting viaIR. It began failing when the tests were upgraded to solc 0.8.24

solidity-coverage/test/units/statements.js

Lines 54 to 58 in 6739db6

    
           // FAILING WITH 0.8.x ... unskip when viaIR work is done 
        
           it.skip('should instrument without triggering stack-too-deep', () => { 
        
             const info = util.instrumentAndCompile('statements/stack-too-deep'); 
        
             util.report(info.solcOutput.errors); 
        
           });

cgewecke added the viaIR label

cgewecke mentioned this issue

Enable coverage when viaIR compiler flag is true #854

Merged

7 tasks

Member Author

cgewecke commented Feb 9, 2024

Done in #854

cgewecke closed this as completed

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