Skip to content

Conversation

KeynihAV
Copy link

No description provided.

Copy link
Member

Choose a reason for hiding this comment

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

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

Copy link
Member

Choose a reason for hiding this comment

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

Другие изменения готов принять.
По НачалоДня предлагаю обсудить.

@KeynihAV
Copy link
Author

В таком случае можно убрать и КонецДня(), чтобы брать произвольный период.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

Наиболее популярный сценарий - указать период в виде двух дат без времени.
В этом случае первая дата сразу указывает на начало дня и это правильно.
А вот для второй даты нужно выполнять приведение к концу дня и для пользователя это неудобно.
Так что фича с преобразованием к концу дня полезна и ее лучше оставить.

…ментовМетаданногоПоОтбору теперь вызывает ПолучитьКоличествоЭлементовМетаданногоПоОтбору.
@KeynihAV
Copy link
Author

Опечатался в коммите. Там удален код из метода ПолучитьКоличествоЭлементовСправочникаПоОтбору.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

А можешь вкратце описать сценарии использования этих методов в своем коде?

@KeynihAV
Copy link
Author

Не доводилось их еще использовать, в рабочей конфигурации я пока всего с десяток тестов написал. Интересно посмотреть как внутри устроено, что за магические слова fork и pull request, заодно и помочь проекту. Поэтому я взял попавшуюся на глаза задачу.

@KeynihAV
Copy link
Author

Еще вижу задачу #297. Получается, что она давно сделана но не закрыта.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

По #297 подразумевалось наличие клиентских методов в управляемой форме раннера, т.к. сейчас из серверного кода теста при запуске в упр.режиме сложно достучаться до серверного кода раннера.

@artbear
Copy link
Member

artbear commented Aug 10, 2015

Интересно посмотреть как внутри устроено, что за магические слова fork и pull request, заодно и помочь проекту. Поэтому я взял попавшуюся на глаза задачу.

Хорошее дело, приветствую.
Спасибо!

@artbear artbear modified the milestones: second, 4.3.0.0, 4.Х.0.0 Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment