Unverified Commit f35a61d8 authored by Felix Stupp's avatar Felix Stupp
Browse files

lib.modules: fix test line using empty "$@" 

Probably a missed left over from somewhere in
the commit 58f385f6.
As can be seen in that commit where this line was introduced,
"$@" was also just emptied by the last `set` call in line 169.
This line is currently valid, but breaks suddenly when somewhere earlier
a `set --` instruction is used in the future.
Neither in commit 58f385f6
nor in PR https://github.com/NixOS/nixpkgs/pull/197547
have I found anything stating that this "defect" was intentional.
parent abb12714

lib/tests/modules.sh

+1 −1
Original line number Diff line number Diff line
@@ -535,7 +535,7 @@ checkConfigError 'A submoduleWith option is declared multiple times with conflic 


# _type check

checkConfigError 'Expected a module, but found a value of type .*"flake".*, while trying to load a module into .*/module-imports-_type-check.nix' config.ok.config ./module-imports-_type-check.nix

checkConfigOutput '^true$' "$@" config.enable ./declare-enable.nix ./define-enable-with-top-level-mkIf.nix

checkConfigOutput '^true$' config.enable ./declare-enable.nix ./define-enable-with-top-level-mkIf.nix

checkConfigError 'Expected a module, but found a value of type .*"configuration".*, while trying to load a module into .*/import-configuration.nix.' config ./import-configuration.nix

checkConfigError 'please only import the modules that make up the configuration' config ./import-configuration.nix