fix: nil error on test single function

[michaldziurowski]

Removed reference from not initialized file_id variable to value.id.
Fixes: #47

[zankich]

Just tried this branch out and it fixes the issue!

[sergii4]

Hi @michaldziurowski, it fails test lua/spec/neotest-go/init_spec.lua. Ideally, it shouldn't fix one issue and create another.

Success ||      prepare_results check that we have file level test result as well as all nested results in cases_test.go
Fail    ||      prepare_results check that we have correct file level test result status cases_test.go
            ...re/nvim/site/pack/packer/start/neotest/lua/nio/tests.lua:27: Test task failed with message:
            The coroutine failed with this message: 
            ...acker/start/neotest-go/lua/spec/neotest-go/init_spec.lua:272: Expected objects to be equal.
            Passed in:
            (string) 'passed'
            Expected:
            (string) 'failed'
            stack traceback:
                [C]: in function 'error'
                ...e/pack/packer/start/plenary.nvim/lua/luassert/assert.lua:51: in function 'equals'
                ...acker/start/neotest-go/lua/spec/neotest-go/init_spec.lua:272: in function <...acker/start/neotest-go/lua/spec/neotest-go/init_spec.l
ua:244>
            
            stack traceback:
                ...re/nvim/site/pack/packer/start/neotest/lua/nio/tests.lua:27: in function <...re/nvim/site/pack/packer/start/neotest/lua/nio/tests.lu
a:12>
            

[zankich]

@sergii4 what are the steps for running the test suite?

[michaldziurowski]

Hi @michaldziurowski, it fails test lua/spec/neotest-go/init_spec.lua. Ideally, it shouldn't fix one issue and create another.

Yep, I rushed a bit with removal of file_id, sorry. Fixed. Will provide some additional tests later.
Sidenote: it seems that there are other test which are also failing in init_spec.lua not related to this change, or is it just my env ?

what are the steps for running the test suite?

I'm new to lua so maybe there is better way but I'm using plenary test runner via :PlenaryBustedDirectory lua/spec from inside of neotest-go dir.

[zankich]

hey @michaldziurowski I also have a PR which fixes this and related diagnostic errors while parsing test output #49

[akarl]

Isn't file_id always nil here?

[sergii4]

Why should it be? We always have file level test result

[sergii4]

Hi @michaldziurowski to run test you need:
#49 (comment)

[sergii4]

Hi @michaldziurowski, yeah, it seems pretty logical that attempt index nil value caused by nil. I tested your change, it works for me and tests pass.

Thank you for sparing your time on it