Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Unload fixture in tearDown #44

Merged
merged 1 commit into from
May 22, 2017
Merged

Unload fixture in tearDown #44

merged 1 commit into from
May 22, 2017

Conversation

alexpts
Copy link
Contributor

@alexpts alexpts commented May 22, 2017

Текущее поведение: Предыдущий класс с тестами загружает набор своих фикстур. Следующий тест выгружает уже свои фикстуры и загружает свои фикстуры. При этом состояние БД может хранить все или частичные данные из классов предыдущих фикстур.

Ожидается: каждый класс с тетами изолирует свое состояние. Выгружает все, что загрузил. Каждый новый класс с тестами имеет чистую базу.

Видел, что проект deprecated. Но пока не все проекты перенесли на чистый codeception.

Q A
Is bugfix? yes/no
New feature? yes/no
Breaks BC? yes/no
Tests pass? yes/no
Fixed issues comma-separated list of tickets # fixed by the PR, if any

Текущее поведение: Предыдущий класс с тестами загружает набор своих фикстур. Следующий тест выгружает уже свои фикстуры и загружает свои фикстуры. При этом состояние БД может хранить все или частичные данные из классов предыдущих фикстур.

Ожидается: каждый класс с тетами изолирует свое состояние. Выгружает все, что загрузил. Каждый новый класс с тестами имеет чистую базу.

Видел, что проект deprecated. Но пока не все проекты перенесли на чистый codeception.
@alexpts
Copy link
Contributor Author

alexpts commented May 22, 2017

retest this please

@samdark
Copy link
Member

samdark commented May 22, 2017

I can merge it and tag release but don't currently have time to test it.

@samdark samdark self-assigned this May 22, 2017
@alexpts
Copy link
Contributor Author

alexpts commented May 22, 2017

It will be great.
I tried this on our test suites. Our working set of tests works well.
So I was not able to run all project tests. Apparently, Travis doesn't work correctly.

@samdark samdark merged commit 086c8c2 into yiisoft:master May 22, 2017
@samdark
Copy link
Member

samdark commented May 22, 2017

Merged and tagged.

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.

2 participants