added test cases to increase code coverage

diff --git a/spec/spec/bare.js b/spec/spec/bare.js
index 185a9652166580a2e7d0a3781c8b6a9442b66df5..9dbb8c2efe796a6895ad250f172953babe27d431 100644 (file)
--- a/spec/spec/bare.js
+++ b/spec/spec/bare.js
@@ -13,6 +13,10 @@ describe('Bare', function() {
     it('creates element in called parent', function() {
       expect(element.parent()).toBe(draw)
     })
+    it('inherits from given parent', function() {
+      expect(draw.element('g', SVG.Container).rect).toBeTruthy()
+      expect(draw.element('g', SVG.Container).group).toBeTruthy()
+    })
   })
 
   describe('words()', function() {
@@ -20,5 +24,10 @@ describe('Bare', function() {
       var element = draw.element('title').words('These are some words.').id(null)
       expect(element.svg()).toBe('<title>These are some words.</title>')
     })
+    it('removes all nodes before adding words', function() {
+      var element = draw.element('title').words('These are some words.').id(null)
+      element.words('These are some words.')
+      expect(element.svg()).toBe('<title>These are some words.</title>')
+    })
   })
-})
\ No newline at end of file
+})

diff --git a/spec/spec/gradient.js b/spec/spec/gradient.js
index 25eeff33136db85f52f5f37ecd4db762653550f0..e0b061a6a67d58e6df8b9ab17fa84dae0635cfc8 100644 (file)
--- a/spec/spec/gradient.js
+++ b/spec/spec/gradient.js
@@ -13,7 +13,7 @@ describe('Gradient', function() {
     rect.remove()
     gradient.remove()
   })
-  
+
   it('is an instance of SVG.Gradient', function() {
     expect(gradient instanceof SVG.Gradient).toBe(true)
   })
@@ -22,13 +22,19 @@ describe('Gradient', function() {
     gradient = draw.gradient('linear')
     expect(gradient.children().length).toBe(0)
   })
-  
+
   describe('fill()', function() {
     it('returns the id of the gradient wrapped in url()', function() {
       expect(gradient.fill()).toBe('url(#' + gradient.attr('id') + ')')
     })
   })
 
+  describe('attr()', function() {
+    it('will catch transform attribues and convert them to gradientTransform', function() {
+      expect(gradient.translate(100,100).attr('gradientTransform')).toBe('matrix(1,0,0,1,100,100)')
+    })
+  })
+
   describe('toString()', function() {
     it('returns the id of the gradient wrapped in url()', function() {
       expect(gradient + '').toBe('url(#' + gradient.attr('id') + ')')
@@ -95,7 +101,7 @@ describe('Gradient', function() {
       expect(s2.attr('stop-color')).toBe('#ffffff')
       expect(s2.attr('stop-opacity')).toBe(0.5)
     })
-    
+
   })
 
   describe('get()', function() {
@@ -111,5 +117,5 @@ describe('Gradient', function() {
     })
 
   })
-  
-})
\ No newline at end of file
+
+})

diff --git a/spec/spec/pattern.js b/spec/spec/pattern.js
index 4abdadcbd64c24957faa5a0d1b1d822324a6cfd5..e12d11e1ff523310e00857086b8aeb17e3f911c7 100644 (file)
--- a/spec/spec/pattern.js
+++ b/spec/spec/pattern.js
@@ -13,7 +13,7 @@ describe('Pattern', function() {
     rect.remove()
     pattern.remove()
   })
-  
+
   it('is an instance of SVG.Pattern', function() {
     expect(pattern instanceof SVG.Pattern).toBe(true)
   })
@@ -22,13 +22,19 @@ describe('Pattern', function() {
     pattern = draw.pattern(10,30)
     expect(pattern.children().length).toBe(0)
   })
-  
+
   describe('fill()', function() {
     it('returns the id of the pattern wrapped in url()', function() {
       expect(pattern.fill()).toBe('url(#' + pattern.attr('id') + ')')
     })
   })
 
+  describe('attr()', function() {
+    it('will catch transform attribues and convert them to patternTransform', function() {
+      expect(pattern.translate(100,100).attr('patternTransform')).toBe('matrix(1,0,0,1,100,100)')
+    })
+  })
+
   describe('toString()', function() {
     it('returns the id of the pattern wrapped in url()', function() {
       expect(pattern + '').toBe('url(#' + pattern.attr('id') + ')')
@@ -59,5 +65,5 @@ describe('Pattern', function() {
     })
 
   })
-  
-})
\ No newline at end of file
+
+})