Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Change time limit for unit tests to 60 seconds and introduce a decorator for long running tests #2661

Closed
wants to merge 1 commit into from
Closed

Change time limit for unit tests to 60 seconds and introduce a decorator for long running tests #2661

wants to merge 1 commit into from

Commits on Aug 22, 2024

  1. Change time limit for unit tests to 60 seconds and introduce a decora… 

    …tor for long running tests (facebook#2661)

Summary:
Pull Request resolved: facebook#2661

This diff is to change the time limit for unit tests to 60 seconds and introduce a decorator for long running tests.

We'll use the `ax_long_text` decorator to identify all long running tests, which we can fix over time by creating backlog tasks for test-owners to update these long-running tests with a descriptive message.

This change also introduces this decorator to all tests running over 60 seconds. For now, the provided reason for these long running tests are placeholders.

Reviewed By: mpolson64

Differential Revision: D61212556
    @paschai @facebook-github-bot
    paschai authored and facebook-github-bot committed Aug 22, 2024
    Configuration menu
    Copy the full SHA
    e1e5a13 View commit details
    Browse the repository at this point in the history