Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

Fix jasmine wrong test result #100

Conversation

BorisOsipov
Copy link
Contributor

Fix issue
#94

Signed-off-by: Boris Osipov <osipov.boris@gmail.com>
Signed-off-by: Boris Osipov <osipov.boris@gmail.com>
Signed-off-by: Boris Osipov <osipov.boris@gmail.com>
Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

This is excellent! Thank you so much! One tiny issue but after that this is good to go!

const result = results[0]
expect(result('test-case label[name="feature"]').eq(0).attr('value')).to.equal('A passing feature')
expect(result('test-case label[name="story"]').eq(0).attr('value')).to.equal('A passing scenario')
})
})

it.only('should detect passed case', () => {
Copy link
Contributor

Choose a reason for hiding this comment

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

.only

@christian-bromann
Copy link
Contributor

Not sure why the cucumber test is failing but looks like a legit problem

Signed-off-by: Boris Osipov <osipov.boris@gmail.com>
@BorisOsipov
Copy link
Contributor Author

@christian-bromann Thank you for careful review. I fixed "it.only" mistake and failed cucumber test.

Copy link
Contributor

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

👍

@christian-bromann christian-bromann merged commit b880f27 into webdriverio-boneyard:master Mar 7, 2018
@christian-bromann
Copy link
Contributor

wdio-allure-reporter 0.4.0 published 🎉

@BorisOsipov BorisOsipov deleted the Fix_jasmine_test_result branch March 7, 2018 17:46
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.

None yet

2 participants