Skip to content

Feature-test/mchtctn-nuc116-label-component #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

mchtctn
Copy link
Contributor

@mchtctn mchtctn commented May 13, 2019

No description provided.

@mchtctn mchtctn changed the title Feature test/mchtctn-nuc116-label-component Feature-test/mchtctn-nuc116-label-component May 13, 2019
@codecov-io
Copy link

codecov-io commented May 13, 2019

Codecov Report

Merging #67 into integration/v0.3.0 will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                 Coverage Diff                 @@
##           integration/v0.3.0      #67   +/-   ##
===================================================
  Coverage               92.53%   92.53%           
===================================================
  Files                      37       37           
  Lines                     415      415           
  Branches                   82       82           
===================================================
  Hits                      384      384           
  Misses                     26       26           
  Partials                    5        5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0cd881f...4c9d705. Read the comment docs.

osmanertem
osmanertem previously approved these changes May 13, 2019
ercang
ercang previously approved these changes May 14, 2019
before: function(client, done) {
this.currentPage = client.maximizeWindow().page.labelPage();
this.currentPage
.navigate('http://localhost:8080/components/label.html')
Copy link
Contributor

Choose a reason for hiding this comment

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

can we set domain in a shared variable? 'localhost:8080' might change in the future

Copy link
Contributor Author

Choose a reason for hiding this comment

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

domain set in globals.js

@mchtctn mchtctn dismissed stale reviews from ercang and osmanertem via 9b630a9 May 14, 2019 10:45
ercang
ercang previously approved these changes May 14, 2019
osmanertem
osmanertem previously approved these changes May 14, 2019
mylmz10
mylmz10 previously approved these changes May 14, 2019
@mchtctn mchtctn dismissed stale reviews from mylmz10 and osmanertem via b6ae18a May 14, 2019 11:44
mylmz10
mylmz10 previously approved these changes May 14, 2019
ercang
ercang previously approved these changes May 14, 2019
osmanertem
osmanertem previously approved these changes May 14, 2019
@mchtctn mchtctn changed the base branch from integration/v0.0.1 to integration/v0.3.0 May 15, 2019 12:09
@aylino aylino changed the title Feature-test/mchtctn-nuc116-label-component [WIP] Feature-test/mchtctn-nuc116-label-component May 16, 2019
esracaglar
esracaglar previously approved these changes May 16, 2019
@ahmetceylan ahmetceylan dismissed stale reviews from esracaglar, osmanertem, ercang, and mylmz10 via 42b9efb May 17, 2019 09:29
@mchtctn mchtctn changed the title [WIP] Feature-test/mchtctn-nuc116-label-component Feature-test/mchtctn-nuc116-label-component May 20, 2019
@osmanertem osmanertem merged commit ad54bf6 into integration/v0.3.0 May 22, 2019
@mylmz10 mylmz10 deleted the feature-test/mchtctn-nuc116-labelComponent branch June 26, 2019 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

8 participants