Хотя я понял тебя.

Скобки?

тебе бы ООП почитать

$this->db->executeOne($sql); По сути здесь же вызов свойства db объекта $this где db в свою очередь является объектом и имеет свой метод executeOne?

ага

Спасибо!

не за что

Вышла ошибочка

$data = file_get_contents($this->apiUri);вот, сделано дешево и сердито

И чтобы замокать такой ответ апи, достаточно на диске создать файл с данными и в apiUri вставить путь к файлу ?

еретик

на костре тебя сожжем

фу, файлы

все время кажется что файлы зло

хотя на самом деле добро

все есть файл

хотя думать надо, можно диск задрочить )

да

все есть файл

а если не файл, то папка. Хотя тоже файл

хотя думать надо, можно диск задрочить )

Диск не задрочишь просто так, там все кешируется в памяти многое.

Просто так нет. Надо с особым цинизмом это делать. чтобы кеша не хватало.

И чтобы замокать такой ответ апи, достаточно на диске создать файл с данными и в apiUri вставить путь к файлу ?

Это не замокать, а подменить. А вот была бы у тебя в классе зависимость от HttpClient или даже SomeApiClient, мог бы реально замокать :)

Застабить :)

Как плюс мока, ты еще и проверяешь, что нужный метод реально был вызван и всего один раз

Где-то читал, что Стаб - это "Заглушка для метода или объекта", так что хз, фикстура какая-нибудь или просто заглушка)

А зачем проверять что он был вызван 1 раз? Пусть хоть 50 раз вызывается, мне все равно.

Ну если у тебя при выполнении одного атомарного действия апи дергается 50 раз - это обычно не к добру)

Не к добру, но в данный момент мне это неважно

Тут очередной легаси код, чтобы его переписать это надо отдельную таску создавать и сидеть перекраивать 5 классов, внедрять их в контейнер и еще 5 часов потратить.

Инъекция зависимостей - это не обязатнльно контейнер, а легаси да, оно такое(

Никому не попадалось расширение для симфоневский форм, позволяющее кастомизировать формы произвольной вложенности?

Вроде collection_row_N_widget

И чтобы замокать такой ответ апи, достаточно на диске создать файл с данными и в apiUri вставить путь к файлу ?

Моя гениальная идея потерпела фиаско. Относительный путь к файлу меняется в завимости от того из какого места мы запускаем тесты и они нестабильны. Шах и мат, любители простоты!

так это

а что больше ничего не инклудится?

ну так все остальное инклудится нормально через __DIR__ и все такое. А у нас конфиг в json и там не подсунешь никаких динамически высчитываемых констант.

json можно обработать на пхп. Некоторые с этой же целью, чтобы путь вычислить, CSS через пхп гонят

Моя гениальная идея потерпела фиаско. Относительный путь к файлу меняется в завимости от того из какого места мы запускаем тесты и они нестабильны. Шах и мат, любители простоты!

Потому что относительный путь считается от текущей рабочий директории, я уже об этом плакался в чате)

*и это абсолютно нормальное поведение

В конфиге можно и абсолютные пути прописать

нет как же ты пропишешь абсолютный путь, если на тестовой тачке он один, а у разработчиков может быть другой

Ну так подразумевается, что и тестовый конфиг будет другой

Пути нужно выносить в конфиги, только если это действительно переменные значения, вроду директории логов, каких-то внешних данных итд. А если это пути к файлам, которые хранятся в том же репозитории, что и код, их можно тупо хардкодить, ну или задавать относительно какого-нибудь APP_ROOT

Ваш кэп (:

> или задавать относительно какого-нибудь APP_ROOT > В конфиге можно и абсолютные пути прописать Я вот на вторую фразу отвечал, а первая в каком-то смысле с ней антогонистична.

Я имел ввиду, что "переменные" пути прописываются как абсоютные, а остальные хардкодятся

> или задавать относительно какого-нибудь APP_ROOT > В конфиге можно и абсолютные пути прописать Я вот на вторую фразу отвечал, а первая в каком-то смысле с ней антогонистична.

А если путь от константы __DIR__ сделать?

Тоже как бы получится в разных исходниках разный путь до файла стаба

Скинь изначальную задачу

$data = file_get_contents($this->apiUri);вот, сделано дешево и сердито

Нужно застабить один ответ для тестов.

Вместо $this->apiUri подставить не урл API а что-то еще, или как-то еще в $data получить тестовые данные.

Вместо $this->apiUri подставить не урл API а что-то еще, или как-то еще в $data получить тестовые данные.

Попробуй сделать на поддомене тестовом этот файлик и обращаться к нему в тестах

И подставлять урл только другой

Урл можно в env запихнуть

И на каждой машине будет свой, на деве и тесте фацлик на проде настоящий

Как люди не извращаются.. Вынеси свой file_get_contents в другой метод и мокай его

Вместо $this->apiUri подставить не урл API а что-то еще, или как-то еще в $data получить тестовые данные.

А еще лучше текстом в $data подставь результат, без внешнего получения и все

Это же для тестов, значит ты знаешь какой результат ожидать

в сторону https://github.com/InterNations/http-mock не смотрел?

в сторону https://github.com/InterNations/http-mock не смотрел?

может из пушки по воробяьм, но у guzzle есть встроенная поддержка стабов для тестирования

может из пушки по воробяьм, но у guzzle есть встроенная поддержка стабов для тестирования

всяко лучше чем на костылях бегать

А еще лучше текстом в $data подставь результат, без внешнего получения и все

Не лучше

http://docs.guzzlephp.org/en/latest/testing.html

ERROR: S client not available

http://docs.guzzlephp.org/en/latest/testing.html

В данном случае разумно было бы мокнуть весь http клиент

Вы же не сетевое взаимодействие институте и не прослойку над ней.

если дальше продолжать мысль про разумность, то можно разделить ответственности, и выделить некий клиент, который предоставляет данные, и мокать уже его, без привязки к транспорту

Так и должно быть, в простейшем случае этим клиентом может быть класс с одним методом и file_get_contets внутри

ага

Так и должно быть, в простейшем случае этим клиентом может быть класс с одним методом и file_get_contets внутри

О, охрененный костыль, заберу к себе.

В чем тут костыль?

это не костыть, это IOC и SRP)

Главное, @todo Воткнуть guzzle не забыть написать)

А по делу, никто не будет ругаться, если это будет не класс, а отдельный метод в том же классе. Это же PHP :)

Паттерны ради паттернов.

с таких фраз и начинаются все войны

Не начинайте.

Тут есть кто?

закрыто на переучёт

Пхп макаки собрались)))

RewriteRule ^test/([%_\-\+a-zA-Z0-9\/\s]+)$

Почему данная регулярка не пропускает %

RewriteRule ^test/([%_\-\+a-zA-Z0-9\/\s]+)$

#^test/([%_\-\+a-zA-Z0-9\/\s]+)$#

#^test/([%_\-\+a-zA-Z0-9\/\s]+)$#

не не работает

preg_match('#^test/([%_\-\+a-zA-Z0-9\/\s]+)$#i', $subject)

RewriteRule ^test/(.*)$ ?page=settoken&token=$1 // test/http%3A%2F%2Fwww.url-encode-decode.com // ?page=settoken&token=http%3A%2F%2Fwww.url-encode-decode.com

1-ое не работает

2 работает

и еще надо что бы пустой $1 тоже читал test/

#^test/([%_\-\+a-zA-Z0-9\/\s]+)$# так вообще не работает test/ Not Found The requested URL /test/123 was not found on this server.

вы хоть немножко читали про regexp? экранирование, модификаторы, ^$ я вам посоветую установите себе небольшую утилитку RegExBuddy и протестируйте все свои регулярки на исходных данных

более того, канал по php, а вы скидываете регулярку RewriteRule

как сделать чтобы эксепшен не выкидывал используя в свиче анонимку?

есть каналы по php, фишки/новости и т.д. ?