[DO NOT MERGE] Test SKIP_CERTIFICATE_DOWNLOAD

[iangmaia]

Testing SKIP_CERTIFICATE_DOWNLOAD variable.

[iangmaia]

@AliSoftware re: skipping certificates download. I've tested it here and it worked!

See the builds:

• Skipping certificate download
• Downloading certificate

Not downloading them still had the tests passing (🎉), though downloading them is very fast so we don't save a lot of time not downloading them. Perhaps it's worth it to still skip the download, but regardless it was a nice "TIL" moment 🙂

[AliSoftware]

Nice! It's good to have confirmation they're not needed for tests 🙂

I think at some point it'd make sense to separate the cert download in its own .ps1, and have it called from prepare_windows_host… unless -SkipDownloadCertificate switch is set etc… if only for consistency and clear separation of concerns/responsibilities in our toolkit scripts, and for the (admittedly very limited) slight improved security of not having such secrets on the machine if not necessary, more than for the ridiculous time it'd save.

But all that could also be a Linear issue for later.

[iangmaia]

I think at some point it'd make sense to separate the cert download in its own .ps1, and have it called from prepare_windows_host… unless -SkipDownloadCertificate switch is set etc… if only for consistency and clear separation of concerns/responsibilities in our toolkit scripts, and for the (admittedly very limited) slight improved security of not having such secrets on the machine if not necessary, more than for the ridiculous time it'd save.

Agreed 👍 I've created the issue AINFRA-1046 also referring the discussion about another script break down at #1608 (comment).