From a07cd87c2c540f4dc5f3ca3fb47a5228476ea6a4 Mon Sep 17 00:00:00 2001 From: Nathan Sobo Date: Tue, 26 Mar 2013 15:46:19 -0600 Subject: [PATCH] Move testing of deferred deserializers to atom-spec --- spec/app/atom-package-spec.coffee | 20 -------------------- spec/app/atom-spec.coffee | 10 ++++++++++ 2 files changed, 10 insertions(+), 20 deletions(-) delete mode 100644 spec/app/atom-package-spec.coffee diff --git a/spec/app/atom-package-spec.coffee b/spec/app/atom-package-spec.coffee deleted file mode 100644 index 2f6c25061..000000000 --- a/spec/app/atom-package-spec.coffee +++ /dev/null @@ -1,20 +0,0 @@ -RootView = require 'root-view' -AtomPackage = require 'atom-package' -fs = require 'fs-utils' - -describe "AtomPackage", -> - [packageMainModule, pack] = [] - - beforeEach -> - pack = new AtomPackage(fs.resolve(config.packageDirPaths..., 'package-with-activation-events')) - pack.load() - - describe ".load()", -> - describe "if the package's metadata has a `deferredDeserializers` array", -> - it "requires the package's main module attempting to use deserializers named in the array", -> - expect(pack.mainModule).toBeNull() - object = deserialize(deserializer: 'Foo', data: "Hello") - expect(object.constructor.name).toBe 'Foo' - expect(object.data).toBe 'Hello' - expect(pack.mainModule).toBeDefined() - expect(pack.mainModule.activateCallCount).toBe 0 diff --git a/spec/app/atom-spec.coffee b/spec/app/atom-spec.coffee index 6680d2631..06a0a6dca 100644 --- a/spec/app/atom-spec.coffee +++ b/spec/app/atom-spec.coffee @@ -8,6 +8,16 @@ describe "the `atom` global", -> window.rootView = new RootView describe "package lifecycle methods", -> + describe ".loadPackage(id)", -> + describe "when the package has deferred deserializers", -> + it "requires the package's main module if one of its deferred deserializers is referenced", -> + pack = atom.loadPackage('package-with-activation-events') + expect(pack.mainModule).toBeNull() + object = deserialize({deserializer: 'Foo', data: 5}) + expect(pack.mainModule).toBeDefined() + expect(object.constructor.name).toBe 'Foo' + expect(object.data).toBe 5 + describe ".activatePackage(id)", -> describe "atom packages", -> describe "when the package has a main module", ->