[🐛 Bug]: shared-store-service setValue does not work in onPrepare() as documented

[yunj]

WebdriverIO Version

7.16.0

Node.js Version

16.4.2

Mode

WDIO Testrunner

Which capabilities are you using?

{
    maxInstances: 1,
    browserName: 'chrome',
    'goog:chromeOptions': {
      args: ['--window-size=1200,800'],
    },
}

What happened?

We want to store environment-specific variables into shared store in onPrepare before running all tests and retrieve from each test's worker process, as documented at README.md. But val is always undefined unless we put await setValue('hello', 'world') in before().

await setValue('hello', 'world'); // set in onPrepare()
const val = await getValue('hello'); // get in each test

What is your expected behavior?

After setting values through shared store in onPrepare, all workers should be able to retrieve from the shared store.

How to reproduce the bug.

onPrepare: async function(config, capabilities) {
  await setValue('hello', 'world');
}

Relevant log output

Return undefined when getting the value through a key, await getValue('hello').

Code of Conduct

• I agree to follow this project's Code of Conduct

Is there an existing issue for this?

• I have searched the existing issues

[yunj]

Hi @Slashgear could you please have a look at why setValue in onPrepare() is not working as described in here? Thank you very much in advance.

[christian-bromann]

But val is always undefined unless we put await setValue('hello', 'world') in before().

setValue is an async operation and need to be called with await, as described in the example. I can tweak the docs to explicitly emphasise this.

[yunj]

Hi @christian-bromann thanks for your reply. My code snippet shows await is being used to call setValue() otherwise it didn't work either in before() as described in the request. We want to do in onPrepare() so only execute once and can be accessed by all tests.

[christian-bromann]

After investigation it appeared that this functionality was broken. A pull request was applied with a fix for it. Thanks for bringing this up!