> On 6. Apr 2021, at 7.23, Jun T wrote: > > Aha! > It was a mystery for me how you could run Y01 successfully. > Sorry for not going back to your original post. > I believe Oliver was just too busy when committing the patch. > > But the test still fails as follows: > > --- /tmp/zsh.ztst.34237/ztst.out 2021-04-06 13:06:20.000000000 +0900 > +++ /tmp/zsh.ztst.34237/ztst.tout 2021-04-06 13:06:20.000000000 +0900 > @@ -8,9 +8,9 @@ > NO:{dir1 dir2 file1 file2} > DESCRIPTION:{original} > NO:{*} > -line: {: dir1/}{} > line: {: dir2/}{} > line: {: file1 }{} > line: {: file2 }{} > line: {: dir1 dir2 file1 file2 }{} > line: {: *}{} > +line: {: dir1/}{} > Test ./Y01completion.ztst failed: output differs from expected as shown above for: > comptest $': *\t\t\t\t\t\t\t' > Was testing: _expand shows file types > ./Y01completion.ztst: test failed. > > Can you reproduce this? > Is it OK to replace "comptest $': *\t\t\t\t\t\t\t'" by "comptest $': *\t'" > as in my patch in workers/48363⁩? Yes, I can reproduce it. However, I don’t think we should discard all of the additional `\t`s, since it also tests the suffixes that _expand adds, which are different from _complete. I would like to suggest a slightly different fix. See attached patch.