Skip to content

Commit 726888e

Browse files
szaimenszaimen
committed
fix tests
Signed-off-by: szaimen <szaimen@e.mail.de>
1 parent f78af16 commit 726888e

File tree

2 files changed

+42
-18
lines changed

2 files changed

+42
-18
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: 36 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -225,7 +225,8 @@ describe('OC.SetupChecks tests', function() {
225225
JSON.stringify({
226226
hasFileinfoInstalled: true,
227227
isGetenvServerWorking: true,
228-
isReadOnlyConfig: false,
228+
isReadOnlyConfig: true,
229+
wasEmailTestSuccessful: false,
229230
hasWorkingFileLocking: true,
230231
hasValidTransactionIsolationLevel: true,
231232
suggestedOverwriteCliURL: '',
@@ -282,7 +283,8 @@ describe('OC.SetupChecks tests', function() {
282283
JSON.stringify({
283284
hasFileinfoInstalled: true,
284285
isGetenvServerWorking: true,
285-
isReadOnlyConfig: false,
286+
isReadOnlyConfig: true,
287+
wasEmailTestSuccessful: false,
286288
hasWorkingFileLocking: true,
287289
hasValidTransactionIsolationLevel: true,
288290
suggestedOverwriteCliURL: '',
@@ -340,7 +342,8 @@ describe('OC.SetupChecks tests', function() {
340342
JSON.stringify({
341343
hasFileinfoInstalled: true,
342344
isGetenvServerWorking: true,
343-
isReadOnlyConfig: false,
345+
isReadOnlyConfig: true,
346+
wasEmailTestSuccessful: false,
344347
hasWorkingFileLocking: true,
345348
hasValidTransactionIsolationLevel: true,
346349
suggestedOverwriteCliURL: '',
@@ -395,7 +398,8 @@ describe('OC.SetupChecks tests', function() {
395398
JSON.stringify({
396399
hasFileinfoInstalled: true,
397400
isGetenvServerWorking: true,
398-
isReadOnlyConfig: false,
401+
isReadOnlyConfig: true,
402+
wasEmailTestSuccessful: false,
399403
hasWorkingFileLocking: true,
400404
hasValidTransactionIsolationLevel: true,
401405
suggestedOverwriteCliURL: '',
@@ -449,7 +453,8 @@ describe('OC.SetupChecks tests', function() {
449453
JSON.stringify({
450454
hasFileinfoInstalled: true,
451455
isGetenvServerWorking: true,
452-
isReadOnlyConfig: false,
456+
isReadOnlyConfig: true,
457+
wasEmailTestSuccessful: false,
453458
hasWorkingFileLocking: true,
454459
hasValidTransactionIsolationLevel: true,
455460
suggestedOverwriteCliURL: '',
@@ -503,7 +508,8 @@ describe('OC.SetupChecks tests', function() {
503508
JSON.stringify({
504509
hasFileinfoInstalled: true,
505510
isGetenvServerWorking: true,
506-
isReadOnlyConfig: false,
511+
isReadOnlyConfig: true,
512+
wasEmailTestSuccessful: false,
507513
hasWorkingFileLocking: true,
508514
hasValidTransactionIsolationLevel: true,
509515
suggestedOverwriteCliURL: '',
@@ -559,7 +565,8 @@ describe('OC.SetupChecks tests', function() {
559565
JSON.stringify({
560566
hasFileinfoInstalled: true,
561567
isGetenvServerWorking: true,
562-
isReadOnlyConfig: false,
568+
isReadOnlyConfig: true,
569+
wasEmailTestSuccessful: false,
563570
hasWorkingFileLocking: true,
564571
hasValidTransactionIsolationLevel: true,
565572
suggestedOverwriteCliURL: '',
@@ -613,7 +620,8 @@ describe('OC.SetupChecks tests', function() {
613620
JSON.stringify({
614621
hasFileinfoInstalled: true,
615622
isGetenvServerWorking: true,
616-
isReadOnlyConfig: false,
623+
isReadOnlyConfig: true,
624+
wasEmailTestSuccessful: false,
617625
hasWorkingFileLocking: true,
618626
hasValidTransactionIsolationLevel: true,
619627
suggestedOverwriteCliURL: '',
@@ -667,7 +675,8 @@ describe('OC.SetupChecks tests', function() {
667675
JSON.stringify({
668676
hasFileinfoInstalled: true,
669677
isGetenvServerWorking: true,
670-
isReadOnlyConfig: false,
678+
isReadOnlyConfig: true,
679+
wasEmailTestSuccessful: false,
671680
hasWorkingFileLocking: true,
672681
hasValidTransactionIsolationLevel: true,
673682
suggestedOverwriteCliURL: '',
@@ -741,7 +750,8 @@ describe('OC.SetupChecks tests', function() {
741750
JSON.stringify({
742751
hasFileinfoInstalled: true,
743752
isGetenvServerWorking: true,
744-
isReadOnlyConfig: false,
753+
isReadOnlyConfig: true,
754+
wasEmailTestSuccessful: false,
745755
hasWorkingFileLocking: true,
746756
hasValidTransactionIsolationLevel: true,
747757
suggestedOverwriteCliURL: '',
@@ -796,7 +806,8 @@ describe('OC.SetupChecks tests', function() {
796806
JSON.stringify({
797807
hasFileinfoInstalled: true,
798808
isGetenvServerWorking: true,
799-
isReadOnlyConfig: false,
809+
isReadOnlyConfig: true,
810+
wasEmailTestSuccessful: false,
800811
hasWorkingFileLocking: true,
801812
hasValidTransactionIsolationLevel: true,
802813
suggestedOverwriteCliURL: '',
@@ -851,7 +862,8 @@ describe('OC.SetupChecks tests', function() {
851862
JSON.stringify({
852863
hasFileinfoInstalled: true,
853864
isGetenvServerWorking: true,
854-
isReadOnlyConfig: false,
865+
isReadOnlyConfig: true,
866+
wasEmailTestSuccessful: false,
855867
hasWorkingFileLocking: true,
856868
hasValidTransactionIsolationLevel: true,
857869
suggestedOverwriteCliURL: '',
@@ -906,7 +918,8 @@ describe('OC.SetupChecks tests', function() {
906918
JSON.stringify({
907919
hasFileinfoInstalled: true,
908920
isGetenvServerWorking: true,
909-
isReadOnlyConfig: false,
921+
isReadOnlyConfig: true,
922+
wasEmailTestSuccessful: false,
910923
hasWorkingFileLocking: true,
911924
hasValidTransactionIsolationLevel: true,
912925
suggestedOverwriteCliURL: '',
@@ -964,7 +977,8 @@ describe('OC.SetupChecks tests', function() {
964977
JSON.stringify({
965978
hasFileinfoInstalled: true,
966979
isGetenvServerWorking: true,
967-
isReadOnlyConfig: false,
980+
isReadOnlyConfig: true,
981+
wasEmailTestSuccessful: false,
968982
hasWorkingFileLocking: true,
969983
hasValidTransactionIsolationLevel: true,
970984
suggestedOverwriteCliURL: '',
@@ -1019,7 +1033,8 @@ describe('OC.SetupChecks tests', function() {
10191033
JSON.stringify({
10201034
hasFileinfoInstalled: true,
10211035
isGetenvServerWorking: true,
1022-
isReadOnlyConfig: false,
1036+
isReadOnlyConfig: true,
1037+
wasEmailTestSuccessful: false,
10231038
hasWorkingFileLocking: true,
10241039
hasValidTransactionIsolationLevel: true,
10251040
suggestedOverwriteCliURL: '',
@@ -1071,7 +1086,8 @@ describe('OC.SetupChecks tests', function() {
10711086
JSON.stringify({
10721087
hasFileinfoInstalled: true,
10731088
isGetenvServerWorking: true,
1074-
isReadOnlyConfig: false,
1089+
isReadOnlyConfig: true,
1090+
wasEmailTestSuccessful: false,
10751091
hasWorkingFileLocking: true,
10761092
hasValidTransactionIsolationLevel: true,
10771093
suggestedOverwriteCliURL: '',
@@ -1125,7 +1141,8 @@ describe('OC.SetupChecks tests', function() {
11251141
JSON.stringify({
11261142
hasFileinfoInstalled: true,
11271143
isGetenvServerWorking: true,
1128-
isReadOnlyConfig: false,
1144+
isReadOnlyConfig: true,
1145+
wasEmailTestSuccessful: false,
11291146
hasWorkingFileLocking: true,
11301147
hasValidTransactionIsolationLevel: true,
11311148
suggestedOverwriteCliURL: '',
@@ -1179,7 +1196,8 @@ describe('OC.SetupChecks tests', function() {
11791196
JSON.stringify({
11801197
hasFileinfoInstalled: true,
11811198
isGetenvServerWorking: true,
1182-
isReadOnlyConfig: false,
1199+
isReadOnlyConfig: true,
1200+
wasEmailTestSuccessful: false,
11831201
hasWorkingFileLocking: true,
11841202
hasValidTransactionIsolationLevel: true,
11851203
suggestedOverwriteCliURL: '',

0 commit comments

Comments
 (0)