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

[stable12] fix postgresql tests #6792

Merged
merged 1 commit into from Oct 9, 2017

Conversation

Projects
None yet
3 participants
@blizzz
Member

blizzz commented Oct 9, 2017

No description provided.

fix postgresql tests
we don't support pgsql 10 yet. because Doctrine does not either.

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

@blizzz blizzz added this to the Nextcloud 12.0.4 milestone Oct 9, 2017

@blizzz blizzz requested review from rullzer and nickvergessen Oct 9, 2017

@blizzz blizzz referenced this pull request Oct 9, 2017

Merged

fix postgresql tests #6790

@rullzer

rullzer approved these changes Oct 9, 2017

@codecov

This comment has been minimized.

Show comment
Hide comment
@codecov

codecov bot Oct 9, 2017

Codecov Report

Merging #6792 into stable12 will decrease coverage by 22.99%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             stable12    #6792      +/-   ##
==============================================
- Coverage       53.77%   30.78%     -23%     
  Complexity      22560    22560              
==============================================
  Files            1383     1383              
  Lines           86591    86591              
  Branches         1328     1328              
==============================================
- Hits            46566    26653   -19913     
- Misses          40025    59938   +19913
Impacted Files Coverage Δ Complexity Δ
apps/files_versions/lib/Command/Expire.php 0% <0%> (-100%) 3% <0%> (ø)
apps/user_ldap/lib/Migration/UUIDFixGroup.php 0% <0%> (-100%) 1% <0%> (ø)
apps/dav/lib/Connector/Sabre/Server.php 0% <0%> (-100%) 1% <0%> (ø)
lib/public/Files/ForbiddenException.php 0% <0%> (-100%) 2% <0%> (ø)
lib/private/Comments/ManagerFactory.php 0% <0%> (-100%) 2% <0%> (ø)
lib/private/SystemTag/ManagerFactory.php 0% <0%> (-100%) 3% <0%> (ø)
lib/private/Files/Mount/LocalHomeMountProvider.php 0% <0%> (-100%) 1% <0%> (ø)
apps/user_ldap/lib/Migration/UUIDFixUser.php 0% <0%> (-100%) 1% <0%> (ø)
apps/files_sharing/lib/External/MountProvider.php 0% <0%> (-100%) 4% <0%> (ø)
lib/private/TagManager.php 0% <0%> (-100%) 4% <0%> (ø)
... and 367 more

codecov bot commented Oct 9, 2017

Codecov Report

Merging #6792 into stable12 will decrease coverage by 22.99%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             stable12    #6792      +/-   ##
==============================================
- Coverage       53.77%   30.78%     -23%     
  Complexity      22560    22560              
==============================================
  Files            1383     1383              
  Lines           86591    86591              
  Branches         1328     1328              
==============================================
- Hits            46566    26653   -19913     
- Misses          40025    59938   +19913
Impacted Files Coverage Δ Complexity Δ
apps/files_versions/lib/Command/Expire.php 0% <0%> (-100%) 3% <0%> (ø)
apps/user_ldap/lib/Migration/UUIDFixGroup.php 0% <0%> (-100%) 1% <0%> (ø)
apps/dav/lib/Connector/Sabre/Server.php 0% <0%> (-100%) 1% <0%> (ø)
lib/public/Files/ForbiddenException.php 0% <0%> (-100%) 2% <0%> (ø)
lib/private/Comments/ManagerFactory.php 0% <0%> (-100%) 2% <0%> (ø)
lib/private/SystemTag/ManagerFactory.php 0% <0%> (-100%) 3% <0%> (ø)
lib/private/Files/Mount/LocalHomeMountProvider.php 0% <0%> (-100%) 1% <0%> (ø)
apps/user_ldap/lib/Migration/UUIDFixUser.php 0% <0%> (-100%) 1% <0%> (ø)
apps/files_sharing/lib/External/MountProvider.php 0% <0%> (-100%) 4% <0%> (ø)
lib/private/TagManager.php 0% <0%> (-100%) 4% <0%> (ø)
... and 367 more

@nickvergessen nickvergessen merged commit 0577336 into stable12 Oct 9, 2017

2 of 3 checks passed

codecov/project 30.78% (-23%) compared to 35641b5
Details
codecov/patch Coverage not affected when comparing 35641b5...be97297
Details
continuous-integration/drone/pr the build was successful
Details

@nickvergessen nickvergessen deleted the 6790-bp-s12 branch Oct 9, 2017

@MorrisJobke MorrisJobke referenced this pull request Nov 20, 2017

Merged

12.0.4 RC #7225

1 of 1 task complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment