Skip to content

Commit

Permalink
Moves tests that should be always executed to 'always' directory (#11948
Browse files Browse the repository at this point in the history
)

Some tests (testing the structure and importability of
example) should be always run even if core part was not modified.

That's why we move it to "always" directory.
  • Loading branch information
potiuk authored Oct 30, 2020
1 parent f000a69 commit 2124453
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 2 deletions.
1 change: 1 addition & 0 deletions CI.rst
Original file line number Diff line number Diff line change
Expand Up @@ -708,6 +708,7 @@ contributors in case of simpler changes.

We have the following test types (separated by packages in which they are):

* Always - those are tests that should be always executed (always folder)
* Core - for the core Airflow functionality (core folder)
* API - Tests for the Airflow API (api and api_connexion folders)
* CLI - Tests for the Airflow CLI (cli folder)
Expand Down
4 changes: 2 additions & 2 deletions scripts/ci/selective_ci_checks.sh
Original file line number Diff line number Diff line change
Expand Up @@ -173,7 +173,7 @@ function set_basic_checks_only() {
initialization::ga_output basic-checks-only "${@}"
}

ALL_TESTS="Core Other API CLI Providers WWW Integration Heisentests"
ALL_TESTS="Always Core Other API CLI Providers WWW Integration Heisentests"
readonly ALL_TESTS

function set_outputs_run_everything_and_exit() {
Expand Down Expand Up @@ -480,7 +480,7 @@ function calculate_test_types_to_run() {
echo
SELECTED_TESTS="${SELECTED_TESTS} WWW"
fi
initialization::ga_output test-types "Integration Heisentests ${SELECTED_TESTS}"
initialization::ga_output test-types "Always Integration Heisentests ${SELECTED_TESTS}"
fi
}

Expand Down
5 changes: 5 additions & 0 deletions scripts/in_container/entrypoint_ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -236,6 +236,7 @@ else
CLI_TESTS=("tests/cli")
API_TESTS=("tests/api" "tests/api_connexion")
PROVIDERS_TESTS=("tests/providers")
ALWAYS_TESTS=("tests/always")
CORE_TESTS=(
"tests/core"
"tests/executors"
Expand All @@ -253,6 +254,7 @@ else
"${API_TESTS[@]}"
"${PROVIDERS_TESTS[@]}"
"${CORE_TESTS[@]}"
"${ALWAYS_TESTS[@]}"
"${WWW_TESTS[@]}"
)

Expand All @@ -264,6 +266,8 @@ else
SELECTED_TESTS=("${PROVIDERS_TESTS[@]}")
elif [[ ${TEST_TYPE:=""} == "Core" ]]; then
SELECTED_TESTS=("${CORE_TESTS[@]}")
elif [[ ${TEST_TYPE:=""} == "Always" ]]; then
SELECTED_TESTS=("${ALWAYS_TESTS[@]}")
elif [[ ${TEST_TYPE:=""} == "WWW" ]]; then
SELECTED_TESTS=("${WWW_TESTS[@]}")
elif [[ ${TEST_TYPE:=""} == "Helm" ]]; then
Expand All @@ -272,6 +276,7 @@ else
find_all_other_tests
SELECTED_TESTS=("${ALL_OTHER_TESTS[@]}")
elif [[ ${TEST_TYPE:=""} == "All" || ${TEST_TYPE} == "Quarantined" || \
${TEST_TYPE} == "Always" || \
${TEST_TYPE} == "Postgres" || ${TEST_TYPE} == "MySQL" || \
${TEST_TYPE} == "Heisentests" || ${TEST_TYPE} == "Long" || \
${TEST_TYPE} == "Integration" ]]; then
Expand Down
File renamed without changes.
File renamed without changes.

0 comments on commit 2124453

Please sign in to comment.