search

azat-co / expressworks

Learn Express.js from the author of one of the best books on Express.js—Pro Express.js— with this workshop that will teach you basics of Express.js.
MIT License
715 stars 220 forks source link

Jade problem checks against solution for hello world... #8

Closed philipkobernik closed 10 years ago

philipkobernik commented 10 years ago 
$ expressworks verify jade.js 
Verifying "JADE"...

ACTUAL                             EXPECTED
------                             --------
"<h1>Hello World</h1><p>Today is Wed Dec 11 2013.</p>" != "Hello World!"                 
# FAIL

Your solution to JADE didn't match the expected output.
Try again!

thanks for putting these lessons together!

sgoodwin commented 10 years ago

This happens to me too. What gives? I don't see anything crazy going on in the code for that problem.

philipkobernik commented 10 years ago

My thought was that when the solution calls res.render('index' ... )

Perhaps it is somehow erroneously loading the 'index' template for the hello_world exercise. Just a hunch.

aaroncm commented 10 years ago

Same happened to me; looks like this is the same issue as #7. The reference process from 'hello world' is still hanging around, so it sends back the wrong data to check against.

azat-co commented 10 years ago

this might be related to https://github.com/nodeschool/discussions/issues/214#issuecomment-35823677

aaroncm commented 10 years ago

Speaking only for myself, when this issue occurred for me it was definitely not that I had selected the wrong scenario.

For whatever reason, the solution process for exercise 1 was still running after I had moved on to exercise 2, so when I tried to verify ex 2, workshopper silently failed to start the solution process for ex 2, and then instead checked my script against the result for exercise 1. Once I killed the leftover node process, it worked properly without having to re-select the exercise.

azat-co commented 10 years ago

@aaroncm so killall node? sounds good to me ;-)