Skip to content

Supply custom error message on timeout tests #4

Description

@Voileexperiments

Currently both JS and Python versions have test constructs to test if some pieces of code runs under a certain time limit, but they do not allow the use of custom error message like every other test constructs do.

So, like @timeout(sec) should become @timeout(sec, msg) with a default value?

Activity

FArekkusu commented on Mar 27, 2020

@FArekkusu
Contributor

Is this really required? The only error message which makes sense for the timeout decorator is "Exceeded time limit of N seconds", and that's exactly what it says right now.

Moreover, the default error message must always be prepended in case somebody decides to pass something useless as the msg argument (e.g. like it's done with assert_equals); so the purpose of this custom messages is... to add a comment?

added a commit that references this issue on Jun 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions