fixing analysis errors

Author: tomarra

test/src/command_runner/commands/handle_completion_command_test.dart

@@ -24,6 +24,7 @@ class _TestCompletionCommandRunner extends CompletionCommandRunner<int> {
   String get executableName => 'test_cli';
 
   @override
+  // Override acceptable for test files
   // ignore: overridden_fields
   final Logger completionLogger = MockLogger();
 }

test/src/command_runner/commands/install_completion_files_command_test.dart

@@ -13,6 +13,7 @@ class _TestCompletionCommandRunner extends CompletionCommandRunner<int> {
   _TestCompletionCommandRunner() : super('test', 'Test command runner');
 
   @override
+  // Override acceptable for test files
   // ignore: overridden_fields
   final Logger completionInstallationLogger = _MockLogger();
 

test/src/command_runner/commands/uninstall_completion_files_command_test.dart

@@ -13,6 +13,7 @@ class _TestCompletionCommandRunner extends CompletionCommandRunner<int> {
   _TestCompletionCommandRunner() : super('test', 'Test command runner');
 
   @override
+  // Override acceptable for test files
   // ignore: overridden_fields
   final Logger completionInstallationLogger = _MockLogger();
 

test/src/command_runner/completion_command_runner_test.dart

@@ -18,10 +18,12 @@ class _TestCompletionCommandRunner extends CompletionCommandRunner<int> {
   bool enableAutoInstall = true;
 
   @override
+  // Override acceptable for test files
   // ignore: overridden_fields
   final Logger completionLogger = MockLogger();
 
   @override
+  // Override acceptable for test files
   // ignore: overridden_fields
   final Logger completionInstallationLogger = MockLogger();
 

test/src/installer/completion_configuration_test.dart

@@ -1,3 +1,4 @@
+// Not required for test files
 // ignore_for_file: prefer_const_constructors
 
 import 'dart:collection';

test/src/installer/completion_installation_test.dart

@@ -224,6 +224,7 @@ void main() {
 
         expect(configFile.existsSync(), true);
 
+        // Different format needed for matching cli output
         // ignore: leading_newlines_in_multiline_strings
         expect(configFile.readAsStringSync(), '''
 \n## [very_good]
@@ -285,6 +286,7 @@ void main() {
 
         installation.writeToShellConfigFile('very_good');
 
+        // Different format needed for matching cli output
         // ignore: leading_newlines_in_multiline_strings
         expect(rcFile.readAsStringSync(), '''
 \n## [Completion]
@@ -432,6 +434,7 @@ void main() {
 
           // rc fle includes one reference to the global config
 
+          // Different format needed for matching cli output
           // ignore: leading_newlines_in_multiline_strings
           expect(rcFile.readAsStringSync(), '''
 \n## [Completion]
@@ -446,6 +449,7 @@ void main() {
             path.join(configDir.path, 'zsh-config.zsh'),
           );
 
+          // Different format needed for matching cli output
           // ignore: leading_newlines_in_multiline_strings
           expect(globalConfig.readAsStringSync(), '''
 \n## [very_good]
@@ -486,6 +490,7 @@ void main() {
             ..install('very_good')
             ..install('not_good');
 
+          // Different format needed for matching cli output
           // ignore: leading_newlines_in_multiline_strings
           expect(bashProfile.readAsStringSync(), '''
 \n## [Completion]