From 37f6e7bac5cfc1e015d749b6d83cc7eda39d7b5c Mon Sep 17 00:00:00 2001 From: Caleb Date: Thu, 17 Aug 2017 13:25:54 -0600 Subject: [PATCH] Fix deletion for the `test-repo` backend. Deletion was added in #485, but the function for the `test-repo` backend was `deleteEntry` instead of `deleteFile` like it was supposed to be. Also, setting the key for a deleted file to `undefined` did not really remove that file from the object, so there were then errors stating `file.content` is not defined. `delete`ing the "file" from the object fixes this bug. --- src/backends/test-repo/implementation.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backends/test-repo/implementation.js b/src/backends/test-repo/implementation.js index 61f0bebf..18180aa3 100644 --- a/src/backends/test-repo/implementation.js +++ b/src/backends/test-repo/implementation.js @@ -93,10 +93,10 @@ export default class TestRepo { return Promise.resolve(); } - deleteEntry(path, commitMessage) { + deleteFile(path, commitMessage) { const folder = path.substring(0, path.lastIndexOf('/')); const fileName = path.substring(path.lastIndexOf('/') + 1); - window.repoFiles[folder][fileName] = undefined; + delete window.repoFiles[folder][fileName]; return Promise.resolve(); } }