Service account tests run slowly if gcloud credentials present

[craigcitro]

This is the root cause of the slowness from #89. From the back-and-forth in #85, it looks like the tests happen to pick up existing credentials from the well-known file, which means our tests aren't self-contained (badness), and are slow to boot.

(Moving the gcloud directory is a temporary workaround, but clearly this needs fixed.)

[dhermes]

Monkey patching _get_well_known_file for those tests seems to be the easiest fix.

[craigcitro]

sure -- but we want to make sure _get_well_known_file still has a test. (maybe it does elsewhere, i just haven't looked.)

[dhermes]

I just meant in that test case, not in the whole suite.

[craigcitro]

totes! I'm just curious if there's another test to cover it ;)

On Mon, Dec 15, 2014, 6:38 PM Danny Hermes notifications@github.com wrote:

I just meant in that test case, not in the whole suite.

—
Reply to this email directly or view it on GitHub
#92 (comment).

[craigcitro]

just sat down and skimmed the code; i'm really curious about what's in your ~/.config/gcloud directory. in particular, all the tests validate that the files fetched match some fixed dummy values -- if you are loading values from disk, then either (1) you've got some weird values in your gcloud directory, or (2) we're spending time on something before loading the right files. if it's the latter, i want to know where that time is going.

[craigcitro]

turns out this was a red herring -- opening a new issue.