From 064f161accef7f6c467ff1d6c447aae8388a7334 Mon Sep 17 00:00:00 2001
From: Dave Halter <davidhalter88@gmail.com>
Date: Fri, 14 Mar 2014 13:32:03 +0100
Subject: [PATCH] signature is now a subclass of definitions

---
 jedi/api/__init__.py                  |  2 +-
 jedi/api/classes.py                   | 24 ++++++++++++------------
 test/test_api/test_call_signatures.py |  6 ++++--
 3 files changed, 17 insertions(+), 15 deletions(-)

diff --git a/jedi/api/__init__.py b/jedi/api/__init__.py
index c94de681..77481305 100644
--- a/jedi/api/__init__.py
+++ b/jedi/api/__init__.py
@@ -137,7 +137,7 @@ class Script(object):
             # add named params
             for call_sig in self.call_signatures():
                 # allow protected access, because it's a public API.
-                module = call_sig._executable.get_parent_until()
+                module = call_sig._definition.get_parent_until()
                 # Compiled modules typically don't allow keyword arguments.
                 if not isinstance(module, compiled.CompiledObject):
                     for p in call_sig.params:
diff --git a/jedi/api/classes.py b/jedi/api/classes.py
index eab09b54..6641e70a 100644
--- a/jedi/api/classes.py
+++ b/jedi/api/classes.py
@@ -541,30 +541,30 @@ class Definition(BaseDefinition):
         return defined_names(self._evaluator, d)
 
 
-class CallSignature(object):
+class CallSignature(Definition):
     """
     `CallSignature` objects is the return value of `Script.function_definition`.
     It knows what functions you are currently in. e.g. `isinstance(` would
     return the `isinstance` function. without `(` it would return nothing.
     """
     def __init__(self, evaluator, executable, call, index):
-        self._evaluator = evaluator
-        self._executable = executable
+        super(CallSignature, self).__init__(evaluator, executable)
         self.index = index
+        """ The param index of the current call. """
         self._call = call
 
     @property
     def params(self):
-        if self._executable.isinstance(er.Function):
-            if isinstance(self._executable, er.InstanceElement):
-                params = self._executable.params[1:]
+        if self._definition.isinstance(er.Function):
+            if isinstance(self._definition, er.InstanceElement):
+                params = self._definition.params[1:]
             else:
-                params = self._executable.params
-        elif self._executable.isinstance(er.compiled.CompiledObject):
-            params = self._executable.params
+                params = self._definition.params
+        elif self._definition.isinstance(er.compiled.CompiledObject):
+            params = self._definition.params
         else:
             try:
-                sub = self._executable.get_subscope_by_name('__init__')
+                sub = self._definition.get_subscope_by_name('__init__')
                 params = sub.params[1:]  # ignore self
             except KeyError:
                 return []
@@ -584,10 +584,10 @@ class CallSignature(object):
     @property
     def call_name(self):
         """ The name (e.g. 'isinstance') as a string. """
-        return unicode(self._executable.name)
+        return unicode(self._definition.name)
 
     def __repr__(self):
-        return '<%s: %s index %s>' % (type(self).__name__, self._executable,
+        return '<%s: %s index %s>' % (type(self).__name__, self._definition,
                                       self.index)
 
 
diff --git a/test/test_api/test_call_signatures.py b/test/test_api/test_call_signatures.py
index ef6cd9a5..8e2e274f 100644
--- a/test/test_api/test_call_signatures.py
+++ b/test/test_api/test_call_signatures.py
@@ -180,7 +180,7 @@ class TestParams(TestCase):
         assert p[1].name == 'mode'
 
 
-def test_signature_is_definition(TestCase):
+def test_signature_is_definition():
     """
     Through inheritance, a call signature is a sub class of Definition.
     Check if the attributes match.
@@ -193,9 +193,11 @@ def test_signature_is_definition(TestCase):
 
     # Now compare all the attributes that a CallSignature must also have.
     for attr_name in dir(definition):
+        if attr_name.startswith('_') or attr_name == 'defined_names':
+            continue
         attribute = getattr(definition, attr_name)
         signature_attribute = getattr(signature, attr_name)
-        if inspect.isfunction(attribute):
+        if inspect.ismethod(attribute):
             assert attribute() == signature_attribute()
         else:
             assert attribute == signature_attribute