Skip to content

Conversation

@vgamula
Copy link

@vgamula vgamula commented Jun 2, 2014

See readme.md in Task1 directory

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

@vgamula
0. Гуд ;)

  1. Код на CoffeeScript, тести на plain JS. Давай в одному стилі вже.
  2. Яка різниця між assert.deepEqual(consoleEmulator.load(input), expected); и expect(consoleEmulator.load(input)).to.be.deep.equal(expected);?

@vgamula
Copy link
Author

vgamula commented Jun 2, 2014

Різниці ніякої, просто в 2 варіанті гарніше читається :)
Тести переписав.
А можна якось змінити поточний пулл реквест? Чи треба створювати новий?

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

Якщо різниці ніякої, чому ж тоді одні тести назвав TDD, а інші BDD?
На друге питання відповідь вже видно на цій сторінці =)

@vgamula
Copy link
Author

vgamula commented Jun 2, 2014

Якщо я правильно зрозумів різницю між TDD i BDD, то в BDD має бути більш інтуїтивний синтаксис.
Мені здається, що
expect(consoleEmulator.load(input)).to.be.deep.equal(expected); читається лешше ніж
assert.deepEqual(consoleEmulator.load(input), expected);
Того так і назвав)

Про друге питання: кнопкою "Edit" я можу тікльки змінити назву issue

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

image
Твій коміт вже тут, в PR, якщо ти про це.

Ні, між TDD і BDD така ж різниця, як між мишою і слоном. В тебе зараз і там, і там — просто unit-тести. TDD(BDD) — це техніка розробки, а не лише тести. Бажано не плутати ці поняття.

@vgamula
Copy link
Author

vgamula commented Jun 2, 2014

Тобто, чи юніт, чи функціональні тести можна писати, використовуючи будь-яку техніку, TDD чи BDD, залежно від ситуації, я правильно зрозумів?

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

Для TDD, так (але TDD через функціональні тести — це жизньболь).
https://github.com/cucumber/cucumber-js — а ось так виглядає інструмент для BDD

@vgamula
Copy link
Author

vgamula commented Jun 2, 2014

Ну все, я спробував Cucumber :)

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

@vgamula Відчув різницю? =)

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

Ще пару бажків є:

Uncaught TypeError: Cannot read property '0' of undefined ConsoleEmulator.js:55
ConsoleEmulator.load ConsoleEmulator.js:55
run main.js:12
onclick (index):38
Uncaught TypeError: Cannot read property 'length' of null ConsoleEmulator.js:48
ConsoleEmulator.load ConsoleEmulator.js:48
run main.js:12
onclick (index):38

@ftrrtf
Copy link
Owner

ftrrtf commented Jun 2, 2014

I навіть бажки з логікою є, але про них не буду писати. Спробуй сам знайти і пофіксити.

@vgamula
Copy link
Author

vgamula commented Jun 2, 2014

@ftrrtf, так, різниця колосальна)

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.

2 participants