Test Results:
? Test Passed: Value == '['WEST']'
? Test Passed: Value == '[]'
? Test Passed: Value == '['NORTH']'
? Expected: '['EAST', 'NORTH']', instead got: '['NORTH', 'EAST']'

.................................... Frusturating

now i passed 1 but failed 1 still.
error:

Expected: 0, instead got: undefined

. i was returning 0
if (array == null || array === 1 ) {
return 0;
}

not sure about the issue

Change return a>b to return b-a and you should be good :). You should always sort an array based on their value and not a comparsion of their values. Otherwise it will act on their Boolean value (true or false), since it's a comparsion operator, and will not sort based on their actual value. See more here.

You are in the method, the variables are not global.
You take the first value of the array "numbers". But this is only possible, when the array is not null and have values (length > 0). Otherwise the access to the array will force an exception.
So you have to check for null and for a length > 0 before! ;-)

Many thanks for your speedy reply which worked - would you mind explaining why I needed to do this please? I was under the impression that variable declaration took place at the top of the class?

Ryan

You should take the first two lines (int max =..., int min = ...) with access to numbers after the check for null and length.

And just a hint but no error: Why you go twice through the numbers? You could add the actual number to "sum" in the first for. ;-)

typeof(null) = "object"

You are doing if typeof(null) == "null" and therefore the code fails. You should directly compare a null and not use a type comparison.

What is your concrete problem/error message?