tidy up TODOs

Author: liamcervante

internal/backend/local/test.go

@@ -310,7 +310,7 @@ func (runner *TestFileRunner) Test(file *moduletest.File) {
 
 	// Update the manifest file with the reason why each state file was created.
 	err := runner.Suite.Manifest.WriteManifest()
-	diags = diags.Append(err) // TODO(liamcervante): diagnostic?
+	diags = diags.Append(err)
 
 	// If the graph walk was terminated, we don't want to add the diagnostics.
 	// The error the user receives will just be:

internal/moduletest/graph/node_state_cleanup.go

@@ -57,7 +57,6 @@ func (n *NodeStateCleanup) Execute(evalCtx *EvalContext) {
 	// If the state is empty, we still write it so we can store the
 	// output values, but we don't need to run a destroy operation.
 	if n.emptyState(state.State) {
-		// TODO(liamcervante): No diagnostics here!
 		diags = diags.Append(evalCtx.WriteFileState(n.stateKey, state))
 		evalCtx.Renderer().DestroySummary(diags, state.Run, file, state.State)
 		return