graphql: Minor delete mutation msg fix

[vardhanapoorv]

This PR changes the delete msg reported for the case when no nodes are deleted to No nodes were deleted. In addition, it corrects some tests which were wrong earlier.

Fixes #GRAPHQL-423.

This change is 

Docs Preview:

[MichaelJCompton]

Looks fine.

But looks like there's some other places in tests where that message was being relied on.

Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @manishrjain)

[MichaelJCompton]

Awesome!!! Love it.

Merge this and then either fix or remove that TODO.

Reviewed 4 of 4 files at r2.
Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @manishrjain and @vardhanapoorv)

---

ee/acl/acl_test.go, line 90 at r2 (raw file):

func deleteUser(t *testing.T, accessToken, username string, confirmDeletion bool) {
	// TODO - Verify that only one uid got deleted once numUids are returned as part of the payload.

Can we also fix this TODO ? Just merge this PR and do separately.

---

ee/acl/acl_test.go, line 169 at r2 (raw file):

func TestGetCurrentUser(t *testing.T) {
	accessJwt, _ := testutil.GrootHttpLogin(adminEndpoint)

Thanks - always good to see code get cleaned up.

---

ee/acl/acl_test.go, line 199 at r2 (raw file):

	// adding the user again should fail
	accessJwt, _ := testutil.GrootHttpLogin(adminEndpoint)

again nice simplification. - looks like there has been some repeated code in here.

---

graphql/e2e/common/mutation.go, line 2298 at r2 (raw file):

func cleanupStarwars(t *testing.T, starshipID, humanID, droidID string) {
	// Delete everything
	multiMutationParams := &GraphQLParams{

love it!

[abhimanyusinghgaur]

Reviewable status: all files reviewed, 4 unresolved discussions (waiting on @manishrjain, @MichaelJCompton, and @vardhanapoorv)

---

ee/acl/acl_test.go, line 90 at r2 (raw file):

Previously, MichaelJCompton (Michael Compton) wrote…

Can we also fix this TODO ? Just merge this PR and do separately.

Ok, will do it separately.