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.

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

Add Codeception tool #73

Open
tormi opened this issue Sep 30, 2019 · 3 comments
Open

Add Codeception tool #73

tormi opened this issue Sep 30, 2019 · 3 comments
Assignees

Comments

@tormi
Copy link
Member

tormi commented Sep 30, 2019

There is currently a lando codeception tool defined in .lando.yml, but it has never worked because there's no codeception package added to composer.json:

tooling:
  codeception:
    service: appserver
    description: Run codeception
    cmd:
      - ./vendor/bin/codecept
      - "--env=lando"

Let's add either codeception/codeception or guncha25/drupal-codeception package + testing setup.

See also:

cc: @guncha25 , @floretan

@tormi tormi changed the title Add Codeception tool Add a lando codecept tool Sep 30, 2019
@tormi
Copy link
Member Author

tormi commented Sep 30, 2019

I've removed non-functional lando codecept tool for now here: 9bf67d5

@tormi tormi changed the title Add a lando codecept tool Fix Codeception tool Sep 30, 2019
@tormi
Copy link
Member Author

tormi commented Sep 30, 2019

Readme:

Tool definition:

  codecept:
    description: Runs codeception tests
    cmd:
      - appserver: ./vendor/bin/codecept

@guncha25 guncha25 self-assigned this Oct 1, 2019
@tormi tormi changed the title Fix Codeception tool Add Codeception tool Oct 2, 2019
@tormi
Copy link
Member Author

tormi commented Mar 3, 2021

Our internal guide which needs to be implemented here: https://intra.wunder.io/info/documentation-and-best-practices/drupal-project-testing-setup

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants