diff options
| author | David Göransson <david.goransson@mullvad.net> | 2025-05-20 10:11:35 +0200 |
|---|---|---|
| committer | David Göransson <david.goransson@mullvad.net> | 2025-05-21 11:56:46 +0200 |
| commit | 39eccff2fbdd35e7a427877f77a31eef1510284a (patch) | |
| tree | 428c51ffd972ec1f1e31b52a1473260bcd6899b5 | |
| parent | bf7f22e42500ba570776a08d132b6ee1e3a93df9 (diff) | |
| download | mullvadvpn-39eccff2fbdd35e7a427877f77a31eef1510284a.tar.xz mullvadvpn-39eccff2fbdd35e7a427877f77a31eef1510284a.zip | |
Update CI jobs to use new property naming scheme
| -rw-r--r-- | .github/workflows/android-app.yml | 7 | ||||
| -rwxr-xr-x | android/scripts/run-instrumented-tests.sh | 28 |
2 files changed, 24 insertions, 11 deletions
diff --git a/.github/workflows/android-app.yml b/.github/workflows/android-app.yml index caf8a9e700..5896844b3a 100644 --- a/.github/workflows/android-app.yml +++ b/.github/workflows/android-app.yml @@ -488,7 +488,10 @@ jobs: VALID_TEST_ACCOUNT_NUMBER: ${{ env.RESOLVED_TEST_ACCOUNT }} INVALID_TEST_ACCOUNT_NUMBER: '0000000000000000' ENABLE_HIGHLY_RATE_LIMITED_TESTS: ${{ github.event_name == 'schedule' && 'true' || 'false' }} - ENABLE_ACCESS_TO_LOCAL_API_TESTS: true + ENABLE_RAAS_TESTS: true + RAAS_HOST: '192.168.105.1' + RAAS_TRAFFIC_GENERATOR_TARGET_HOST: '45.83.223.209' + RAAS_TRAFFIC_GENERATOR_TARGET_PORT: '80' REPORT_DIR: ${{ steps.prepare-report-dir.outputs.report_dir }} run: ./android/scripts/run-instrumented-tests-repeat.sh ${{ needs.prepare.outputs.E2E_TEST_REPEAT }} @@ -513,7 +516,7 @@ jobs: invalid_test_account_number=0000000000000000,\ ENABLE_HIGHLY_RATE_LIMITED_TESTS=${{ github.event_name == 'schedule' && 'true' || 'false' }},\ partner_auth=${{ secrets.STAGEMOLE_PARTNER_AUTH }},\ - ENABLE_ACCESS_TO_LOCAL_API_TESTS=false" + ENABLE_RAAS_TESTS=false" strategy: fail-fast: false matrix: diff --git a/android/scripts/run-instrumented-tests.sh b/android/scripts/run-instrumented-tests.sh index 7606a2c634..96d8dc8eb6 100755 --- a/android/scripts/run-instrumented-tests.sh +++ b/android/scripts/run-instrumented-tests.sh @@ -17,7 +17,10 @@ PARTNER_AUTH="${PARTNER_AUTH:-}" VALID_TEST_ACCOUNT_NUMBER="${VALID_TEST_ACCOUNT_NUMBER:-}" INVALID_TEST_ACCOUNT_NUMBER="${INVALID_TEST_ACCOUNT_NUMBER:-}" ENABLE_HIGHLY_RATE_LIMITED_TESTS="${ENABLE_HIGHLY_RATE_LIMITED_TESTS:-false}" -ENABLE_ACCESS_TO_LOCAL_API_TESTS="${ENABLE_ACCESS_TO_LOCAL_API_TESTS:-false}" +ENABLE_RAAS_TESTS="${ENABLE_RAAS_TESTS:-false}" +RAAS_HOST="${RAAS_HOST:-}" +RAAS_TRAFFIC_GENERATOR_TARGET_HOST="${RAAS_TRAFFIC_GENERATOR_TARGET_HOST:-}" +RAAS_TRAFFIC_GENERATOR_TARGET_PORT="${RAAS_TRAFFIC_GENERATOR_TARGET_PORT:-}" REPORT_DIR="${REPORT_DIR:-}" while [[ "$#" -gt 0 ]]; do @@ -114,26 +117,35 @@ case "$TEST_TYPE" in echo "Error: The 'e2e' test type with billing flavor 'oss' require infra flavor 'prod'." exit 1 fi + OPTIONAL_TEST_ARGUMENTS="" if [[ -n ${INVALID_TEST_ACCOUNT_NUMBER-} ]]; then - OPTIONAL_TEST_ARGUMENTS+=" -e invalid_test_account_number $INVALID_TEST_ACCOUNT_NUMBER" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.$INFRA_FLAVOR.accountNumber.invalid $INVALID_TEST_ACCOUNT_NUMBER" else echo "Error: The variable INVALID_TEST_ACCOUNT_NUMBER must be set." exit 1 fi if [[ -n ${PARTNER_AUTH} ]]; then echo "Test account used for e2e test (provided/partner): partner" - OPTIONAL_TEST_ARGUMENTS+=" -e partner_auth $PARTNER_AUTH" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.$INFRA_FLAVOR.partnerAuth $PARTNER_AUTH" elif [[ -n ${VALID_TEST_ACCOUNT_NUMBER} ]]; then echo "Test account used for e2e test (provided/partner): provided" - OPTIONAL_TEST_ARGUMENTS+=" -e valid_test_account_number $VALID_TEST_ACCOUNT_NUMBER" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.$INFRA_FLAVOR.accountNumber.valid $VALID_TEST_ACCOUNT_NUMBER" else echo "" echo "Error: The variable PARTNER_AUTH or VALID_TEST_ACCOUNT_NUMBER must be set." exit 1 fi - OPTIONAL_TEST_ARGUMENTS+=" -e enable_access_to_local_api_tests $ENABLE_ACCESS_TO_LOCAL_API_TESTS" - OPTIONAL_TEST_ARGUMENTS+=" -e enable_highly_rate_limited_tests $ENABLE_HIGHLY_RATE_LIMITED_TESTS" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.config.raas.enable $ENABLE_RAAS_TESTS" + + if [[ -n ${ENABLE_RAAS_TESTS} ]]; then + echo "Tests dependent on local API enabled" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.config.raas.host $RAAS_HOST" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.config.raas.trafficGenerator.target.host $RAAS_TRAFFIC_GENERATOR_TARGET_HOST" + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.config.raas.trafficGenerator.target.port $RAAS_TRAFFIC_GENERATOR_TARGET_PORT" + fi + + OPTIONAL_TEST_ARGUMENTS+=" -e mullvad.test.e2e.config.runHighlyRateLimitedTests $ENABLE_HIGHLY_RATE_LIMITED_TESTS" USE_ORCHESTRATOR="true" PACKAGE_NAME="net.mullvad.mullvadvpn" if [[ "$INFRA_FLAVOR" =~ ^(devmole|stagemole)$ ]]; then @@ -150,8 +162,6 @@ if [[ -z $REPORT_DIR || ! -d $REPORT_DIR ]]; then exit 1 fi -GRADLE_ENVIRONMENT_VARIABLES="TEST_E2E_ENABLEACCESSTOLOCALAPITESTS=$ENABLE_ACCESS_TO_LOCAL_API_TESTS" - INSTRUMENTATION_LOG_FILE_PATH="$REPORT_DIR/instrumentation-log.txt" LOGCAT_FILE_PATH="$REPORT_DIR/logcat.txt" LOCAL_SCREENSHOT_PATH="$REPORT_DIR/screenshots" @@ -229,7 +239,7 @@ else -e runnerBuilder de.mannodermaus.junit5.AndroidJUnit5Builder \ $TEST_PACKAGE_NAME/androidx.test.runner.AndroidJUnitRunner" fi -adb shell "$GRADLE_ENVIRONMENT_VARIABLES $INSTRUMENTATION_COMMAND" | tee "$INSTRUMENTATION_LOG_FILE_PATH" +adb shell "$INSTRUMENTATION_COMMAND" | tee "$INSTRUMENTATION_LOG_FILE_PATH" echo "" echo "### Stop logging ###" |
