Added method to set logger of a task, used to set logger to nil to fi…

…x task comparison unit test

controller/driver_factory_test.go

@@ -356,7 +356,7 @@ func TestNewDriverTask(t *testing.T) {
 
 	for _, tc := range testCases {
 		t.Run(tc.name, func(t *testing.T) {
-			tc.conf.Finalize()
+			_ = tc.conf.Finalize()
 
 			var providerConfigs []driver.TerraformProviderBlock
 			if tc.conf != nil && tc.conf.TerraformProviders != nil {
@@ -368,7 +368,16 @@ func TestNewDriverTask(t *testing.T) {
 
 			tasks, err := newTestDriverTasks(tc.conf, providerConfigs)
 			assert.NoError(t, err)
-			assert.Equal(t, tc.tasks, tasks)
+
+			assert.Equal(t, len(tc.tasks), len(tasks))
+			for i := 0; i < len(tc.tasks); i++ {
+				expected := tc.tasks[i]
+				actual := tasks[i]
+
+				expected.SetLogger(nil)
+				actual.SetLogger(nil)
+				assert.Equal(t, expected, actual)
+			}
 		})
 	}
 }

driver/task.go

@@ -225,7 +225,7 @@ func (t *Task) Env() map[string]string {
 	return env
 }
 
-// ProviderNames returns the list of providers that the task has configured
+// Providers returns the list of providers that the task has configured
 func (t *Task) Providers() TerraformProviderBlocks {
 	t.mu.RLock()
 	defer t.mu.RUnlock()
@@ -335,6 +335,10 @@ func (s Service) copy() Service {
 	return cp
 }
 
+func (t *Task) SetLogger(logger logging.Logger) {
+	t.logger = logger
+}
+
 // configureRootModuleInput sets task values for the module input.
 func (t *Task) configureRootModuleInput(input *tftmpl.RootModuleInputData) error {
 	t.mu.RLock()