From 6c554f2e76bc861ca20feb83d9b9864d9d1b1902 Mon Sep 17 00:00:00 2001
From: "Scott E. Graves" <scott.e.graves@protonmail.com>
Date: Wed, 3 Mar 2021 15:22:22 -0600
Subject: [PATCH] file tests

---
 .gitignore                 |  1 +
 package.json               |  3 ++-
 src/__tests__/file.test.js | 49 ++++++++++++++++++++++++++++++++++++++
 src/io/file.js             |  2 +-
 4 files changed, 53 insertions(+), 2 deletions(-)
 create mode 100644 src/__tests__/file.test.js

diff --git a/.gitignore b/.gitignore
index fdd41e0..163f79a 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
 package-lock.json
 .idea/
 node_modules/
+coverage/
diff --git a/package.json b/package.json
index 89439b7..3597ddf 100644
--- a/package.json
+++ b/package.json
@@ -3,7 +3,8 @@
   "version": "1.3.1-r1",
   "description": "A Node.js module for interfacing with Repertory's remote mount API",
   "scripts": {
-    "test": "jest"
+    "test": "jest",
+    "coverage": "jest --collect-coverage"
   },
   "keywords": [
     "repertory",
diff --git a/src/__tests__/file.test.js b/src/__tests__/file.test.js
new file mode 100644
index 0000000..6e1ab4f
--- /dev/null
+++ b/src/__tests__/file.test.js
@@ -0,0 +1,49 @@
+import file from '../io/file';
+
+jest.mock('../ops/index.js', () => (
+  {
+    ...(jest.requireActual('../ops/index.js')),
+    close_file: jest.fn(),
+  }
+));
+
+import {close_file} from '../ops/index';
+
+test(`can close a closed file`, async () => {
+  const f = new file();
+  expect(await f.close()).toEqual(0);
+});
+
+test(`'get_size' fails on closed file`, async () => {
+  const f = new file();
+  await expect(f.get_size()).rejects.toThrow(Error);
+});
+
+test(`'read' fails on closed file`, async () => {
+  const f = new file();
+  await expect(f.read(0, 10)).rejects.toThrow(Error);
+});
+
+test(`'truncate' fails on closed file`, async () => {
+  const f = new file();
+  await expect(f.truncate(0)).rejects.toThrow(Error);
+});
+
+test(`'write' fails on closed file`, async () => {
+  const f = new file();
+  await expect(f.write(0, Buffer.alloc(2))).rejects.toThrow(Error);
+});
+
+test(`handle is set to null on close`, async () => {
+  const f = new file(null, 1, '/path');
+  close_file.mockReturnValue(0);
+  expect(await f.close()).toEqual(0);
+  expect(f.handle).toBeNull();
+});
+
+test(`handle is not changed on close if return is not 0`, async () => {
+  const f = new file(null, 1, '/path');
+  close_file.mockReturnValue(1);
+  expect(await f.close()).toEqual(1);
+  expect(f.handle).toBe(1);
+});
diff --git a/src/io/file.js b/src/io/file.js
index 11773a2..091cbbc 100644
--- a/src/io/file.js
+++ b/src/io/file.js
@@ -5,7 +5,7 @@ let next_thread_id = 1;
 export default class file {
   constructor(conn, handle, remote_path) {
     this.conn = conn;
-    this.handle = handle;
+    this.handle = handle || null;
     this.remote_path = remote_path;
     this.thread_id = next_thread_id++;
   }