require('nightwatch-cucumber').client's keys are hidden

[getkey]

I'm submitting a ... (check one with "x")

[x] bug report
[ ] feature request
[ ] support request => Please do not submit support request here, instead use [Stack Overflow](https://stackoverflow.com/questions/tagged/nightwatch.js+cucumber)

Current behavior
My setup is very similar to the demo.
Logging the client object yields these results.

const { client } = require('nightwatch-cucumber');
console.log(client); // {}
console.log(Object.keys(client)); // []
console.log(Object.getOwnPropertyNames(client)); // []
console.log(client.options); // an object containing a bunch of stuff

So the properties are there, they are just invisible when you don't know their names. My guess is that it's caused by a Proxy with an ownKeys trap wrapping the actual client.

Expected/desired behavior
Logging or listing the keys of client should yield the same results as they would on the client that nigtwatch provides. Not doing so is confusing and make implementing nightwatch-cucumber harder for the user.

Reproduction of the problem
See current behavior.

What is the motivation / use case for changing the behavior?
Bugfixing.

Please tell us about your environment:
In case of a bug report please provide the following information:

• output of the following command from your project root npm ls nightwatch cucumber nightwatch-cucumber && node --version
• output of running nightwatch with --verbose flag

yarn list v1.2.1
warning You are using Node "9.2.0" which is not supported and may encounter bugs or unexpected behavior. Yarn supports the following semver range: "^4.8.0 || ^5.7.0 || ^6.2.2 || ^8.0.0"
warning Filtering by arguments is deprecated. Please use the pattern option instead.
├─ cucumber@3.1.0
├─ nightwatch-cucumber@8.2.10
└─ nightwatch@0.9.19
Done in 0.32s.
v9.2.0

[mucsi96]

Thanks for reporting this. I think the issue here that the client is not yet ready after

const { client } = require('nightwatch-cucumber');

statement. Can you retry the consoles within step definitions? The client should be ready at that moment. Basically this problem is the main case I introduced the Proxy.

[getkey]

I just tried inside the callback of the Given(/^I open Google's search page$/, () => {, I got the same result. Are you able to reproduce?

[anatoliyarkhipov]

As I understand, the client proxy should implement ownKeys in order to make Object.getOwnPropertyNames work correctly. Like that, for example:

function getClientProxy (subPages) {
  return new Proxy({}, {
+    ownKeys (target) {
+      return Object.getOwnPropertyNames(runner.nightwatchApi.getClientApi())
+    },
    // ...
  })
}

[mucsi96]

Thanks @anatoliyarkhipov for the tip! Will add this