Skip to content
This repository has been archived by the owner on Nov 16, 2021. It is now read-only.

MULTISITE-22187: Run behat by default by excluding @wip tags. #54

Merged
merged 2 commits into from
Jan 8, 2020

Conversation

ghost
Copy link

@ghost ghost commented Jan 7, 2020

@ghost ghost mentioned this pull request Jan 7, 2020
jonhy81
jonhy81 previously approved these changes Jan 7, 2020
runner.yml.dist Outdated
@@ -9,6 +9,9 @@ drupal:
name: ${env.DRUPAL_ACCOUNT_USERNAME}
password: ${env.DRUPAL_ACCOUNT_PASSWORD}

behat:
tags: "~@clone"
Copy link
Member

Choose a reason for hiding this comment

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

The issue refers to allowing developers to decide which tests to exclude, I'll change this by defaulting it to @wip as that's the default from behat documentation, then they will be able to change it in their projects.

Copy link
Author

@ghost ghost Jan 8, 2020

Choose a reason for hiding this comment

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

Okey. But then we'll probably add ~@wip~@clone to the pipeline to avoid @clone tests from being run on a clean installation. And vica versa for the clone installation.

https://github.com/ec-europa/ne-pipelines/blob/master/drone/test/toolkit4.yml#L114
https://github.com/ec-europa/ne-pipelines/blob/master/drone/test/toolkit4.yml#L160

@ademarco ademarco changed the title MULTISITE-22187: Run behat by default by excluding @clone tags. MULTISITE-22187: Run behat by default by excluding @wip tags. Jan 8, 2020
@ademarco ademarco merged commit 0911378 into master Jan 8, 2020
@ademarco ademarco deleted the MULTISITE-22187 branch January 8, 2020 14:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants