Slaves should pull git repos from the master

- This changes our original design of passing the
project type parameters down to the slaves verbatim.
We now modify them so the url points to the master's
Git repo
Just a random minor thought on style: one thing I've been doing while writing tests that helps me when I go back to read them is putting blank lines between the arrange, act, and assert sections of the test. Personally I've found it helps me to organize the test if I separate the test into three phases like that.

Also, when I go back to a test later it helps me quickly determine what is just setup ("arrange"), the code or method I'm actually testing ("act") and then the assertions I'm making ("assert"). I kind of did this in another test in this file for the `slave.teardown()` tests. 

Definitely a personal preference type of thing, but I thought I'd just call it out in case it would help anyone else. :smile: