Throw error on ScrollComponent.wrap() with nonexisting layer

koenbok · Jan 7, 2016 · 3ee6d07 · 3ee6d07
1 parent f797a36
commit 3ee6d07
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/framer/Components/ScrollComponent.coffee b/framer/Components/ScrollComponent.coffee
@@ -391,6 +391,9 @@ class exports.ScrollComponent extends Layer
 
 wrapComponent = (instance, layer, options = {correct:true}) ->
 
+	if not (layer instanceof Layer)
+		throw new Error("ScrollComponent.wrap expects a layer, not #{layer}. Are you sure the layer exists?")
+
 	# This function wraps the given layer into a scroll or page component. This is
 	# great for importing from Sketch or Photoshop.
 

diff --git a/test/tests/ScrollComponentTest.coffee b/test/tests/ScrollComponentTest.coffee
@@ -77,3 +77,7 @@ describe "ScrollComponent", ->
 
 			delete layerA._properties.backgroundColor
 			scroll = ScrollComponent.wrap(layerA)
+
+		it "should throw a warning on no layer", ->
+			f = -> ScrollComponent.wrap()
+			f.should.throw()