neomutt: fix duplicated extraConfig in account (#1546)

The `accounts.email.accounts.<name>.neomutt.extraConfig` option is
included twice in the resulting config file for the account. One time as
part of the `mraSection`, one time as part of `accountStr` (`accountStr`
includes the `mraSection`). This removes that duplication. I opted to
keep the one in `accounStr`, since `extraConfig` doesn't necessarily
have anything to do with the `mraSection`.

modules/programs/neomutt.nix

@@ -124,7 +124,7 @@ let
     let
       folderHook = mapAttrsToList setOption (genCommonFolderHooks account // {
         folder = "'${account.maildir.absPath}'";
-      }) ++ optional (neomutt.extraConfig != "") neomutt.extraConfig;
+      });
     in ''
       ${concatStringsSep "\n" folderHook}
     '';

tests/modules/programs/neomutt/hm-example.com-expected

@@ -25,8 +25,6 @@ set realname='H. M. Test'
 set record='+Sent'
 set spoolfile='+Inbox'
 set trash='+Trash'
-color status cyan default
-
 
 
 # Extra configuration

tests/modules/programs/neomutt/hm-example.com-msmtp-expected.conf

@@ -24,8 +24,6 @@ set realname='H. M. Test'
 set record='+Sent'
 set spoolfile='+Inbox'
 set trash='+Trash'
-color status cyan default
-
 
 
 # Extra configuration