Merge #153424

153424: changefeedccl: replace incorrect usage of QueryRow in tests r=log-head a=andyyang890

Two recently added ALTER CHANGEFEED unit tests use `QueryRow` instead of
`Exec` for queries that don't return values, which technically could
allow an error to go unnoticed. This patch corrects that.

Epic: None

Release note: None

Co-authored-by: Andy Yang <[email protected]>

Author: craig[bot]

pkg/ccl/changefeedccl/alter_changefeed_test.go

@@ -317,7 +317,7 @@ func TestAlterChangefeedAddTargetAfterInitialScan(t *testing.T) {
 				t.Fatalf("unknown initial scan type %q", initialScan)
 			}
 
-			sqlDB.QueryRow(t, `INSERT INTO foo VALUES(2)`)
+			sqlDB.Exec(t, `INSERT INTO foo VALUES(2)`)
 			assertPayloads(t, testFeed, []string{
 				`foo: [2]->{"after": {"a": 2}}`,
 			})
@@ -1946,7 +1946,7 @@ func TestAlterChangefeedAddDropSameTarget(t *testing.T) {
 		sqlDB.Exec(t, fmt.Sprintf(`ALTER CHANGEFEED %d ADD bar DROP bar`, feed.JobID()))
 		require.NoError(t, feed.Resume())
 		var tsStr string
-		sqlDB.QueryRow(t, `INSERT INTO bar VALUES(1)`)
+		sqlDB.Exec(t, `INSERT INTO bar VALUES(1)`)
 		sqlDB.QueryRow(t, `INSERT INTO foo VALUES(2) RETURNING cluster_logical_timestamp()`).Scan(&tsStr)
 		ts := parseTimeToHLC(t, tsStr)
 		require.NoError(t, feed.WaitForHighWaterMark(ts))