Issues with tests for scopes challenge

[arku]

The challenge asks us to create a function foo and lexically scope a variable bar.
My solution was

var foo = function(){
  var bar;
}

However one of the tests fail. I think the above solution is also right and should be accepted.
Correct me if I am wrong.

[lenguyenthanh]

The solution is below:

function foo() {
    var bar
}

[LisaMieth]

@lenguyenthanh
I've been running into the same problem. I tried your solution but I get a syntax error. Same if I try
function foo(){
var bar;
}
Anybody any ideas what I'm doing wrong?

[lenguyenthanh]

@LisaKhadijah what is your syntax error? It looks correct.

[LisaMieth]

@lenguyenthanh
I don't get any specific error. If i run scope-chains-closures verify <myfile.js> all I get is the console message "syntaxerror".

[domfarolino]

@lenguyenthanh shouldn't @arun1595 solution be valid too though?

[NonPolynomial]

@lenguyenthanh the solution of @arun1595 is a valid kind to define a function.
this workshop shouldn't make a difference between function declaration and function expression.

[martinheidegger]

Assigning a function to a variable did not give it a name in Node versions prior to 6:

var foo = function () {}
console.log(foo.name)
console.log((function bar() {}).name)

So that, and I think that the validation is harder were the reasons why this doesn't work. A PR that fixes this would be good imo.