Apply code review

Author: d2dyno1

src/Sdk/SecureFolderFS.Sdk/ViewModels/Views/Wizard/DataSources/AccountSourceWizardViewModel.cs

@@ -188,7 +188,7 @@ private async Task SelectAccountAsync(AccountViewModel? accountViewModel, Cancel
         public override void Dispose()
         {
             NavigationRequested = null;
-            Accounts.Select(x => x != SelectedAccount).DisposeAll();
+            Accounts.Where(x => x != SelectedAccount).DisposeAll();
             Accounts.Clear();
         }
     }

src/Shared/SecureFolderFS.SourceGenerator/InjectGenerator.cs

@@ -15,9 +15,6 @@ namespace SecureFolderFS.SourceGenerator
     [Generator(LanguageNames.CSharp)]
     public sealed class InjectGenerator : AttributeWithTypeGenerator
     {
-        private const string LoggerFullName = "Microsoft.Extensions.Logging.ILogger";
-        private const string LoggerFactoryFullName = "Microsoft.Extensions.Logging.ILoggerFactory";
-
         protected override string AttributeNamespace { get; } = $"{nameof(SecureFolderFS)}.Sdk.Attributes.InjectAttribute`1";
 
         protected override string? GetCode(INamedTypeSymbol typeSymbol, ImmutableArray<AttributeData> attributes)

tests/SecureFolderFS.Tests/CliTests/Commands/VaultInfoCommandTests.cs

@@ -21,6 +21,7 @@ public async Task VaultInfo_OnMissingPath_ShouldReturnVaultUnreadable()
     }
 
     [Test]
+    [Ignore("Since Cli currently uses hardcoded SystemFolderEx, we cannot test against MemoryFolderEx")]
     public async Task VaultInfo_OnExistingVault_ShouldReturnInformation()
     {
         // Arrange