mirror of
https://github.com/RIOT-OS/RIOT.git
synced 2024-12-29 04:50:03 +01:00
kconfiglib: remove TEST_KCONFIG symbol
This commit is contained in:
parent
e0fdc3c16c
commit
be8a2c84a1
9
dist/tools/kconfiglib/genconfig.py
vendored
9
dist/tools/kconfiglib/genconfig.py
vendored
@ -245,16 +245,9 @@ def check_configs(kconf):
|
||||
- A configuration parameter could not be set to value defined by the
|
||||
user.
|
||||
"""
|
||||
test_kconfig = os.getenv("TEST_KCONFIG")
|
||||
|
||||
if (test_kconfig):
|
||||
app_check = check_application_symbol(kconf)
|
||||
else:
|
||||
app_check = True
|
||||
|
||||
sym_check = check_config_symbols(kconf)
|
||||
choice_check = check_config_choices(kconf)
|
||||
return app_check and sym_check and choice_check
|
||||
return sym_check and choice_check
|
||||
|
||||
|
||||
def get_sym_missing_deps(sym):
|
||||
|
16
dist/tools/kconfiglib/tests/test.sh
vendored
16
dist/tools/kconfiglib/tests/test.sh
vendored
@ -18,8 +18,6 @@ KCONFIGLIB_DIR="$(readlink -f "$(dirname "$0")/..")"
|
||||
TESTS_DIR=${KCONFIGLIB_DIR}/tests
|
||||
GENCONFIG=${KCONFIGLIB_DIR}/genconfig.py
|
||||
|
||||
export TEST_KCONFIG=1
|
||||
|
||||
KCONFIG_FILE="${TESTS_DIR}/Kconfig.test"
|
||||
CONFIG_FOO_Y="${TESTS_DIR}/foo_y.config"
|
||||
CONFIG_BAR_Y="${TESTS_DIR}/bar_y.config"
|
||||
@ -93,19 +91,6 @@ function check_files_exist {
|
||||
fi
|
||||
}
|
||||
|
||||
# TEST
|
||||
# verifies that when the APPLICATION symbol is present, the script fails when
|
||||
# its value is not 'y' because of missing dependencies (e.g. missing features).
|
||||
function application_missing_deps_should_fail {
|
||||
if OUT=$(${GENCONFIG} --kconfig-filename "${KCONFIG_FILE}" 2>&1)
|
||||
then
|
||||
1>&2 echo -n "$OUT"
|
||||
return 1
|
||||
else
|
||||
return 0
|
||||
fi
|
||||
}
|
||||
|
||||
# TEST
|
||||
# verifies that when the dependencies for the APPLICATION symbol are present
|
||||
# the symbol gets the 'y' value and the script succeeds.
|
||||
@ -178,7 +163,6 @@ function choice_with_deps_should_succeed {
|
||||
}
|
||||
|
||||
run_test check_files_exist
|
||||
run_test application_missing_deps_should_fail
|
||||
run_test application_with_deps_should_succeed
|
||||
run_test missing_module_should_fail
|
||||
run_test module_with_deps_should_succeed
|
||||
|
Loading…
Reference in New Issue
Block a user