Differential D25767 Diff 70968 autotests/kautosavefiletest.cpp

Changeset View

Standalone View

autotests/kautosavefiletest.cpp

Show First 20 Lines • Show All 72 Lines • ▼ Show 20 Line(s)		68	{
73	QString outText = ts.readAll();	73		QString outText = ts.readAll();
74		74
75	QCOMPARE(outText, inText);	75		QCOMPARE(outText, inText);
76	}	76		}
77		77
78	filesToRemove << file.fileName();	78		filesToRemove << file.fileName();
79	}	79		}
80		80
		81		void KAutoSaveFileTest::test_fileNameMaxLength()
		82		{
		83		// In KAutoSaveFilePrivate::tempFile() the name of the kautosavefile that's going to be created
		84		// is concatinated in the form:
			dfaureUnsubmitted Not Done typo: concatenated dfaure: typo: concatenated
		85		// fileName + junk.truncated + protocol + _ + path.truncated + junk
		86		// see tempFile() for details.
		87		//
		88		// Make sure that the generated filename (e.g. as you would get from QUrl::fileName()) doesn't
		89		// exceed NAME_MAX (the maximum length allowed for filenames, see e.g. /usr/include/linux/limits.h)
		90		// otherwise the file can't be opened.
		91		//
		92		// see https://phabricator.kde.org/D24489
		93
		94		QString s;
		95		s = s.fill(QLatin1Char('b'), 80);
			dfaureUnsubmitted Not Done `fill` modifies the string, no need for `s =` dfaure: `fill` modifies the string, no need for `s = `
		96		// create a long path that:
		97		// - exceeds NAME_MAX (255)
		98		// - is less than the maximum allowed path length, PATH_MAX (4096)
			dfaureUnsubmitted Not Done Let's hope none of the supported OSes have a smaller PATH_MAX :) FreeBSD, Windows, macOS, who knows what limits they have. dfaure: Let's hope none of the supported OSes have a smaller PATH_MAX :) FreeBSD, Windows, macOS, who…
			dhaumannUnsubmitted Not Done What I wonder is: PATH_MAX etc are #defines, right? Since on Windows there is a registry flag to enable much longer paths/file names. My intention is not to block this series of patches, but we might end up with restrictions in our code that do not exist on the filesystem at hand. dhaumann: What I wonder is: PATH_MAX etc are #defines, right? Since on Windows there is a registry flag…
			ahmadsamirAuthorUnsubmitted Not Done On windows the extended-length path is 32,767[1] characters, which is a bit unreasonable. I only skimmed those docs though. [1] https://docs.microsoft.com/en-us/windows/win32/fileio/naming-a-file FTR, according to lxr.kde.org KAutoSaveFile is used by kdenlive, lokalize and libkeduvocdocument. ahmadsamir: On windows the extended-length path is 32,767[1] characters, which is a bit unreasonable. I…
		99		// see e.g. /usr/include/linux/limits.h
		100		const QString path = QDir::tempPath() + QLatin1Char('/')
		101		+ s + QLatin1Char('/')
		102		+ s + QLatin1Char('/')
		103		+ s + QLatin1Char('/')
		104		+ s;
		105
		106
		107		QFile file;
		108		file.setFileName(path + QLatin1Char('/') + QLatin1String("testFile.txt"));
			dfaureUnsubmitted Not Done merge with previous line using the QFile(QString) constructor dfaure: merge with previous line using the QFile(QString) constructor
		109
		110		qInfo() << "The lenght of the whole path to the file:" << file.fileName().size();
			dfaureUnsubmitted Not Done Doesn't belong in unittests dfaure: Doesn't belong in unittests
		111
		112		QUrl normalFile = QUrl::fromLocalFile(QFileInfo(file).absoluteFilePath());
			dfaureUnsubmitted Not Done Why the QFileInfo::absoluteFilePath call? It's already an absolute path. dfaure: Why the QFileInfo::absoluteFilePath call? It's already an absolute path.
		113
		114		KAutoSaveFile saveFile(normalFile);
		115
		116		QVERIFY(!QFile::exists(saveFile.fileName()));
		117		QVERIFY(saveFile.open(QIODevice::ReadWrite));
		118
		119		filesToRemove << file.fileName();
		120		}
		121
81	void KAutoSaveFileTest::test_fileStaleFiles()	122		void KAutoSaveFileTest::test_fileStaleFiles()
82	{	123		{
83	QUrl normalFile = QUrl::fromLocalFile(QDir::temp().absoluteFilePath(QStringLiteral("test directory/tîst me.txt")));	124		QUrl normalFile = QUrl::fromLocalFile(QDir::temp().absoluteFilePath(QStringLiteral("test directory/tîst me.txt")));
84		125
85	KAutoSaveFile saveFile(normalFile);	126		KAutoSaveFile saveFile(normalFile);
86	QVERIFY(saveFile.open(QIODevice::ReadWrite));	127		QVERIFY(saveFile.open(QIODevice::ReadWrite));
87	saveFile.write("testdata");	128		saveFile.write("testdata");
88	// Make sure the stale file is found	129		// Make sure the stale file is found
▲ Show 20 Lines • Show All 41 Lines • Show Last 20 Lines