Skip to content

Acceptance tests#104

Merged
individual-it merged 1 commit into
owncloud:masterfrom
JankariTech:acceptanceTests
May 3, 2019
Merged

Acceptance tests#104
individual-it merged 1 commit into
owncloud:masterfrom
JankariTech:acceptanceTests

Conversation

@bhawanaprasain

Copy link
Copy Markdown
Contributor

When users are created in ownCloud for the first time or when previously created users log into ownCloud this code does automated testing on activities of Firstrunwizard .

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/firstrunwizard.feature Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php
Comment thread tests/acceptance/features/webUIFirstrunwizard/firstrunwizard.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/firstrunwizard.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/firstrunwizard.feature Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
@codecov

codecov Bot commented Apr 25, 2019

Copy link
Copy Markdown

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #104   +/-   ##
=========================================
  Coverage     28.07%   28.07%           
  Complexity       16       16           
=========================================
  Files             9        9           
  Lines           114      114           
=========================================
  Hits             32       32           
  Misses           82       82

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 629c8b9...930a234. Read the comment docs.

@codecov

codecov Bot commented Apr 25, 2019

Copy link
Copy Markdown

Codecov Report

Merging #104 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #104   +/-   ##
=========================================
  Coverage     28.07%   28.07%           
  Complexity       16       16           
=========================================
  Files             9        9           
  Lines           114      114           
=========================================
  Hits             32       32           
  Misses           82       82

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 629c8b9...f4e3570. Read the comment docs.

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/lib/FirstrunwizardPage.php Outdated
Comment thread tests/acceptance/features/lib/FirstrunwizardPage.php
Comment thread tests/acceptance/features/lib/FirstrunwizardPage.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
@skshetry

Copy link
Copy Markdown

Tests are failing as composer.json is missing.

+ make test-acceptance-webui
composer install
Do not run Composer as root/super user! See https://getcomposer.org/root for details
Loading composer repositories with package information
Installing dependencies (including require-dev) from lock file
Package operations: 1 install, 0 updates, 0 removals
  - Installing bamarni/composer-bin-plugin (v1.3.0): Downloading (connecting...)���������������������������Downloading (0%)           ���������������������������Downloading (55%)�����������������Downloading (100%)
Generating autoload files
make: *** No rule to make target 'vendor-bin/behat/composer.json', needed by 'vendor-bin/behat/composer.lock'. Stop.

@bhawanaprasain bhawanaprasain force-pushed the acceptanceTests branch 4 times, most recently from 36bf07f to 1a4dbd8 Compare April 29, 2019 07:42
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/firstrunwizard.feature Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
Comment thread tests/acceptance/features/webUIFirstrunwizard/reset_all.feature Outdated
@bhawanaprasain bhawanaprasain force-pushed the acceptanceTests branch 2 times, most recently from aac5022 to 19c42e1 Compare April 30, 2019 07:13
@bhawanaprasain

Copy link
Copy Markdown
Contributor Author

@phil-davis please review again

@phil-davis phil-davis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I don't see changes yet. Did you forget to push to GitHub?

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
@bhawanaprasain

Copy link
Copy Markdown
Contributor Author

@phil-davis please review again

@dpakach dpakach self-requested a review May 2, 2019 06:08
@dpakach dpakach dismissed their stale review May 2, 2019 06:08

Needs further review

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php
@bhawanaprasain bhawanaprasain force-pushed the acceptanceTests branch 2 times, most recently from e5d0522 to 237587a Compare May 2, 2019 08:09
@individual-it

Copy link
Copy Markdown
Member

@phil-davis @skshetry @dpakach please review again

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
@skshetry

skshetry commented May 2, 2019

Copy link
Copy Markdown

Looks good to me. Just a minor suggestion from me.

Next PRs could have more scenarios and share steps between each scenarios. 👍

Comment thread tests/acceptance/features/bootstrap/WebUIFirstrunwizardContext.php Outdated
@bhawanaprasain

Copy link
Copy Markdown
Contributor Author

@dpakach @skshetry please review again.

Comment thread tests/acceptance/features/bootstrap/bootstrap.php
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants