From b59bd4bc7b9bac516afadaf332cd23a92fcdcf68 Mon Sep 17 00:00:00 2001 From: Corey Johnson & Nathan Sobo Date: Mon, 2 Jul 2012 17:48:30 -0700 Subject: [PATCH] Saving a buffer does not trigger a "contents-change" event --- spec/app/buffer-spec.coffee | 12 +++++++++++- src/app/buffer.coffee | 1 + 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/spec/app/buffer-spec.coffee b/spec/app/buffer-spec.coffee index 2cc5ed279..a12885af5 100644 --- a/spec/app/buffer-spec.coffee +++ b/spec/app/buffer-spec.coffee @@ -57,13 +57,23 @@ describe 'Buffer', -> path = "/tmp/tmp.txt" fs.write(path, "first") buffer.destroy() + buffer = new Buffer(path) afterEach -> fs.remove(path) + it "does not trigger a contents-change event when Atom modifies the file", -> + buffer.insert([0,0], "HELLO!") + changeHandler = jasmine.createSpy("buffer changed") + buffer.on "change", changeHandler + buffer.save() + + waits 30 + runs -> + expect(changeHandler).not.toHaveBeenCalled() + describe "when the buffer is unmodified", -> it "triggers 'change' event and buffer remains unmodified", -> - buffer = new Buffer(path) changeHandler = jasmine.createSpy('changeHandler') buffer.on 'change', changeHandler fs.write(path, "second") diff --git a/src/app/buffer.coffee b/src/app/buffer.coffee index 0c9d6bd11..0c720121e 100644 --- a/src/app/buffer.coffee +++ b/src/app/buffer.coffee @@ -172,6 +172,7 @@ class Buffer @trigger 'before-save' fs.write path, @getText() + @file?.updateMd5() @modified = false @setPath(path) @trigger 'after-save'