Skip to content

Commit 7a21a9d

Browse files
committed
test: process APP_NAME when deciding what apps to disable
When running acceptance tests, we disable the firstrunwizard and notifications app by default. These apps are often provided in a bundled ownCloud core tarball. But they can get in the way of webUI tests, because: - the firstrunwizard might display itself at the very start of testing - notifications can appear at the top of the webUI, and might obscure things that are being tested. But if we are actually testing one of these apps, then we don't want to disable the app. The logic in acceptance ytetst run.sh is enhanced to avoid disabling the app when it is the app-under-test.
1 parent bd12a8e commit 7a21a9d

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

tests/acceptance/run.sh

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1063,10 +1063,21 @@ then
10631063
done
10641064
fi
10651065

1066+
if [ -z "${APP_NAME}" ]
1067+
then
1068+
APP_NAME=""
1069+
fi
1070+
10661071
#Enable and disable apps as required for default
10671072
if [ -z "${APPS_TO_DISABLE}" ]
10681073
then
1069-
APPS_TO_DISABLE="firstrunwizard notifications"
1074+
if [[ "${APP_NAME}" == "notifications" ]]; then
1075+
APPS_TO_DISABLE="firstrunwizard"
1076+
elif [[ "${APP_NAME}" == "firstrunwizard" ]]; then
1077+
APPS_TO_DISABLE="notifications"
1078+
else
1079+
APPS_TO_DISABLE="firstrunwizard notifications"
1080+
fi
10701081
fi
10711082

10721083
if [ -z "${APPS_TO_ENABLE}" ]

0 commit comments

Comments
 (0)