fix(cli/coverage): invalid line id in html reporter

[lowlighter]

The name for the anchors are prefixed with a L, but the links are currently missing it so they don't work currently:

[bartlomieju]

Thanks, any chance you could update the existing HTML reporter test to exercise this change?

[lowlighter]

@bartlomieju I can, however I'm not sure where it is located. Is it supposed to be somewhere in the same rust file or is it like in another test folder ?

[bartlomieju]

@lowlighter there is a test_html_reporter test in tests/integration/coverage_tests.rs that I think you could update.

[kt3k]

The change looks good to me.

@lowlighter
The test case executes something like the below:

deno test --quiet --coverage=cov_file tests/testdata/coverage/multisource/
deno coverage cov_file --html

and asserts specific part of output html files. Maybe let's assert foo.ts.html has <a name='L1'></a><a href='#L1'>1</a>?

Nevermind. You've already done it.

[kt3k]

LGTM. Thanks for the fix!

[bartlomieju]

LGTM too. Thanks!