Skip to content

Conversation

crazedr0m
Copy link
Contributor

черновой набросок теста для View и PartViewer

@AlexeyDsov
Copy link
Member

Тесты нужны, с этим не поспоришь.

@AlexeyDsov
Copy link
Member

Сейчас утром еще раз смотрю тесты и обратил внимание - как-то оно необычно для тестов фреймворка использовать конструктор тестового класса. Обычно для подобных целей используется setUp метод, если нужно что-то инициализировать. Если же ну очень хочется инитить один раз что-то, а не для каждого тестового метода, то есть такой метод как setUpBeforeClass.
http://www.phpunit.de/manual/3.5/en/fixtures.html#fixtures.examples.TemplateMethodsTest.php

@crazedr0m
Copy link
Contributor Author

Спасибо.

@AlexeyDsov
Copy link
Member

А что в итоге с этим реквестом? Автор все что хотел - сделал? А то ведь написано в шапке что это черновой набросок.

@crazedr0m
Copy link
Contributor Author

Автор все что хотел, сделал, да. )

@dovg
Copy link
Member

dovg commented Nov 23, 2011

Может вольем? Тесты лишними не будут.

@AlexeyDsov
Copy link
Member

Да, конечно давайте. Надо тока тому кто вольет не забыть поправить doc/Changelog. Еще на выходных хотел сделать, но не добрался.

AlexeyDsov added a commit that referenced this pull request Nov 29, 2011
+ add simple view render test
@AlexeyDsov AlexeyDsov merged commit b221309 into onPHP:master Nov 29, 2011
AlexeyDsov added a commit that referenced this pull request Nov 29, 2011
@dovg
Copy link
Member

dovg commented Nov 29, 2011

А у вас этот тест проходит?

Я со стандартными настройками получаю F, т.к. toString('testPartView')?> внутри testViewToString не отрабатывает.

ЧЯДНТ?

@dovg
Copy link
Member

dovg commented Nov 29, 2011

Извините, был напуган. в cli shor_open_tags должно быть включено. Все работает.

dovg added a commit to dovg/onphp-framework that referenced this pull request Dec 2, 2011
* onphp-master:
  update doc/ChangeLog for onPHP#46
  * view test tune
  * view test initialization fix
  + add simple view render test

Conflicts:
	doc/ChangeLog
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 this pull request may close these issues.

3 participants