Skip to content

Commit 0fb7812

Browse files
szaimenszaimen
committed
fix tests
Signed-off-by: szaimen <szaimen@e.mail.de>
1 parent 83856e8 commit 0fb7812

File tree

2 files changed

+24
-0
lines changed

2 files changed

+24
-0
lines changed

‎apps/settings/tests/Controller/CheckSetupControllerTest.php‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -172,6 +172,7 @@ protected function setUp(): void {
172172
])
173173
->setMethods([
174174
'isReadOnlyConfig',
175+
'wasEmailTestSuccessful',
175176
'hasValidTransactionIsolationLevel',
176177
'hasFileinfoInstalled',
177178
'hasWorkingFileLocking',
@@ -496,6 +497,10 @@ public function testCheck() {
496497
->expects($this->once())
497498
->method('isReadOnlyConfig')
498499
->willReturn(false);
500+
$this->checkSetupController
501+
->expects($this->once())
502+
->method('wasEmailTestSuccessful')
503+
->willReturn(false);
499504
$this->checkSetupController
500505
->expects($this->once())
501506
->method('hasValidTransactionIsolationLevel')
@@ -594,6 +599,7 @@ public function testCheck() {
594599
[
595600
'isGetenvServerWorking' => true,
596601
'isReadOnlyConfig' => false,
602+
'wasEmailTestSuccessful' => false,
597603
'hasValidTransactionIsolationLevel' => true,
598604
'hasFileinfoInstalled' => true,
599605
'hasWorkingFileLocking' => true,

‎core/js/tests/specs/setupchecksSpec.js‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -226,6 +226,7 @@ describe('OC.SetupChecks tests', function() {
226226
hasFileinfoInstalled: true,
227227
isGetenvServerWorking: true,
228228
isReadOnlyConfig: false,
229+
wasEmailTestSuccessful: true,
229230
hasWorkingFileLocking: true,
230231
hasValidTransactionIsolationLevel: true,
231232
suggestedOverwriteCliURL: '',
@@ -283,6 +284,7 @@ describe('OC.SetupChecks tests', function() {
283284
hasFileinfoInstalled: true,
284285
isGetenvServerWorking: true,
285286
isReadOnlyConfig: false,
287+
wasEmailTestSuccessful: true,
286288
hasWorkingFileLocking: true,
287289
hasValidTransactionIsolationLevel: true,
288290
suggestedOverwriteCliURL: '',
@@ -341,6 +343,7 @@ describe('OC.SetupChecks tests', function() {
341343
hasFileinfoInstalled: true,
342344
isGetenvServerWorking: true,
343345
isReadOnlyConfig: false,
346+
wasEmailTestSuccessful: true,
344347
hasWorkingFileLocking: true,
345348
hasValidTransactionIsolationLevel: true,
346349
suggestedOverwriteCliURL: '',
@@ -396,6 +399,7 @@ describe('OC.SetupChecks tests', function() {
396399
hasFileinfoInstalled: true,
397400
isGetenvServerWorking: true,
398401
isReadOnlyConfig: false,
402+
wasEmailTestSuccessful: true,
399403
hasWorkingFileLocking: true,
400404
hasValidTransactionIsolationLevel: true,
401405
suggestedOverwriteCliURL: '',
@@ -450,6 +454,7 @@ describe('OC.SetupChecks tests', function() {
450454
hasFileinfoInstalled: true,
451455
isGetenvServerWorking: true,
452456
isReadOnlyConfig: false,
457+
wasEmailTestSuccessful: true,
453458
hasWorkingFileLocking: true,
454459
hasValidTransactionIsolationLevel: true,
455460
suggestedOverwriteCliURL: '',
@@ -504,6 +509,7 @@ describe('OC.SetupChecks tests', function() {
504509
hasFileinfoInstalled: true,
505510
isGetenvServerWorking: true,
506511
isReadOnlyConfig: false,
512+
wasEmailTestSuccessful: true,
507513
hasWorkingFileLocking: true,
508514
hasValidTransactionIsolationLevel: true,
509515
suggestedOverwriteCliURL: '',
@@ -560,6 +566,7 @@ describe('OC.SetupChecks tests', function() {
560566
hasFileinfoInstalled: true,
561567
isGetenvServerWorking: true,
562568
isReadOnlyConfig: false,
569+
wasEmailTestSuccessful: true,
563570
hasWorkingFileLocking: true,
564571
hasValidTransactionIsolationLevel: true,
565572
suggestedOverwriteCliURL: '',
@@ -614,6 +621,7 @@ describe('OC.SetupChecks tests', function() {
614621
hasFileinfoInstalled: true,
615622
isGetenvServerWorking: true,
616623
isReadOnlyConfig: false,
624+
wasEmailTestSuccessful: true,
617625
hasWorkingFileLocking: true,
618626
hasValidTransactionIsolationLevel: true,
619627
suggestedOverwriteCliURL: '',
@@ -668,6 +676,7 @@ describe('OC.SetupChecks tests', function() {
668676
hasFileinfoInstalled: true,
669677
isGetenvServerWorking: true,
670678
isReadOnlyConfig: false,
679+
wasEmailTestSuccessful: true,
671680
hasWorkingFileLocking: true,
672681
hasValidTransactionIsolationLevel: true,
673682
suggestedOverwriteCliURL: '',
@@ -742,6 +751,7 @@ describe('OC.SetupChecks tests', function() {
742751
hasFileinfoInstalled: true,
743752
isGetenvServerWorking: true,
744753
isReadOnlyConfig: false,
754+
wasEmailTestSuccessful: true,
745755
hasWorkingFileLocking: true,
746756
hasValidTransactionIsolationLevel: true,
747757
suggestedOverwriteCliURL: '',
@@ -797,6 +807,7 @@ describe('OC.SetupChecks tests', function() {
797807
hasFileinfoInstalled: true,
798808
isGetenvServerWorking: true,
799809
isReadOnlyConfig: false,
810+
wasEmailTestSuccessful: true,
800811
hasWorkingFileLocking: true,
801812
hasValidTransactionIsolationLevel: true,
802813
suggestedOverwriteCliURL: '',
@@ -852,6 +863,7 @@ describe('OC.SetupChecks tests', function() {
852863
hasFileinfoInstalled: true,
853864
isGetenvServerWorking: true,
854865
isReadOnlyConfig: false,
866+
wasEmailTestSuccessful: true,
855867
hasWorkingFileLocking: true,
856868
hasValidTransactionIsolationLevel: true,
857869
suggestedOverwriteCliURL: '',
@@ -907,6 +919,7 @@ describe('OC.SetupChecks tests', function() {
907919
hasFileinfoInstalled: true,
908920
isGetenvServerWorking: true,
909921
isReadOnlyConfig: false,
922+
wasEmailTestSuccessful: true,
910923
hasWorkingFileLocking: true,
911924
hasValidTransactionIsolationLevel: true,
912925
suggestedOverwriteCliURL: '',
@@ -965,6 +978,7 @@ describe('OC.SetupChecks tests', function() {
965978
hasFileinfoInstalled: true,
966979
isGetenvServerWorking: true,
967980
isReadOnlyConfig: false,
981+
wasEmailTestSuccessful: true,
968982
hasWorkingFileLocking: true,
969983
hasValidTransactionIsolationLevel: true,
970984
suggestedOverwriteCliURL: '',
@@ -1020,6 +1034,7 @@ describe('OC.SetupChecks tests', function() {
10201034
hasFileinfoInstalled: true,
10211035
isGetenvServerWorking: true,
10221036
isReadOnlyConfig: false,
1037+
wasEmailTestSuccessful: true,
10231038
hasWorkingFileLocking: true,
10241039
hasValidTransactionIsolationLevel: true,
10251040
suggestedOverwriteCliURL: '',
@@ -1072,6 +1087,7 @@ describe('OC.SetupChecks tests', function() {
10721087
hasFileinfoInstalled: true,
10731088
isGetenvServerWorking: true,
10741089
isReadOnlyConfig: false,
1090+
wasEmailTestSuccessful: true,
10751091
hasWorkingFileLocking: true,
10761092
hasValidTransactionIsolationLevel: true,
10771093
suggestedOverwriteCliURL: '',
@@ -1126,6 +1142,7 @@ describe('OC.SetupChecks tests', function() {
11261142
hasFileinfoInstalled: true,
11271143
isGetenvServerWorking: true,
11281144
isReadOnlyConfig: false,
1145+
wasEmailTestSuccessful: true,
11291146
hasWorkingFileLocking: true,
11301147
hasValidTransactionIsolationLevel: true,
11311148
suggestedOverwriteCliURL: '',
@@ -1180,6 +1197,7 @@ describe('OC.SetupChecks tests', function() {
11801197
hasFileinfoInstalled: true,
11811198
isGetenvServerWorking: true,
11821199
isReadOnlyConfig: false,
1200+
wasEmailTestSuccessful: true,
11831201
hasWorkingFileLocking: true,
11841202
hasValidTransactionIsolationLevel: true,
11851203
suggestedOverwriteCliURL: '',

0 commit comments

Comments
 (0)