retornam/mcom-tests — comment on commit 8cdb2b2fd23cda1c0b592e4714c336bee65202bc
File:  line 0
By stephendonner:

Thanks for working on this, Raymond! Just a few quick notes:

Mozilla.org/.com is served, by default, over http, so, although testing for HTTPS redirects is OK, and even wanted, we want to ensure the primary case is covered too, and first.

I'd also like to see us testing https://bugzilla.mozilla.org/show_bug.cgi?id=683782 (and other redirect behavior in general) more completely, by ensuring that loading:

[scheme/subdomain/top-level domain], where those are:

1) scheme = http / https
2) www / [empty/null string]
3) .org / .com

all do the right thing, whether that's redirecting (and via 301 or 301) or just returning a 200 OK.

e.g. https://www.mozilla.org, http://mozilla.org, https://www.mozilla.com, http://www.mozilla.com

If we could have a class or function that does that, then test_redirect_with_trailing_slash could pass in the appropriate test from the above list, rather than reprinting a bunch of code -- let me know how crazy that idea is :-)