Merge pull request #3860 from nextcloud/fix_master_after_3802
Fix unit tests of master
This commit is contained in:
commit
793d7d1bd7
|
@ -156,6 +156,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -188,6 +189,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -221,6 +223,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -252,6 +255,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -281,6 +285,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: false,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -310,6 +315,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -322,6 +328,36 @@ describe('OC.SetupChecks tests', function() {
|
|||
});
|
||||
});
|
||||
|
||||
it('should return an error if set_time_limit is unavailable', function(done) {
|
||||
var async = OC.SetupChecks.checkSetup();
|
||||
|
||||
suite.server.requests[0].respond(
|
||||
200,
|
||||
{
|
||||
'Content-Type': 'application/json',
|
||||
},
|
||||
JSON.stringify({
|
||||
isUrandomAvailable: true,
|
||||
serverHasInternetConnection: true,
|
||||
isMemcacheConfigured: true,
|
||||
forwardedForHeadersWorking: true,
|
||||
reverseProxyDocs: 'https://docs.owncloud.org/foo/bar.html',
|
||||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: false
|
||||
})
|
||||
);
|
||||
|
||||
async.done(function( data, s, x ){
|
||||
expect(data).toEqual([{
|
||||
msg: 'The PHP function "set_time_limit" is not available. This could result in scripts being halted mid-execution, breaking your installation. We strongly recommend enabling this function.',
|
||||
type: OC.SetupChecks.MESSAGE_TYPE_WARNING
|
||||
}]);
|
||||
done();
|
||||
});
|
||||
});
|
||||
|
||||
it('should return an error if the response has no statuscode 200', function(done) {
|
||||
var async = OC.SetupChecks.checkSetup();
|
||||
|
||||
|
@ -360,6 +396,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
isCorrectMemcachedPHPModuleInstalled: true,
|
||||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: true,
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
@ -390,6 +427,7 @@ describe('OC.SetupChecks tests', function() {
|
|||
hasPassedCodeIntegrityCheck: true,
|
||||
isOpcacheProperlySetup: false,
|
||||
phpOpcacheDocumentation: 'https://example.org/link/to/doc',
|
||||
isSettimelimitAvailable: true
|
||||
})
|
||||
);
|
||||
|
||||
|
|
|
@ -339,6 +339,7 @@ class CheckSetupControllerTest extends TestCase {
|
|||
'codeIntegrityCheckerDocumentation' => 'http://doc.owncloud.org/server/go.php?to=admin-code-integrity',
|
||||
'isOpcacheProperlySetup' => false,
|
||||
'phpOpcacheDocumentation' => 'http://doc.owncloud.org/server/go.php?to=admin-php-opcache',
|
||||
'isSettimelimitAvailable' => true,
|
||||
]
|
||||
);
|
||||
$this->assertEquals($expected, $this->checkSetupController->check());
|
||||
|
|
Loading…
Reference in New Issue