Ping @dfawley

LGTM. I'm a bit concerned about the classification of these as "end to end tests", since they don't actually test gRPC servers or clients, or anything we expose externally to the user for that matter. These are really system level tests for the xdsclient package, right? (And as such, probably belong under the xdsclient directory somewhere instead, as they were before.)

These are e2e tests from the point of view of the xds client, since it uses only exported APIs (and creates a new client and tests the scenarios being tested). The main reason I want these tests in a separate package is because our previous tests lived in the same package and ended up relying too much on internal state and being over-specified. I could have put them in the same directory, but in an xdsclient_test package (but I thought being in a separate directory helps a little since there is a lot of code in this directory).

That's fine, but I do think a package named e2e should be reserved for gRPC end to end tests, not tests done at the API level for a single package. Using a separate directory is OK, but I think it should be a different one, ideally under the xdsclient directory, e.g. xdsclient/test.

Fair point. I can change that once the last test PR (in this round) is in.