Changeset 597


Ignore:
Timestamp:
Nov 22, 2015, 8:25:02 AM (4 years ago)
Author:
cito
Message:

Use better and fully qualified Python class names

That way, the module name is automatically made available on the class.

Location:
trunk/module
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/module/TEST_PyGreSQL_classic_connection.py

    r596 r597  
    7878
    7979    def testClassName(self):
    80         self.assertEqual(self.connection.__class__.__name__, 'pgconnobject')
     80        self.assertEqual(self.connection.__class__.__name__, 'Connection')
    8181
    8282    def testModuleName(self):
    83         self.assertEqual(self.connection.__module__, 'pg')
     83        self.assertEqual(self.connection.__class__.__module__, 'pg')
     84
     85    def testStr(self):
     86        r = str(self.connection)
     87        self.assertTrue(r.startswith('<pg.Connection object'), r)
     88
     89    def testRepr(self):
     90        r = repr(self.connection)
     91        self.assertTrue(r.startswith('<pg.Connection object'), r)
    8492
    8593    def testAllConnectAttributes(self):
     
    244252        self.c.close()
    245253
     254    def testClassName(self):
     255        r = self.c.query("select 1")
     256        self.assertEqual(r.__class__.__name__, 'Query')
     257
     258    def testModuleName(self):
     259        r = self.c.query("select 1")
     260        self.assertEqual(r.__class__.__module__, 'pg')
     261
    246262    def testStr(self):
    247263        q = ("select 1 as a, 'hello' as h, 'w' as world"
     
    257273    def testRepr(self):
    258274        r = repr(self.c.query("select 1"))
    259         self.assertTrue(r.startswith('<pgqueryobject object'), r)
     275        self.assertTrue(r.startswith('<pg.Query object'), r)
    260276
    261277    def testSelect0(self):
  • trunk/module/TEST_PyGreSQL_classic_largeobj.py

    r596 r597  
    160160
    161161    def testClassName(self):
    162         self.assertEqual(self.obj.__class__.__name__, 'pglarge')
     162        self.assertEqual(self.obj.__class__.__name__, 'LargeObject')
    163163
    164164    def testModuleName(self):
    165         self.assertEqual(self.obj.__module__, 'pg')
     165        self.assertEqual(self.obj.__class__.__module__, 'pg')
    166166
    167167    def testOid(self):
     
    189189    def testRepr(self):
    190190        r = repr(self.obj)
    191         self.assertTrue(r.startswith('<pglarge object'), r)
     191        self.assertTrue(r.startswith('<pg.LargeObject object'), r)
    192192
    193193    def testOpen(self):
  • trunk/module/pgmodule.c

    r596 r597  
    981981static PyTypeObject largeType = {
    982982        PyVarObject_HEAD_INIT(NULL, 0)
    983         "pglarge",                                              /* tp_name */
     983        "pg.LargeObject",                               /* tp_name */
    984984        sizeof(largeObject),                    /* tp_basicsize */
    985985        0,                                                              /* tp_itemsize */
     
    22952295static PyTypeObject connType = {
    22962296        PyVarObject_HEAD_INIT(NULL, 0)
    2297         "pgconnobject",                         /* tp_name */
     2297        "pg.Connection",                        /* tp_name */
    22982298        sizeof(connObject),                     /* tp_basicsize */
    22992299        0,                                                      /* tp_itemsize */
     
    29482948static PyTypeObject sourceType = {
    29492949        PyVarObject_HEAD_INIT(NULL, 0)
    2950         "pgsourceobject",                               /* tp_name */
     2950        "pgdb.Source",                                  /* tp_name */
    29512951        sizeof(sourceObject),                   /* tp_basicsize */
    29522952        0,                                                              /* tp_itemsize */
     
    36093609static PyTypeObject noticeType = {
    36103610        PyVarObject_HEAD_INIT(NULL, 0)
    3611         "pgnoticeobject",                               /* tp_name */
     3611        "pg.Notice",                                    /* tp_name */
    36123612        sizeof(noticeObject),                   /* tp_basicsize */
    36133613        0,                                                              /* tp_itemsize */
     
    36613661static PyTypeObject queryType = {
    36623662        PyVarObject_HEAD_INIT(NULL, 0)
    3663         "pgqueryobject",                                /* tp_name */
     3663        "pg.Query",                                             /* tp_name */
    36643664        sizeof(queryObject),                    /* tp_basicsize */
    36653665        0,                                                              /* tp_itemsize */
     
    42904290                ) return NULL;
    42914291
    4292         /* make the module names available */
    4293         s = PyStr_FromString("pg");
    4294         PyDict_SetItemString(connType.tp_dict, "__module__", s);
    4295         PyDict_SetItemString(noticeType.tp_dict, "__module__", s);
    4296         PyDict_SetItemString(queryType.tp_dict, "__module__", s);
    4297 #ifdef LARGE_OBJECTS
    4298         PyDict_SetItemString(largeType.tp_dict, "__module__", s);
    4299 #endif
    4300         Py_DECREF(s);
    4301         s = PyStr_FromString("pgdb");
    4302         PyDict_SetItemString(sourceType.tp_dict, "__module__", s);
    4303         Py_DECREF(s);
    4304 
    43054292        dict = PyModule_GetDict(mod);
    43064293
Note: See TracChangeset for help on using the changeset viewer.