[libcamera-devel] [PATCH 04/11] test: Add File class tests

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Apr 8 01:28:57 CEST 2020


Hi Niklas,

On Tue, Apr 07, 2020 at 10:24:04PM +0200, Niklas Söderlund wrote:
> On 2020-04-04 04:56:17 +0300, Laurent Pinchart wrote:
> > Add tests for the File class API.
> > 
> > Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> > ---
> >  test/file.cpp    | 285 +++++++++++++++++++++++++++++++++++++++++++++++
> >  test/meson.build |   1 +
> >  2 files changed, 286 insertions(+)
> >  create mode 100644 test/file.cpp
> > 
> > diff --git a/test/file.cpp b/test/file.cpp
> > new file mode 100644
> > index 000000000000..c046bced1c13
> > --- /dev/null
> > +++ b/test/file.cpp
> > @@ -0,0 +1,285 @@
> > +/* SPDX-License-Identifier: GPL-2.0-or-later */
> > +/*
> > + * Copyright (C) 2020, Google Inc.
> > + *
> > + * file.cpp - File I/O operations tests
> > + */
> > +
> > +#include <fstream>
> > +#include <iostream>
> > +#include <string>
> > +#include <string.h>
> > +#include <sys/types.h>
> > +#include <unistd.h>
> > +
> > +#include "file.h"
> > +#include "test.h"
> > +
> > +using namespace std;
> > +using namespace libcamera;
> > +
> > +class FileTest : public Test
> > +{
> > +protected:
> > +	int init()
> > +	{
> > +		fileName_ = "/tmp/libcamera.test." + std::to_string(getpid());
> 
> I think this should use mktemp(), not everyone have the same /tmp ;-P

I'll go for mkstemp() as mktemp() is marked as "never use this function"
:-) This however doesn't handle the temporary directory issue, as both
mktemp() and mkstemp() both take an absolute path as their template
argument.

> As the File interface uses an API where the error is signaled using 
> File::error() shall it not be tested here as well? But we can always add 
> that on top.

I'll add an error() test.

> With mktemp() sorted,
> 
> Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>
> 
> > +
> > +		std::ofstream ostrm(fileName_, ios::binary);
> > +		ostrm << "libcamera";
> > +		ostrm.close();
> > +
> > +		return TestPass;
> > +	}
> > +
> > +	int run()
> > +	{
> > +		/* Test static functions. */
> > +		if (!File::exists("/dev/null")) {
> > +			cerr << "Valid file not found" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (File::exists("/dev/null/invalid")) {
> > +			cerr << "Invalid file should not exist" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test unnamed file. */
> > +		File file;
> > +
> > +		if (!file.fileName().empty()) {
> > +			cerr << "Unnamed file has non-empty file name" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.exists()) {
> > +			cerr << "Unnamed file exists" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.isOpen()) {
> > +			cerr << "File is open after construction" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.openMode() != File::NotOpen) {
> > +			cerr << "File has invalid open mode after construction"
> > +			     << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.size() >= 0) {
> > +			cerr << "Unnamed file has a size" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.open(File::ReadWrite)) {
> > +			cerr << "Opening unnamed file succeeded" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test named file referring to an invalid file. */
> > +		file.setFileName("/dev/null/invalid");
> > +
> > +		if (file.fileName() != "/dev/null/invalid") {
> > +			cerr << "File reports incorrect file name" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.exists()) {
> > +			cerr << "Invalid file exists" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.isOpen()) {
> > +			cerr << "Invalid file is open after construction" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.openMode() != File::NotOpen) {
> > +			cerr << "Invalid file has invalid open mode after construction"
> > +			     << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.size() >= 0) {
> > +			cerr << "Invalid file has a size" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.open(File::ReadWrite)) {
> > +			cerr << "Opening invalid file succeeded" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test named file referring to a valid file. */
> > +		file.setFileName("/dev/null");
> > +
> > +		if (!file.exists()) {
> > +			cerr << "Valid file does not exist" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.isOpen()) {
> > +			cerr << "Valid file is open after construction" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.openMode() != File::NotOpen) {
> > +			cerr << "Valid file has invalid open mode after construction"
> > +			     << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.size() >= 0) {
> > +			cerr << "Invalid file has a size" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test open and close. */
> > +		if (!file.open(File::ReadWrite)) {
> > +			cerr << "Opening file failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (!file.isOpen()) {
> > +			cerr << "Open file reported as closed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.openMode() != File::ReadWrite) {
> > +			cerr << "Open file has invalid open mode" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		file.close();
> > +
> > +		if (file.isOpen()) {
> > +			cerr << "Closed file reported as open" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (file.openMode() != File::NotOpen) {
> > +			cerr << "Closed file has invalid open mode" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test size(). */
> > +		file.setFileName("/proc/self/exe");
> > +
> > +		if (file.size() >= 0) {
> > +			cerr << "File has valid size before open" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		file.open(File::ReadOnly);
> > +
> > +		ssize_t size = file.size();
> > +		if (size <= 0) {
> > +			cerr << "File has invalid size after open" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test mapping and unmapping. */
> > +		Span<uint8_t> data = file.map();
> > +		if (data.empty()) {
> > +			cerr << "Mapping of complete file failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (data.size() != static_cast<size_t>(size)) {
> > +			cerr << "Mapping  of complete file has invalid size" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (!file.unmap(data.data())) {
> > +			cerr << "Unmapping of complete file failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		data = file.map(4096, 8192);
> > +		if (data.empty()) {
> > +			cerr << "Mapping of file region failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (data.size() != 8192) {
> > +			cerr << "Mapping of file region has invalid size" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		if (!file.unmap(data.data())) {
> > +			cerr << "Unmapping of file region failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		file.close();
> > +
> > +		/* Test private mapping. */
> > +		file.setFileName(fileName_);
> > +		file.open(File::ReadWrite);
> > +
> > +		data = file.map(0, -1, File::MapPrivate);
> > +		if (data.empty()) {
> > +			cerr << "Private mapping failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		std::string str{ reinterpret_cast<char *>(data.data()), data.size() };
> > +		if (str != "libcamera") {
> > +			cerr << "Invalid contents of private mapping" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		memcpy(data.data(), "LIBCAMERA", 9);
> > +
> > +		if (!file.unmap(data.data())) {
> > +			cerr << "Private unmapping failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		data = file.map();
> > +
> > +		str = { reinterpret_cast<char *>(data.data()), data.size() };
> > +		if (str != "libcamera") {
> > +			cerr << "Private mapping changed file contents" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		/* Test shared mapping. */
> > +		data = file.map();
> > +		if (data.empty()) {
> > +			cerr << "Shared mapping failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		memcpy(data.data(), "LIBCAMERA", 9);
> > +
> > +		if (!file.unmap(data.data())) {
> > +			cerr << "Shared unmapping failed" << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		data = file.map();
> > +
> > +		str = { reinterpret_cast<char *>(data.data()), data.size() };
> > +		if (str != "LIBCAMERA") {
> > +			cerr << "Shared mapping failed to change file contents"
> > +			     << endl;
> > +			return TestFail;
> > +		}
> > +
> > +		return TestPass;
> > +	}
> > +
> > +	void cleanup()
> > +	{
> > +		unlink(fileName_.c_str());
> > +	}
> > +
> > +private:
> > +	std::string fileName_;
> > +};
> > +
> > +TEST_REGISTER(FileTest)
> > diff --git a/test/meson.build b/test/meson.build
> > index 8ab58ac15a2a..5a45a85effd3 100644
> > --- a/test/meson.build
> > +++ b/test/meson.build
> > @@ -26,6 +26,7 @@ internal_tests = [
> >      ['event',                           'event.cpp'],
> >      ['event-dispatcher',                'event-dispatcher.cpp'],
> >      ['event-thread',                    'event-thread.cpp'],
> > +    ['file',                            'file.cpp'],
> >      ['file-descriptor',                 'file-descriptor.cpp'],
> >      ['message',                         'message.cpp'],
> >      ['object',                          'object.cpp'],

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list