[BUG] [CCI] SerializationError class parameter data should be optional

[timursaurus]

What is the bug?

In lib/Serializer.js, the ndserialize static method of the Serializer class throws a SerializationError with a single message argument when the input data type is not an Array, although the SerializationError class requires two arguments.

What is the expected behavior?

When the Array check fails, two required arguments should be passed to the SerializationError class

[timursaurus]

@nhtruong
Or passing the array to SerializationError would be better?

if (Array.isArray(array) === false) {
      throw new SerializationError('The argument provided is not an array', array);
}

- throw new SerializationError('The argument provided is not an array');
+ throw new SerializationError('The argument provided is not an array', array);

[nhtruong]

Passing the data to the error constructor will give more context to the error so it's preferable. But I don't see the data being used in the Error class other than this.data = data

[timursaurus]

Passing the data to the error constructor will give more context to the error so it's preferable. But I don't see the data being used in the Error class other than this.data = data

Right, changing the PR right now