search

we-sh / 42ShellTester

Integration test suite for Shell implementation
47 stars 5 forks source link

Corrected 21sh/misc/002-simple-command-line #14

Closed enerdhil closed 7 years ago

enerdhil commented 7 years ago

I tested it with bash and zsh and both write 48 in place of 66 on stdout.

Here's what i get with bash http://pastebin.com/WGj7MP6y

gabkk commented 7 years ago

@enerdhil yep you are right, we have modified the Token's size which is generated. For some tests we are still using an hardcoded value instead of a generic one,i ll fix it today thx for the issue report. The correct value for this test is 48.

enerdhil commented 7 years ago

Nice ! Thanks.