From owner-svn-src-projects@freebsd.org  Sat Mar  2 15:32:22 2019
Return-Path: <owner-svn-src-projects@freebsd.org>
Delivered-To: svn-src-projects@mailman.ysv.freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1])
 by mailman.ysv.freebsd.org (Postfix) with ESMTP id 0E1E215015DF
 for <svn-src-projects@mailman.ysv.freebsd.org>;
 Sat,  2 Mar 2019 15:32:22 +0000 (UTC)
 (envelope-from asomers@FreeBSD.org)
Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org
 [IPv6:2610:1c1:1:606c::19:3])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
 server-signature RSA-PSS (4096 bits)
 client-signature RSA-PSS (4096 bits) client-digest SHA256)
 (Client CN "mxrelay.nyi.freebsd.org",
 Issuer "Let's Encrypt Authority X3" (verified OK))
 by mx1.freebsd.org (Postfix) with ESMTPS id B41E06BE9C;
 Sat,  2 Mar 2019 15:32:21 +0000 (UTC)
 (envelope-from asomers@FreeBSD.org)
Received: from repo.freebsd.org (repo.freebsd.org
 [IPv6:2610:1c1:1:6068::e6a:0])
 (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits))
 (Client did not present a certificate)
 by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id A48FE6BDB;
 Sat,  2 Mar 2019 15:32:21 +0000 (UTC)
 (envelope-from asomers@FreeBSD.org)
Received: from repo.freebsd.org ([127.0.1.37])
 by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id x22FWLMu045936;
 Sat, 2 Mar 2019 15:32:21 GMT (envelope-from asomers@FreeBSD.org)
Received: (from asomers@localhost)
 by repo.freebsd.org (8.15.2/8.15.2/Submit) id x22FWLok045933;
 Sat, 2 Mar 2019 15:32:21 GMT (envelope-from asomers@FreeBSD.org)
Message-Id: <201903021532.x22FWLok045933@repo.freebsd.org>
X-Authentication-Warning: repo.freebsd.org: asomers set sender to
 asomers@FreeBSD.org using -f
From: Alan Somers <asomers@FreeBSD.org>
Date: Sat, 2 Mar 2019 15:32:21 +0000 (UTC)
To: src-committers@freebsd.org, svn-src-projects@freebsd.org
Subject: svn commit: r344725 - projects/fuse2/tests/sys/fs/fuse
X-SVN-Group: projects
X-SVN-Commit-Author: asomers
X-SVN-Commit-Paths: projects/fuse2/tests/sys/fs/fuse
X-SVN-Commit-Revision: 344725
X-SVN-Commit-Repository: base
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
X-Rspamd-Queue-Id: B41E06BE9C
X-Spamd-Bar: --
Authentication-Results: mx1.freebsd.org
X-Spamd-Result: default: False [-2.98 / 15.00];
 local_wl_from(0.00)[FreeBSD.org];
 NEURAL_HAM_MEDIUM(-1.00)[-0.999,0];
 NEURAL_HAM_SHORT(-0.98)[-0.979,0];
 ASN(0.00)[asn:11403, ipnet:2610:1c1:1::/48, country:US];
 NEURAL_HAM_LONG(-1.00)[-1.000,0]
X-BeenThere: svn-src-projects@freebsd.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "SVN commit messages for the src &quot; projects&quot;
 tree" <svn-src-projects.freebsd.org>
List-Unsubscribe: <https://lists.freebsd.org/mailman/options/svn-src-projects>, 
 <mailto:svn-src-projects-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-projects/>
List-Post: <mailto:svn-src-projects@freebsd.org>
List-Help: <mailto:svn-src-projects-request@freebsd.org?subject=help>
List-Subscribe: <https://lists.freebsd.org/mailman/listinfo/svn-src-projects>, 
 <mailto:svn-src-projects-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sat, 02 Mar 2019 15:32:22 -0000

Author: asomers
Date: Sat Mar  2 15:32:20 2019
New Revision: 344725
URL: https://svnweb.freebsd.org/changeset/base/344725

Log:
  Add some fuse(4) tests for FUSE_SETATTR
  
  Sponsored by:	The FreeBSD Foundation

Added:
  projects/fuse2/tests/sys/fs/fuse/setattr.cc   (contents, props changed)
Modified:
  projects/fuse2/tests/sys/fs/fuse/Makefile
  projects/fuse2/tests/sys/fs/fuse/mockfs.hh

Modified: projects/fuse2/tests/sys/fs/fuse/Makefile
==============================================================================
--- projects/fuse2/tests/sys/fs/fuse/Makefile	Sat Mar  2 14:30:27 2019	(r344724)
+++ projects/fuse2/tests/sys/fs/fuse/Makefile	Sat Mar  2 15:32:20 2019	(r344725)
@@ -6,6 +6,7 @@ TESTSDIR=	${TESTSBASE}/sys/fs/fuse
 
 ATF_TESTS_CXX+=	getattr
 ATF_TESTS_CXX+=	lookup
+ATF_TESTS_CXX+=	setattr
 
 SRCS.getattr+=	getattr.cc
 SRCS.getattr+=	getmntopts.c
@@ -17,6 +18,12 @@ SRCS.lookup+=	getmntopts.c
 SRCS.lookup+=	mockfs.cc
 SRCS.lookup+=	utils.cc
 
+SRCS.setattr+=	setattr.cc
+SRCS.setattr+=	getmntopts.c
+SRCS.setattr+=	mockfs.cc
+SRCS.setattr+=	utils.cc
+
+# TODO: drastically increase timeout after test development is mostly complete
 TEST_METADATA+= timeout=10
 TEST_METADATA+= required_user=root
 

Modified: projects/fuse2/tests/sys/fs/fuse/mockfs.hh
==============================================================================
--- projects/fuse2/tests/sys/fs/fuse/mockfs.hh	Sat Mar  2 14:30:27 2019	(r344724)
+++ projects/fuse2/tests/sys/fs/fuse/mockfs.hh	Sat Mar  2 15:32:20 2019	(r344725)
@@ -45,11 +45,12 @@ extern "C" {
 extern int verbosity;
 
 union fuse_payloads_in {
+	/* value is from fuse_kern_chan.c in fusefs-libs */
+	uint8_t		bytes[0x21000 - sizeof(struct fuse_in_header)];
 	fuse_forget_in	forget;
 	fuse_init_in	init;
 	char		lookup[0];
-	/* value is from fuse_kern_chan.c in fusefs-libs */
-	uint8_t		bytes[0x21000 - sizeof(struct fuse_in_header)];
+	fuse_setattr_in	setattr;
 };
 
 struct mockfs_buf_in {

Added: projects/fuse2/tests/sys/fs/fuse/setattr.cc
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ projects/fuse2/tests/sys/fs/fuse/setattr.cc	Sat Mar  2 15:32:20 2019	(r344725)
@@ -0,0 +1,386 @@
+/*-
+ * Copyright (c) 2019 The FreeBSD Foundation
+ * All rights reserved.
+ *
+ * This software was developed by BFF Storage Systems, LLC under sponsorship
+ * from the FreeBSD Foundation.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED. IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ */
+
+extern "C" {
+#include <sys/stat.h>
+
+#include <fcntl.h>
+}
+
+#include "mockfs.hh"
+#include "utils.hh"
+
+using namespace testing;
+
+class Setattr : public FuseTest {};
+
+
+/* Change the mode of a file */
+TEST_F(Setattr, chmod)
+{
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+	const mode_t oldmode = 0755;
+	const mode_t newmode = 0644;
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | oldmode;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr.mode = S_IFREG | oldmode;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			/* In protocol 7.23, ctime will be changed too */
+			uint32_t valid = FATTR_MODE;
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino &&
+				in->body.setattr.valid == valid &&
+				in->body.setattr.mode == newmode);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | newmode;
+	}));
+	EXPECT_EQ(0, chmod(FULLPATH, newmode)) << strerror(errno);
+}
+
+/* Change the owner and group of a file */
+TEST_F(Setattr, chown)
+{
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+	const gid_t oldgroup = 66;
+	const gid_t newgroup = 99;
+	const uid_t olduser = 33;
+	const uid_t newuser = 44;
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | 0644;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr.gid = oldgroup;
+		out->body.entry.attr.uid = olduser;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			/* In protocol 7.23, ctime will be changed too */
+			uint32_t valid = FATTR_GID | FATTR_UID;
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino &&
+				in->body.setattr.valid == valid &&
+				in->body.setattr.uid == newuser &&
+				in->body.setattr.gid == newgroup);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.uid = newuser;
+		out->body.attr.attr.gid = newgroup;
+	}));
+	EXPECT_EQ(0, chown(FULLPATH, newuser, newgroup)) << strerror(errno);
+}
+
+
+/* Change the mode of an open file, by its file descriptor */
+//TODO TEST_F(Setattr, fchmod) {}
+
+/* 
+ * FUSE daemons are allowed to check permissions however they like.  If the
+ * daemon returns EPERM, even if the file permissions "should" grant access,
+ * then fuse(4) should return EPERM too.
+ */
+TEST_F(Setattr, eperm)
+{
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | 0777;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr.uid = in->header.uid;
+		out->body.entry.attr.gid = in->header.gid;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		out->header.error = -EPERM;
+		out->header.len = sizeof(out->header);
+	}));
+	EXPECT_NE(0, truncate(FULLPATH, 10));
+	EXPECT_EQ(EPERM, errno);
+}
+
+/* Change the size of the file */
+TEST_F(Setattr, truncate) {
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+	const uint64_t oldsize = 100'000'000;
+	const uint64_t newsize = 20'000'000;
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | 0644;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr.size = oldsize;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			/* In protocol 7.23, ctime will be changed too */
+			uint32_t valid = FATTR_SIZE;
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino &&
+				in->body.setattr.valid == valid &&
+				in->body.setattr.size == newsize);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.size = newsize;
+	}));
+	EXPECT_EQ(0, truncate(FULLPATH, newsize)) << strerror(errno);
+}
+
+/* Change a file's timestamps */
+TEST_F(Setattr, utimensat) {
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+	const timespec oldtimes[2] = {
+		{.tv_sec = 1, .tv_nsec = 2},
+		{.tv_sec = 3, .tv_nsec = 4},
+	};
+	const timespec newtimes[2] = {
+		{.tv_sec = 5, .tv_nsec = 6},
+		{.tv_sec = 7, .tv_nsec = 8},
+	};
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | 0644;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr_valid = UINT64_MAX;
+		out->body.entry.attr.atime = oldtimes[0].tv_sec;
+		out->body.entry.attr.atimensec = oldtimes[0].tv_nsec;
+		out->body.entry.attr.mtime = oldtimes[1].tv_sec;
+		out->body.entry.attr.mtimensec = oldtimes[1].tv_nsec;
+	}));
+
+	/* 
+	 * Until bug 235775 is fixed, utimensat will make an extra FUSE_GETATTR
+	 * call
+	 */ 
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			return (in->header.opcode == FUSE_GETATTR &&
+				in->header.nodeid == ino);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.atime = oldtimes[0].tv_sec;
+		out->body.attr.attr.atimensec = oldtimes[0].tv_nsec;
+		out->body.attr.attr.mtime = oldtimes[1].tv_sec;
+		out->body.attr.attr.mtimensec = oldtimes[1].tv_nsec;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			/* In protocol 7.23, ctime will be changed too */
+			uint32_t valid = FATTR_ATIME | FATTR_MTIME;
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino &&
+				in->body.setattr.valid == valid &&
+				in->body.setattr.atime == newtimes[0].tv_sec &&
+				in->body.setattr.atimensec ==
+					newtimes[0].tv_nsec &&
+				in->body.setattr.mtime == newtimes[1].tv_sec &&
+				in->body.setattr.mtimensec ==
+					newtimes[1].tv_nsec);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.atime = newtimes[0].tv_sec;
+		out->body.attr.attr.atimensec = newtimes[0].tv_nsec;
+		out->body.attr.attr.mtime = newtimes[1].tv_sec;
+		out->body.attr.attr.mtimensec = newtimes[1].tv_nsec;
+	}));
+	EXPECT_EQ(0, utimensat(AT_FDCWD, FULLPATH, &newtimes[0], 0))
+		<< strerror(errno);
+}
+
+/* Change a file mtime but not its atime */
+TEST_F(Setattr, utimensat_mtime_only) {
+	const char FULLPATH[] = "mountpoint/some_file.txt";
+	const char RELPATH[] = "some_file.txt";
+	const uint64_t ino = 42;
+	const timespec oldtimes[2] = {
+		{.tv_sec = 1, .tv_nsec = 2},
+		{.tv_sec = 3, .tv_nsec = 4},
+	};
+	const timespec newtimes[2] = {
+		{.tv_sec = 5, .tv_nsec = UTIME_OMIT},
+		{.tv_sec = 7, .tv_nsec = 8},
+	};
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			return (in->header.opcode == FUSE_LOOKUP &&
+				strcmp(in->body.lookup, RELPATH) == 0);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, entry);
+		out->body.entry.attr.mode = S_IFREG | 0644;
+		out->body.entry.nodeid = ino;
+		out->body.entry.attr_valid = UINT64_MAX;
+		out->body.entry.attr.atime = oldtimes[0].tv_sec;
+		out->body.entry.attr.atimensec = oldtimes[0].tv_nsec;
+		out->body.entry.attr.mtime = oldtimes[1].tv_sec;
+		out->body.entry.attr.mtimensec = oldtimes[1].tv_nsec;
+	}));
+
+	/* 
+	 * Until bug 235775 is fixed, utimensat will make an extra FUSE_GETATTR
+	 * call
+	 */ 
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([](auto in) {
+			return (in->header.opcode == FUSE_GETATTR &&
+				in->header.nodeid == ino);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.atime = oldtimes[0].tv_sec;
+		out->body.attr.attr.atimensec = oldtimes[0].tv_nsec;
+		out->body.attr.attr.mtime = oldtimes[1].tv_sec;
+		out->body.attr.attr.mtimensec = oldtimes[1].tv_nsec;
+	}));
+
+	EXPECT_CALL(*m_mock, process(
+		ResultOf([=](auto in) {
+			/* In protocol 7.23, ctime will be changed too */
+			uint32_t valid = FATTR_MTIME;
+			return (in->header.opcode == FUSE_SETATTR &&
+				in->header.nodeid == ino &&
+				in->body.setattr.valid == valid &&
+				in->body.setattr.mtime == newtimes[1].tv_sec &&
+				in->body.setattr.mtimensec ==
+					newtimes[1].tv_nsec);
+		}, Eq(true)),
+		_)
+	).WillOnce(Invoke([=](auto in, auto out) {
+		out->header.unique = in->header.unique;
+		SET_OUT_HEADER_LEN(out, attr);
+		out->body.attr.attr.ino = ino;	// Must match nodeid
+		out->body.attr.attr.mode = S_IFREG | 0644;
+		out->body.attr.attr.atime = oldtimes[0].tv_sec;
+		out->body.attr.attr.atimensec = oldtimes[0].tv_nsec;
+		out->body.attr.attr.mtime = newtimes[1].tv_sec;
+		out->body.attr.attr.mtimensec = newtimes[1].tv_nsec;
+	}));
+	EXPECT_EQ(0, utimensat(AT_FDCWD, FULLPATH, &newtimes[0], 0))
+		<< strerror(errno);
+}
+
+/* 
+ * Writethrough cache: newly changed attributes should be automatically cached,
+ * if the filesystem allows it
+ */
+//TODO TEST_F(Setattr, writethrough_cache){}