add a test

Signed-off-by: Nick Van Wiggeren <[email protected]>

Author: nickvanw

go/vt/vtgate/executor_test.go

@@ -3355,6 +3355,47 @@ func TestExecutorShowShards(t *testing.T) {
 			destTabletType: topodatapb.TabletType_PRIMARY,
 			wantErr:        "testing error getting keyspace ks1",
 		},
+		{
+			// Test that deleted keyspaces (returning NoNode error) are skipped
+			// rather than causing the entire query to fail.
+			name: "Deleted keyspace (NoNode) should be skipped",
+			srvTopoServer: &fakesrvtopo.FakeSrvTopo{
+				SrvKeyspaceNamesOutput: map[string][]string{
+					localCell: {"ks1", "ks2"},
+				},
+				SrvKeyspaceError: map[string]map[string]error{
+					localCell: {
+						"ks1": topo.NewError(topo.NoNode, "ks1"),
+					},
+				},
+				SrvKeyspaceOutput: map[string]map[string]*topodatapb.SrvKeyspace{
+					localCell: {
+						"ks2": {
+							Partitions: []*topodatapb.SrvKeyspace_KeyspacePartition{
+								{
+									ServedType: topodatapb.TabletType_PRIMARY,
+									ShardReferences: []*topodatapb.ShardReference{
+										{Name: "-40"},
+										{Name: "40-80"},
+										{Name: "80-"},
+									},
+								},
+							},
+						},
+					},
+				},
+			},
+			filter:         nil,
+			destTabletType: topodatapb.TabletType_PRIMARY,
+			want: &sqltypes.Result{
+				Fields: buildVarCharFields("Shards"),
+				Rows: []sqltypes.Row{
+					buildVarCharRow("ks2/-40"),
+					buildVarCharRow("ks2/40-80"),
+					buildVarCharRow("ks2/80-"),
+				},
+			},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {