Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Add timeout support to test_reflective_loader. #14

Merged
merged 2 commits into from
Mar 26, 2018

Conversation

stereotype441
Copy link
Contributor

To use it, annotate a method with @TestTimeout(t) where t is a const
instance of the Timeout class from the test package. E.g.:

@TestTimeout(const Timeout.factor(2))

To use it, annotate a method with @TestTimeout(t) where t is a const
instance of the Timeout class from the `test` package.  E.g.:

  @TestTimeout(const Timeout.factor(2))
@@ -264,6 +274,16 @@ class ReflectiveTest {
const ReflectiveTest();
}

/**
* A marker annotation used to annotate overridden test methods with additional
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it just for overridden test methods, or we can use this also for the declaration of test methods?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, this was a copy-paste error. Fixed.

@stereotype441 stereotype441 merged commit bfd06b8 into master Mar 26, 2018
@stereotype441 stereotype441 deleted the add-timeout-support branch March 26, 2018 17:24
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 11, 2024
…ang/add-timeout-support

Add timeout support to test_reflective_loader.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants