Skip to content
This repository has been archived by the owner on Feb 20, 2019. It is now read-only.

Change integration to acceptance tests #3855

Merged
merged 1 commit into from
Mar 6, 2018

Conversation

phil-davis
Copy link
Contributor

Addresses issue #3854 resulting from core PR owncloud/core#30493

  1. Modify the folder name to reflect the actual new name.
  2. Refer to these tests as "acceptance" tests. This distinguishes them from other places in the documentation that talk about the type of integration test that uses multiple classes together - example https://github.com/owncloud/documentation/blob/master/developer_manual/app/tutorial/testing.rst
  3. Update the folder structure and a few other wordings that I noticed.

Later, as we move toward putting the UI tests in this acceptance folder, we will need to sort out more different words to use to describe the different categories of acceptance tests that we have - API end-to-end, UI end-to-end etc.

Copy link
Contributor

@settermjd settermjd left a comment

Choose a reason for hiding this comment

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

LGTM!

@phil-davis
Copy link
Contributor Author

This is dev/tester doc. It is implemented in core master and stable10, and in master of any repos that have these kind of tests. It is not something that is "released" in the usual sense, so it is "live" now.

Therefore this [can|should] be merged.

@settermjd settermjd merged commit f56aca8 into master Mar 6, 2018
@settermjd settermjd deleted the acceptance-tests-core-30493 branch March 6, 2018 07:00
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