Revert "fix: dry run fails when table/view/assertion already does not exsist …" (#1834)

This reverts commit 4d20929.

Author: Elias Kassell

cli/api/commands/run.ts

@@ -365,7 +365,6 @@ export class Runner {
       // (i.e. it must still be RUNNING, and not FAILED).
       actionResult.status === dataform.ActionResult.ExecutionStatus.RUNNING &&
       !(this.graph.runConfig && this.graph.runConfig.disableSetMetadata) &&
-      !(this.executionOptions?.bigquery?.dryRun) &&
       action.type === "table"
     ) {
       try {

cli/api/dbadapters/execution_sql.ts

@@ -47,10 +47,10 @@ export class ExecutionSql {
   }
 
   public insertInto(target: dataform.ITarget, columns: string[], query: string) {
-    return `
-insert into ${this.resolveTarget(target)}
-(${columns.join(",")})
-select ${columns.join(",")}
+    return `	
+insert into ${this.resolveTarget(target)}	
+(${columns.join(",")})	
+select ${columns.join(",")}	
 from (${query}) as insertions`;
   }
 
@@ -174,6 +174,7 @@ from (${query}) as insertions`;
     const tasks = new Tasks();
     const target = assertion.target;
     tasks.add(Task.statement(this.createOrReplaceView(target, assertion.query)));
+    tasks.add(Task.assertion(`select sum(1) as row_count from ${this.resolveTarget(target)}`));
     return tasks;
   }