humanmade / humanmade/altis-dev-tools-command

Make default timeout for codeception into env var

Open
#36 1 comment 0 reactions 0 assignees View on GitHub
should have testing
Dominant language
PHP
Stars
1
Forks
0
PR merge metrics
No merged PRs in 30d

Description

In some cases, we're seeing timeouts on test runs from the selenium web driver. This might be an inability to resolve the server, but could also be that things are just exceptionally slow sometimes.

Being able to configure the default codeception timeout globally could be very useful for debugging but also for making sure test runs are consistent in their outcome.

Acceptance criteria:

- [ ] Default codeception timeout value can be set via an environment variable at run time called `CODECEPT_TIMEOUT`
- [ ] Validate that the timeout is applied on long running tests
- [ ] Add documentation about the variable

---

## For Altis Team Use

### Ready for Work Checklist

Is this ticket ready to be worked on? See
[the Play Book Definition of Ready](https://playbook.hmn.md/play/product/definition-of-ready/)

- [x] Is the title clear?
- [x] Is the description clear and detailed enough?
- [x] Are acceptance criteria listed?
- [x] Have any (potential) dependencies been identified?
- [x] Is an estimate or time box assigned?
- [x] Is a priority label assigned?
- [ ] Is this ticket added to an epic? (Optional)

### Completion Checklist

Is this ticket done? See
[the Play Book Definition of Ready](https://playbook.hmn.md/play/product/definition-of-ready/)

- [ ] Has the acceptance criteria been met?
- [ ] Is the documentation updated (including README)?
- [ ] Does any code/documentation meet project standards?
- [ ] Are automatic tests in place to verify the fix or new functionality?
- [ ] Or are manual tests documented (at least on this ticket)?
- [ ] Are any Playbook/Handbook pages updated?
- [ ] Has a new module release (patch/minor) been created/scheduled?
- [ ] Have the appropriate `backport` labels been added to the PR?

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.