[pypy-dev] src/pypy/module/test/test_sysmodule.py

Laura Creighton lac at strakt.com
Mon Jul 5 23:00:45 CEST 2004


Over here we have somebody who likes to write unittests like this:


    def test_sys_in_modules(self):
        import sys
        modules = sys.modules
        self.failUnless('sys' in modules, "An entry for sys "
                                        "is not in sys.modules.")
        sys2 = sys.modules['sys']
        self.failUnless(sys is sys2, "import sys is not sys.modules[sys].")
    def test_builtin_in_modules(self):
        import sys
        modules = sys.modules
        self.failUnless('__builtin__' in modules, "An entry for __builtin__ "
                                                    "is not in sys.modules.")
        import __builtin__
        builtin2 = sys.modules['__builtin__']
        self.failUnless(__builtin__ is builtin2, "import __builtin__ "
                                            "is not sys.modules[__builtin__].")

see the fail messages that are two strings, not separated by anything, on 2 separate
lines.

My poor program converts them to:


    def test_sys_in_modules(self):
        import sys
        modules = sys.modules
        assert 'sys' in modules, "An entry for sys "
                                        "is not in sys.modules."

I propose we consider src/pypy/module/test/test_sysmodule.py broken, and we fix that
file, rather than make my program wrap them in parentheses.

Any objections?

Laura



More information about the Pypy-dev mailing list