Charlie Thomas

Results 8 comments of Charlie Thomas

This is also happening to me. Reverting back to 0.38.0 fixed it for me.

> @charlesbthomas do you also use Sentinel One? Looks like it. This is a new workstation.

@tgodzik I will work on putting one together!

@tgodzik Here is an example repo that replicates the issue. https://github.com/charlesbthomas/testng-example Screenshots: Working code lens on the app: No code lens on the tests:

I am not familiar with the metals codebase (just a happy user), but my hunch is that we need to implement something like: https://github.com/scalameta/metals/blob/2c8d344da6dc8b4e920612e4336b28a493f021d0/metals/src/main/scala/scala/meta/internal/metals/testProvider/frameworks/JunitTestFinder.scala#L13 but for TestNG.

> Sure, but without it it should still be possible to run the whole test suite. If you pull that example down, you can see that it is unable to...

@tgodzik any ideas on where someone might start with this? I am pretty motivated to fix it and willing to work on the fix as a contribution, but not sure...

@tgodzik I tried adding sbt-testng to my test project above's classpath and manually editing the bloop config to also include: ``` { "names": [ "de.johoop.testnginterface.TestNGFramework" ] } ``` Still no...