Skip to content

test app version after app was updated to new patch version#35556

Merged
phil-davis merged 1 commit into
masterfrom
appUpdateTest
Jun 17, 2019
Merged

test app version after app was updated to new patch version#35556
phil-davis merged 1 commit into
masterfrom
appUpdateTest

Conversation

@individual-it

@individual-it individual-it commented Jun 17, 2019

Copy link
Copy Markdown
Member

Description

check if the app version is corrects written to the DB after a new patch version of the app is installed
to test minor versions is currently not possible because the require to run occ upgrade and we only can do that through the testing app, the testing app does not work when upgrade is needed

Related Issue

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Database schema changes (next release will require increase of minor version instead of patch)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

Open tasks:

  • Backport (if applicable set "backport-request" label and remove when the backport was done)

@individual-it

Copy link
Copy Markdown
Member Author

composer failed with The "https://packagist.org/p/provider-2018-10%24236f731e4693655198e7f392896463f2e3f30e21815d65d143a55b367743e750.json" file could not be downloaded (HTTP/1.1 404 Not Found)
restarted CI

@individual-it

Copy link
Copy Markdown
Member Author

backport in #35560

@codecov

codecov Bot commented Jun 17, 2019

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #35556   +/-   ##
=========================================
  Coverage     65.66%   65.66%           
  Complexity    18756    18756           
=========================================
  Files          1222     1222           
  Lines         70872    70872           
  Branches       1289     1289           
=========================================
  Hits          46540    46540           
  Misses        23954    23954           
  Partials        378      378
Flag Coverage Δ Complexity Δ
#javascript 53.7% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 67.03% <ø> (ø) 18756 <ø> (ø) ⬇️

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 3ddb3bc...df14f1c. Read the comment docs.

1 similar comment
@codecov

codecov Bot commented Jun 17, 2019

Copy link
Copy Markdown

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff            @@
##             master   #35556   +/-   ##
=========================================
  Coverage     65.66%   65.66%           
  Complexity    18756    18756           
=========================================
  Files          1222     1222           
  Lines         70872    70872           
  Branches       1289     1289           
=========================================
  Hits          46540    46540           
  Misses        23954    23954           
  Partials        378      378
Flag Coverage Δ Complexity Δ
#javascript 53.7% <ø> (ø) 0 <ø> (ø) ⬇️
#phpunit 67.03% <ø> (ø) 18756 <ø> (ø) ⬇️

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 3ddb3bc...df14f1c. Read the comment docs.

@phil-davis
phil-davis merged commit 93dba6f into master Jun 17, 2019
@delete-merged-branch
delete-merged-branch Bot deleted the appUpdateTest branch June 17, 2019 13:26
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.

App update verification

2 participants