Details
Details
- Reviewers
parolanilsson
Diff Detail
Diff Detail
- Repository
- R60 KDevelop Krazy2
- Branch
- master
- Lint
No Linters Available - Unit
No Unit Test Coverage
Comment Actions
Does that fix issues in the unit test? If yes => go for it, otherwise please add an unit test if possible.
If you want to work on kdev-krazy, please try to get the unit tests working. That would be awesome.
For me they fail unrelated to whether krazy2 is installed or not. Both cases should "work", i.e. the test run should succeed.
Comment Actions
I saw that detected problems was off by one line.
When working on kdec-cppcheck I saw that KDevelop::DocumentRange seems to be 0-based while the result from Krazy2 is 1-based.
I now realize I changed the wrong location.
I will also need to update all tests checking line number.
I'll keep looking into this.