Merge lp:~ahasenack/txaws/no-local-dir-for-ca-lookup into lp:txaws

[Andreas Hasenack]

This branch changes get_ca_certs() so that it does not accidentally include the current directory when looking for *.pem CA files to load.

[Jamu Kakar (jkakar)]

[1]

The test is tainting the TXAWS_CERTS_PATH environment variable. You
could do something like the following. Also, I recommend you add a
simple sentence clearly describing the expected behaviour.

def test_get_ca_certs_no_current_dir(self):
    """
    A L{CertsNotFoundError} exception is raised if no certificate paths are
    available.
    """
    original_certs_path = os.environ.get("TXAWS_CERTS_PATH")
    try:
        os.environ["TXAWS_CERTS_PATH"] = "%s:" % self.no_certs_dir
        os.chdir(self.one_cert_dir)
        self.assertRaises(exception.CertsNotFoundError, ssl.get_ca_certs)
    finally:
        if original_certs_path:
            os.environ["TXAWS_CERTS_PATH"] = original_certs_path
        else:
            del os.environ["TXAWS_CERTS_PATH"]

Nice work, +1! :)

review: Approve

[Andreas Hasenack (ahasenack)]

That docstring is not accurate. What I'm testing is that get_ca_certs() does not include the current directory if TXAWS_CERTS_PATH ends with a ":".

Good point about the variable remaining tainted, will fix, thanks.

[Jamu Kakar (jkakar)]

Hehe, that's a good reason to provide clear docstrings with tests.
They can be so easy to misunderstand, especially when you read the
code before the morning coffee has hit your bloodstream. ;b

[Andreas Hasenack (ahasenack)]

Can't I just put the os.chdir() that goes back to the original dir in that finally block too?

[Andreas Hasenack (ahasenack)]

NM, got some addCleanup() introduction from radix :)

[Jamu Kakar (jkakar)]

Yeah, addCleanup is your friend.

[Andreas Hasenack (ahasenack)]

Ok, how about this now.

[Andreas Hasenack (ahasenack)]

Having fun here with such a simple case :)
v3.0!

[lp:~ahasenack/txaws/no-local-dir-for-ca-lookup]

158. By Andreas Hasenack on 2012-10-30

Tidy up

[Christopher Armstrong (radix)]

Looks good. Thanks for moving the os.getcwd() call and fixing the other tests. +1

review: Approve