We shouldn't delete existing user, otherwise

it's not going to find the user, of course, right?

The `user` is already created in the other example,
unfortunately. If we `User.delete_all`, then `user`
would be pointing to a deleted user.

spec/requests/api/users_spec.rb

@@ -203,10 +203,9 @@ describe API::Users do
 
     context "when authenticated and ldap is enabled" do
       it "returns non-ldap user" do
-        User.delete_all
         create :omniauth_user, provider: "ldapserver1"
         get api("/users", user), skip_ldap: "true"
-        expect(response.status).to eq 200
+        expect(response).to have_gitlab_http_status(200)
         expect(json_response).to be_an Array
         username = user.username
         expect(json_response.first["username"]).to eq username