Uncaught TypeError: Cannot convert object to primitive value in console.js

[jordan-sf]

testcafe-hammerhead/src/client/sandbox/console.js

Line 21 in bbc833f

const line = arrayMap.call(args, String).join(' ');

Are you requesting a feature or reporting a bug?

Bug

What is the current behavior?

On this line, when trying to console.log/.error/etc an array containing a "derived" object, an exception is thrown:

Uncaught TypeError: Cannot convert object to primitive value
    at String (<anonymous>)
    at Array.map (native)

I'm using AngularJS's http service, and with a (failed) http response, the return value of response.header() in the following example code has a header that looks like an Object. However, its .__proto__ property is missing, and so it doesn't have the .toString() method, causing the above error.

$http(options)
.then(function successCallback(response) {
    if (!response) {
        deferred.reject(response);
    } else {
        console.debug('Success call URL', options.url, 'Status:', response.status, 'Data:', response.data, 'Headers:', response.headers(), 'Config:', response.config)
        deferred.resolve(response.data);
    }
}, function errorCallback(response) {
    console.error('Failed call URL', options.url, 'Status:', response.status, 'Error:', response.data, 'Headers:', response.headers(), 'Config:', response.config)
    deferred.reject(response);
});

What is the expected behavior?

The proxied console.* methods are able to work as expected without throwing an exception. For a comparison, instead of the Array.prototype.map.call line in the code below, simply run console.log(args).

How would you reproduce the current behavior (if this is a bug)?

let header1 = {"Date": Date()};
let header2 = {"other_data": "some data"};
let header2.__proto__ = null;
let args = [header1, header2];
Array.prototype.map.call(args, String);

Provide the test code and the tested page URL (if applicable)

Tested page URL:

Test code

Specify your

• operating system: Windows 10
• testcafe-hammerhead version: 14.2.7
• node.js version: 8.11.3

[miherlosev]

Hi @jordan-sf

Thank you for the detailed bug description. We will fix it during the next development sprint.

[lock]

This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests. We recommend you ask TestCafe API, usage and configuration inquiries on StackOverflow.