Skip to content

[stable20] Run unit tests against oracle #24049

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 19 commits into from
Nov 12, 2020
Merged

[stable20] Run unit tests against oracle #24049

merged 19 commits into from
Nov 12, 2020

Conversation

@nickvergessen
Copy link
Member

@nickvergessen nickvergessen commented Nov 11, 2020 •
edited by MorrisJobke
Loading

Backport of #23882

@nickvergessen nickvergessen added this to the Nextcloud 20.0.2 milestone Nov 11, 2020
@nickvergessen nickvergessen mentioned this pull request Nov 11, 2020
Merged
4 tasks
@MorrisJobke MorrisJobke mentioned this pull request Nov 11, 2020
Closed
@MorrisJobke
Copy link
Member

Psalm is unhappy about those two:

lib/private/Files/Stream/Encryption.php:449:16:error - UndefinedInterfaceMethod: Method OCP\Files\Cache\ICacheEntry::offsetGet does not exist
lib/private/Files/Stream/Encryption.php:501:59:error - InvalidScalarArgument: Argument 2 of OCP\Encryption\IEncryptionModule::decrypt expects string, int|non-empty-lowercase-string provided

@nickvergessen
Copy link
Member Author

What? I updated the baseline after running it locally....

@rullzer rullzer mentioned this pull request Nov 12, 2020
Merged
juliusknorr and others added 16 commits November 12, 2020 12:57
@juliusknorr @nickvergessen
Add github action for oci8
a35b095 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@nickvergessen
Use a different column for the primary key as we can not insert it on…
f58ffad 
… oracle

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Easier debugging and spell fix
3ec48cd 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Skip the insertIfNotExists() tests on Oracle because it doesn't work …
e3b3ff6 
…with clob

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Make the test pass on repeating calls
48c2f6d 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix "Invalid fetch style: 12" on Oracle
fcef3c0 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Make sure columns with an empty default are nullable for Oracle
8ff0523 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix public calendars as they are stored with null on oracle
b6ce689 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Empty string is returned as null, but empty string in file cache is t…
97b0402 
…he root and exists

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix comparing the empty string for global credentials
a66591e 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Don't try to update on NotNullConstraintViolationException, only on u…
dee4202 
…nique or foreign key

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Replace the credentials table with one that can have empty user
f3c183b 
Primary key columns on Oracle can not have empty strings

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Create primary keys on all tables and add a command to create the aft…
bb52911 
…erwards

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix CS
cd76043 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix naming of jobs and steps
07c28f1 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Fix unique key in test table
dabed84 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
Update baseline, I'm sorry
d09a220 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen nickvergessen force-pushed the backport/23882/stable20 branch from fdfccbf to d09a220 Compare November 12, 2020 12:00
icewind1991
icewind1991 requested changes Nov 12, 2020
View reviewed changes
Copy link
Member

@icewind1991 icewind1991 left a comment

Choose a reason for hiding this comment

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

requires #24072 to be backported to

@icewind1991 @rullzer
fix migration of oc_credentials table
388b310 
Signed-off-by: Robin Appelman <robin@icewind.nl>
@rullzer
Copy link
Member

rullzer commented Nov 12, 2020

requires #24072 to be backported to

cherry-picked on this branch

@icewind1991 icewind1991 self-requested a review November 12, 2020 15:22
@faily-bot
Copy link

faily-bot bot commented Nov 12, 2020

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 35441: failure

mariadb10.1-php7.2

Show full log 
There were 2 warnings:

1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

--

There was 1 failure:

1) Test\SystemTag\SystemTagManagerTest::testGetExistingTag with data set #1 ('one', true, false)
Failed asserting that two strings are equal.
--- Expected
+++ Actual
@@ @@
-'0'
+'32'

/drone/src/tests/lib/SystemTag/SystemTagManagerTest.php:535
/drone/src/tests/lib/SystemTag/SystemTagManagerTest.php:270

@MorrisJobke MorrisJobke mentioned this pull request Nov 12, 2020
Merged
@MorrisJobke
Update psalm-baseline
9497dca 
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke MorrisJobke merged commit 2e3637f into stable20 Nov 12, 2020
@MorrisJobke MorrisJobke deleted the backport/23882/stable20 branch November 12, 2020 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rullzer rullzer rullzer approved these changes

@MorrisJobke MorrisJobke MorrisJobke approved these changes

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@blizzz blizzz Awaiting requested review from blizzz

@juliusknorr juliusknorr Awaiting requested review from juliusknorr

@icewind1991 icewind1991 Awaiting requested review from icewind1991

Assignees

No one assigned

Labels

3. to review Waiting for reviews bug feature: install and update

Projects

None yet

Milestone

Nextcloud 20.0.2

Development

Successfully merging this pull request may close these issues.

6 participants

@nickvergessen @MorrisJobke @rullzer @icewind1991 @juliusknorr