Skip to content

Commit

Permalink
[tests] Bump the timeout to 10min for monotouch-test/xammac_tests on …
Browse files Browse the repository at this point in the history
…older macOS bots. (#17317)

I've started seeing more random network delays on these tests recently - which
the tests themselves handle, but the test run ends up taking much longer, and
we need to give the test run more time to finish.
  • Loading branch information
rolfbjarne committed Jan 24, 2023
1 parent 9c0bfd4 commit 86ee42a
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions tests/packaged-macos-tests.mk
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ LAUNCH_ARGUMENTS=--autostart --autoexit

# Time test runs out after 5 minutes (300 seconds)
RUN_WITH_TIMEOUT=./run-with-timeout.csharp 300
# Some tests need a bit more time... (introspection)
# Some tests need a bit more time... (introspection, monotouch-test/xammac_tests)
RUN_WITH_TIMEOUT_LONGER=./run-with-timeout.csharp 600

.stamp-configure-projects-mac: Makefile xharness/xharness.exe
Expand Down Expand Up @@ -94,7 +94,7 @@ build-mac-modern-xammac_tests: xammac_tests/xammac_tests.csproj .stamp-nuget-res

exec-mac-modern-xammac_tests:
@echo "ℹ️ Executing the 'xammac' test for Xamarin.Mac (Modern profile) ℹ️"
$(Q) $(RUN_WITH_TIMEOUT) xammac_tests/bin/x86/$(CONFIG)/xammac_tests.app/Contents/MacOS/xammac_tests
$(Q) $(RUN_WITH_TIMEOUT_LONGER) xammac_tests/bin/x86/$(CONFIG)/xammac_tests.app/Contents/MacOS/xammac_tests

#
# link all
Expand Down Expand Up @@ -221,7 +221,7 @@ endif
$$(Q) if test -e ".$$@-failure.stamp"; then cat ".$$@-failure.stamp"; rm ".$$@-failure.stamp"; exit 1; fi
endef

$(eval $(call DotNetNormalTest,monotouch-test,monotouchtest))
$(eval $(call DotNetNormalTest,monotouch-test,monotouchtest,_LONGER))
$(eval $(call DotNetNormalTest,introspection,introspection,_LONGER))

### .NET linker tests
Expand Down

0 comments on commit 86ee42a

Please sign in to comment.