Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

feat: bump WDIO version to 8 #334

Merged
merged 6 commits into from
Aug 9, 2023
Merged

feat: bump WDIO version to 8 #334

merged 6 commits into from
Aug 9, 2023

Conversation

FelixZilber
Copy link
Collaborator

BREAKING CHANGE: bumping new WDIO version to 8

Brief Summary of Changes

Reviewer, Please Note:

Testing (what was tested)

@FelixZilber FelixZilber marked this pull request as ready for review August 8, 2023 07:25
@rashelle-cld
Copy link

:)

process.env.TEST_DATA_TAG = 'test-user';
await Reporter.step('getData reads from file');
const data: ITestData = await TestUtils.getData();
assert.equal(data.fileName, 'example');
});

it('Check incorrect user', async () => {
it.skip('Check incorrect user', async () => {
process.env.TEST_DATA_TAG = 'incorrect-user';
await Reporter.step('undefined returned in case of incorrect data tag');
const data: ITestData = await TestUtils.getData();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why r those skipped? :O

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

since wdio.conf is typed now, we can't add things like
configDataFilePath: 'src/test/resources/example.json',
Since I didn't find any usage of it, I skipped the tests. Probably better to remove them

@FelixZilber FelixZilber merged commit 846a302 into master Aug 9, 2023
1 check passed
@FelixZilber FelixZilber deleted the v7-to-v8-upgrade branch August 9, 2023 09:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants