forked from qt-creator/qt-creator
C++: Fix class scope completion for templates.
* You now get completion for std::vector<int>::[complete]. * Also added a test. Conflicts: src/plugins/cpptools/cppcompletion_test.cpp src/plugins/cpptools/cpptoolsplugin.h Change-Id: I596ebf6bd18ec9a347113f8d162cc124c8a0d6b4 Reviewed-by: hjk <qthjk@ovi.com>
This commit is contained in:
Christian Kamm
hjk
198
src/plugins/cpptools/cppcompletion_test.cpp
Normal file
198
src/plugins/cpptools/cppcompletion_test.cpp
Normal file
@@ -0,0 +1,198 @@
|
|||||||
|
/**************************************************************************
|
||||||
|
**
|
||||||
|
** This file is part of Qt Creator
|
||||||
|
**
|
||||||
|
** Copyright (c) 2012 Nokia Corporation and/or its subsidiary(-ies).
|
||||||
|
**
|
||||||
|
** Contact: Nokia Corporation (qt-info@nokia.com)
|
||||||
|
**
|
||||||
|
**
|
||||||
|
** GNU Lesser General Public License Usage
|
||||||
|
**
|
||||||
|
** This file may be used under the terms of the GNU Lesser General Public
|
||||||
|
** License version 2.1 as published by the Free Software Foundation and
|
||||||
|
** appearing in the file LICENSE.LGPL included in the packaging of this file.
|
||||||
|
** Please review the following information to ensure the GNU Lesser General
|
||||||
|
** Public License version 2.1 requirements will be met:
|
||||||
|
** http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html.
|
||||||
|
**
|
||||||
|
** In addition, as a special exception, Nokia gives you certain additional
|
||||||
|
** rights. These rights are described in the Nokia Qt LGPL Exception
|
||||||
|
** version 1.1, included in the file LGPL_EXCEPTION.txt in this package.
|
||||||
|
**
|
||||||
|
** Other Usage
|
||||||
|
**
|
||||||
|
** Alternatively, this file may be used in accordance with the terms and
|
||||||
|
** conditions contained in a signed written agreement between you and Nokia.
|
||||||
|
**
|
||||||
|
** If you have questions regarding the use of this file, please contact
|
||||||
|
** Nokia at qt-info@nokia.com.
|
||||||
|
**
|
||||||
|
**************************************************************************/
|
||||||
|
|
||||||
|
#include "cpptoolsplugin.h"
|
||||||
|
|
||||||
|
#include <AST.h>
|
||||||
|
#include <Control.h>
|
||||||
|
#include <CppDocument.h>
|
||||||
|
#include <DiagnosticClient.h>
|
||||||
|
#include <Scope.h>
|
||||||
|
#include <TranslationUnit.h>
|
||||||
|
#include <Literals.h>
|
||||||
|
#include <Bind.h>
|
||||||
|
#include <Symbols.h>
|
||||||
|
#include <utils/changeset.h>
|
||||||
|
#include <texteditor/basetextdocument.h>
|
||||||
|
#include <texteditor/plaintexteditor.h>
|
||||||
|
#include <texteditor/codeassist/iassistproposal.h>
|
||||||
|
#include <texteditor/codeassist/iassistproposalmodel.h>
|
||||||
|
#include <texteditor/codeassist/basicproposalitemlistmodel.h>
|
||||||
|
#include <cpptools/cpptoolsplugin.h>
|
||||||
|
#include <cpptools/cppcompletionassist.h>
|
||||||
|
#include <extensionsystem/pluginmanager.h>
|
||||||
|
#include <utils/fileutils.h>
|
||||||
|
|
||||||
|
#include <QtTest>
|
||||||
|
#include <QtDebug>
|
||||||
|
#include <QTextDocument>
|
||||||
|
#include <QDir>
|
||||||
|
|
||||||
|
/*!
|
||||||
|
Tests for code completion.
|
||||||
|
*/
|
||||||
|
using namespace CPlusPlus;
|
||||||
|
using namespace CppTools;
|
||||||
|
using namespace CppTools::Internal;
|
||||||
|
using namespace TextEditor;
|
||||||
|
using namespace Core;
|
||||||
|
|
||||||
|
struct TestData
|
||||||
|
{
|
||||||
|
QByteArray srcText;
|
||||||
|
int pos;
|
||||||
|
Snapshot snapshot;
|
||||||
|
BaseTextEditorWidget *editor;
|
||||||
|
QTextDocument *doc;
|
||||||
|
};
|
||||||
|
|
||||||
|
static QStringList getCompletions(TestData &data)
|
||||||
|
{
|
||||||
|
QStringList completions;
|
||||||
|
|
||||||
|
CppCompletionAssistInterface *ai = new CppCompletionAssistInterface(data.editor->document(), data.pos,
|
||||||
|
data.editor->editorDocument(), ExplicitlyInvoked,
|
||||||
|
data.snapshot, QStringList(), QStringList());
|
||||||
|
CppCompletionAssistProcessor processor;
|
||||||
|
IAssistProposal *proposal = processor.perform(ai);
|
||||||
|
if (!proposal)
|
||||||
|
return completions;
|
||||||
|
IAssistProposalModel *model = proposal->model();
|
||||||
|
if (!model)
|
||||||
|
return completions;
|
||||||
|
BasicProposalItemListModel *listmodel = dynamic_cast<BasicProposalItemListModel *>(model);
|
||||||
|
if (!listmodel)
|
||||||
|
return completions;
|
||||||
|
|
||||||
|
for (int i = 0; i < listmodel->size(); ++i)
|
||||||
|
completions << listmodel->text(i);
|
||||||
|
|
||||||
|
return completions;
|
||||||
|
}
|
||||||
|
|
||||||
|
static void setup(TestData *data)
|
||||||
|
{
|
||||||
|
data->pos = data->srcText.indexOf('@');
|
||||||
|
QVERIFY(data->pos != -1);
|
||||||
|
data->srcText[data->pos] = ' ';
|
||||||
|
Document::Ptr src = Document::create(QDir::tempPath() + QLatin1String("/file.h"));
|
||||||
|
Utils::FileSaver srcSaver(src->fileName());
|
||||||
|
srcSaver.write(data->srcText);
|
||||||
|
srcSaver.finalize();
|
||||||
|
src->setUtf8Source(data->srcText);
|
||||||
|
src->parse();
|
||||||
|
src->check();
|
||||||
|
|
||||||
|
data->snapshot.insert(src);
|
||||||
|
|
||||||
|
data->editor = new PlainTextEditorWidget(0);
|
||||||
|
QString error;
|
||||||
|
data->editor->open(&error, src->fileName(), src->fileName());
|
||||||
|
|
||||||
|
data->doc = data->editor->document();
|
||||||
|
}
|
||||||
|
|
||||||
|
void CppToolsPlugin::test_completion_basic_1()
|
||||||
|
{
|
||||||
|
TestData data;
|
||||||
|
data.srcText = "\n"
|
||||||
|
"class Foo\n"
|
||||||
|
"{\n"
|
||||||
|
" void foo();\n"
|
||||||
|
" int m;\n"
|
||||||
|
"};\n"
|
||||||
|
"\n"
|
||||||
|
"void func() {\n"
|
||||||
|
" Foo f;\n"
|
||||||
|
" @\n"
|
||||||
|
" // padding so we get the scope right\n"
|
||||||
|
"}";
|
||||||
|
|
||||||
|
setup(&data);
|
||||||
|
|
||||||
|
QStringList basicCompletions = getCompletions(data);
|
||||||
|
|
||||||
|
QVERIFY(!basicCompletions.contains("foo"));
|
||||||
|
QVERIFY(!basicCompletions.contains("m"));
|
||||||
|
QVERIFY(basicCompletions.contains("Foo"));
|
||||||
|
QVERIFY(basicCompletions.contains("func"));
|
||||||
|
QVERIFY(basicCompletions.contains("f"));
|
||||||
|
|
||||||
|
Utils::ChangeSet change;
|
||||||
|
change.insert(data.pos, "f.");
|
||||||
|
QTextCursor cursor(data.doc);
|
||||||
|
change.apply(&cursor);
|
||||||
|
data.pos += 2;
|
||||||
|
|
||||||
|
QStringList memberCompletions = getCompletions(data);
|
||||||
|
|
||||||
|
QVERIFY(memberCompletions.contains("foo"));
|
||||||
|
QVERIFY(memberCompletions.contains("m"));
|
||||||
|
QVERIFY(!memberCompletions.contains("func"));
|
||||||
|
QVERIFY(!memberCompletions.contains("f"));
|
||||||
|
}
|
||||||
|
|
||||||
|
void CppToolsPlugin::test_completion_template_1()
|
||||||
|
{
|
||||||
|
TestData data;
|
||||||
|
data.srcText = "\n"
|
||||||
|
"template <class T>\n"
|
||||||
|
"class Foo\n"
|
||||||
|
"{\n"
|
||||||
|
" typedef T Type;\n"
|
||||||
|
" T foo();\n"
|
||||||
|
" T m;\n"
|
||||||
|
"};\n"
|
||||||
|
"\n"
|
||||||
|
"void func() {\n"
|
||||||
|
" Foo f;\n"
|
||||||
|
" @\n"
|
||||||
|
" // padding so we get the scope right\n"
|
||||||
|
"}";
|
||||||
|
|
||||||
|
setup(&data);
|
||||||
|
|
||||||
|
Utils::ChangeSet change;
|
||||||
|
change.insert(data.pos, "Foo::");
|
||||||
|
QTextCursor cursor(data.doc);
|
||||||
|
change.apply(&cursor);
|
||||||
|
data.pos += 5;
|
||||||
|
|
||||||
|
QStringList completions = getCompletions(data);
|
||||||
|
|
||||||
|
QVERIFY(completions.contains("Type"));
|
||||||
|
QVERIFY(completions.contains("foo"));
|
||||||
|
QVERIFY(completions.contains("m"));
|
||||||
|
QVERIFY(!completions.contains("T"));
|
||||||
|
QVERIFY(!completions.contains("f"));
|
||||||
|
QVERIFY(!completions.contains("func"));
|
||||||
|
}
|
||||||
@@ -1432,6 +1432,13 @@ bool CppCompletionAssistProcessor::completeScope(const QList<CPlusPlus::LookupIt
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
} else if (Template *templ = ty->asTemplateType()) {
|
||||||
|
if (!result.binding())
|
||||||
|
continue;
|
||||||
|
if (ClassOrNamespace *b = result.binding()->lookupType(templ->name())) {
|
||||||
|
completeClass(b);
|
||||||
|
break;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user