Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Short arch structure serialization to make tests simpler #29

Open
afelix opened this issue Apr 17, 2012 · 4 comments
Open

Short arch structure serialization to make tests simpler #29

afelix opened this issue Apr 17, 2012 · 4 comments

Comments

@afelix
Copy link
Contributor

afelix commented Apr 17, 2012

Чтобы в тестах не писать полотна проверок hasChildren(), hasParents() и hasNode(), удобнее иметь строковый микроформат дампа структуры arch, результат которого уже и сверять с эталоном.
Что-то вроде (лишь для иллюстрации идеи) такого: "A"["B", "C"].

@ghost ghost assigned afelix Apr 17, 2012
@arikon
Copy link
Contributor

arikon commented Apr 18, 2012

Лучше объектную структуру и assert helper

17.04.2012, 17:03, "Sergey Kryzhanovsky" [email protected]:

Чтобы в тестах не писать полотна проверок hasChildren(), hasParents() и hasNode(), удобнее иметь строковый микроформат дампа структуры arch, результат которого уже и сверять с эталоном.
Что-то вроде (лишь для иллюстрации идеи) такого: "A"["B", "C"].


Reply to this email directly or view it on GitHub:
#29

Отправлено из мобильной Яндекс.Почты: http://m.ya.ru/ymail

@afelix
Copy link
Contributor Author

afelix commented Apr 18, 2012

Кажется, если объектная структура, то получится тот же arch, только без "тела" узлов.

@arikon
Copy link
Contributor

arikon commented Apr 18, 2012

Да.

Но конструировать объекты намного проще и удобней, чем строки.

@afelix
Copy link
Contributor Author

afelix commented Apr 19, 2012

Приведи, пожалуйста, пример такого объекта для графа A>B,A>C,B>D,C>D (ромбик). Я пока не понимаю, что может быть проще и лаконичнее такой записи (а эта сериализация в числе прочего должна и решить проблему размера проверяющих тестов).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants