General review of tests

[AnthonySteele]

• remove api requests from testFixtureSetup inBasketCreateTests.cs as setup methods are not designed to be async. Use a method called in the async test instead. Removed the "BusyAwait()" hack as it's unused after this.
• deleted some tests which are ignored because they require special access rights. They don't belong in this solution any more.
• In many tests, separate the steps of building an api request from the step of firing it. Only the firing step with ".Please()" on the end needs "await" at the front of it. They can be all chained up if you like, but I prefer two steps as a matter of taste.
• fixed some poor names. Lots of things were called "artistBrowse" regardless of type.
• type respoinses as "var" as per house style