HTTP FILE SERVER solution contains unnecessary/unexplained line

[piannaf]

The solution includes the line

res.writeHead(200, {'content-type': 'text/plain'})

Neither the instructions nor the hints indicate the student should write this. Verification also doesn't test for this so it passes without.

Either:

1. the solution should not have this line; or
2. The instructions should make it necessary, the hints should mention writeHead, and verification should check the status code and headers

I err on the side of removing the line because this exercise isn't teaching HTTP.

[piannaf]

Just noticed a TODO to add verification for this

[martinheidegger]

@piannaf PR welcome :)

[jpdamon]

@martinheidegger I added the necessary changes to address this issue, however I still need some guidance for doing the verification. I notice that the test will still pass even if the fail event is emitted. Is there something else I need to be doing?