ellipse() is now an alias for oval().

Ellipse is actually a more correct term for the thing we're drawing.

Author: Frederik De Bleser

nodebox/graphics/__init__.py

@@ -172,6 +172,8 @@ def oval(self, x, y, width, height, draw=True, **kwargs):
           path.draw()
         return path
 
+    ellipse = oval
+
     def line(self, x1, y1, x2, y2, draw=True, **kwargs):
         BezierPath.checkKwargs(kwargs)
         p = self.BezierPath(**kwargs)

nodebox/graphics/cocoa.py

@@ -281,6 +281,8 @@ def rect(self, x, y, width, height):
     def oval(self, x, y, width, height):
         self._segment_cache = None
         self._nsBezierPath.appendBezierPathWithOvalInRect_( ((x, y), (width, height)) )
+    
+    ellipse = oval
 
     def line(self, x1, y1, x2, y2):
         self._segment_cache = None

tests/graphics.py

@@ -0,0 +1,24 @@
+import unittest
+import sys
+
+# To run the test, make sure you have at least built the NodeBox Extensions.
+# Run the following command in the Terminal:
+#   xcodebuild -target "Build Extensions"
+sys.path.append('..')
+sys.path.append('../build/libs')
+
+from nodebox.graphics import *
+
+class GraphicsTestCase(unittest.TestCase):
+    
+    def test_ellipse(self):
+        """Test if ellipse is an alias for oval."""
+        ctx = Context()
+        self.assertTrue(hasattr(ctx, "ellipse"))
+        self.assertTrue(ctx.ellipse == ctx.oval)
+        p = BezierPath(ctx)
+        self.assertTrue(hasattr(p, "ellipse"))
+        self.assertTrue(p.ellipse == p.oval)
+
+if __name__=='__main__':
+    unittest.main()