Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Merge branch 'MDL-40207-23' of git://github.com/danpoltawski/moodle i…

…nto MOODLE_23_STABLE
  • Loading branch information...
commit 319b3293676e9fd4ab4f2cf0f7ff9598d9db3f8c 2 parents 3ad26c8 + 5d44714
@marinaglancy marinaglancy authored
Showing with 14 additions and 4 deletions.
  1. +14 −4 lib/tests/rsslib_test.php
View
18 lib/tests/rsslib_test.php
@@ -43,14 +43,18 @@ class moodlesimplepie_testcase extends basic_testcase {
const INVALIDURL = 'http://download.moodle.org/unittest/rsstest-which-doesnt-exist.xml';
# This tinyurl redirects to th rsstest.xml file
const REDIRECTURL = 'http://tinyurl.com/lvyslv';
+ # The number of seconds tests should wait for the server to respond (high to prevent false positives).
+ const TIMEOUT = 10;
function setUp() {
moodle_simplepie::reset_cache();
}
function test_getfeed() {
- $feed = new moodle_simplepie(self::VALIDURL);
-
+ $feed = new moodle_simplepie();
+ $feed->set_timeout(self::TIMEOUT);
+ $feed->set_feed_url(self::VALIDURL);
+ $feed->init();
$this->assertInstanceOf('moodle_simplepie', $feed);
$this->assertNull($feed->error(), "Failed to load the sample RSS file. Please check your proxy settings in Moodle. %s");
@@ -108,7 +112,10 @@ function test_getfeed() {
* Test retrieving a url which doesn't exist
*/
function test_failurl() {
- $feed = @new moodle_simplepie(self::INVALIDURL); // we do not want this in php error log
+ $feed = new moodle_simplepie();
+ $feed->set_timeout(self::TIMEOUT);
+ $feed->set_feed_url(self::INVALIDURL);
+ @$feed->init(); // We do not want this in php error log.
$this->assertNotEmpty($feed->error());
}
@@ -135,7 +142,10 @@ function test_failproxy() {
function test_redirect() {
global $CFG;
- $feed = new moodle_simplepie(self::REDIRECTURL);
+ $feed = new moodle_simplepie();
+ $feed->set_timeout(self::TIMEOUT);
+ $feed->set_feed_url(self::REDIRECTURL);
+ $feed->init();
$this->assertNull($feed->error());
$this->assertEquals($feed->get_title(), 'Moodle News');
Please sign in to comment.
Something went wrong with that request. Please try again.