_zunit_run_testfile:21: tests: assignment to invalid subscript range

[pawamoy]

$ cat tests/test_cover.zunit
#!/usr/bin/env zunit

@test "cover prints help with -h option" {
  run cover -h
  assert $state equals 0
}
    

$ zunit tests/test_cover.zunit
Launching ZUnit                                            ZUnit: 0.8.2
ZSH:   zsh 5.3.1 (x86_64-debian-linux-gnu)                 
_zunit_run_testfile:21: tests: assignment to invalid subscript range

[pawamoy]

OK, seems to be because I used double-quotes for my test name instead of single-quotes. Is it mentioned in the README/docs?

I'm actually giving a try at zunit by migrating bats tests :)

[molovo]

@pawamoy it's not - I wasn't aware it was an issue. It should accept both so I'll look into a fix

[MichaelAquilina]

Can confirm this also occurs on my own tests when swapping ' for "

[pawamoy]

I think it's caused by this line:

zunit/src/commands/run.zsh

Line 259 in e58c49a

testname="${line[(( ${line[(i)[\']]}+1 )),(( ${line[(I)[\']]}-1 ))]}"

You are using \' to get the test name boundaries. When using double-quotes instead, the test name is empty, and then later at

zunit/src/commands/run.zsh

Line 271 in e58c49a

tests[${#test_names}]=''

...you are trying to assign a value at index 0, when arrays start at 1 in Zsh, hence the error we get

_zunit_run_testfile:21: tests: assignment to invalid subscript range

Maybe a simple fix would be to use

-testname="${line[(( ${line[(i)[\']]}+1 )),(( ${line[(I)[\']]}-1 ))]}" 
+testname="${line[(( ${line[(i)[\'\"]]}+1 )),(( ${line[(I)[\'\"]]}-1 ))]}" 

instead? Not sure about the escaping, or if [] really are regex characters ranges.