fix: Correctly pick whole file context

[p12tic]

Currently the current line is duplicated in both "before" and "after" context. This is due to DocumentContextReader::readWholeFileAfter() picking the current line part of which has been already included into the "before" context.

[Palm1r]

It should, because the "suggestion" can be in the middle of string like here

And indeed the problem with symbols after:

    "prompt": "QQuickWindow::setGraphicsApi(QSGRendererInterface::OpenGL);\n\n    QStringList colors{};\n    123",
    "suffix": "\n    QObject::connect(\n        &engine,\n        &QQmlApplicationEngine::objectCreationFailed,\n        &app,\n        []() { QCoreApplication::exit(-1); },\n        Qt::QueuedConnection);\n    engine.loadFromModule(\"TestRowlayout\", \"Main\");\n\n    return app.exec();\n}\n",

I lost here "456"
Do you want to fix that? Or I will take it?

[p12tic]

Do you want to fix that? Or I will take it?

I can fix it. I did not know what is the expected behavior, in fact your approach was how I started fixing the bug :-) I think I have stash somewhere still.

[p12tic]

@Palm1r Pushed fixes. This PR is very similar to your PR, except that I make getContextBetween() more generic so that it can pick characters from any two points in a file.

[p12tic]

There are still semi-related problems, will fix these via separate PRs.

[Palm1r]

but seems startCursorPosition and endCursorPosition can be same

[Palm1r]

just cursorPosition

[p12tic]

No, because the function does not know which cursor cursorPosition refers to. It may be referring to the start line, or it can refer to end line.

[p12tic]

I've added handling for way more edge cases, and also appropriate tests.