Browse Source

Probably test for feed redirection bug

tags/0.8.1
J. King 11 months ago
parent
commit
16530b9a66
3 changed files with 15 additions and 1 deletions
  1. +1
    -1
      CHANGELOG
  2. +8
    -0
      tests/cases/Database/SeriesSubscription.php
  3. +6
    -0
      tests/docroot/Feed/Fetching/RedirectionDuplicate.php

+ 1
- 1
CHANGELOG View File

@@ -3,7 +3,7 @@ Version 0.8.1 (2019-??-??)

Bug fixes:
- Don't crash updating feeds cached solely via ETag
- Don't update feed URLs when fetching
- Don't fail adding a feed which collides with another via redirection

Version 0.8.0 (2019-07-26)
==========================


+ 8
- 0
tests/cases/Database/SeriesSubscription.php View File

@@ -223,6 +223,14 @@ trait SeriesSubscription {
Arsse::$db->subscriptionAdd($this->user, $url);
}

public function testAddADuplicateSubscriptionViaRedirection() {
$url = "http://localhost:8000/Feed/Parsing/Valid";
Arsse::$db->subscriptionAdd($this->user, $url);
$subID = $this->nextID("arsse_subscriptions");
$url = "http://localhost:8000/Feed/Fetching/RedirectionDuplicate";
$this->assertSame($subID, Arsse::$db->subscriptionAdd($this->user, $url));
}

public function testAddASubscriptionWithoutAuthority() {
$url = "http://example.com/feed1";
Phake::when(Arsse::$user)->authorize->thenReturn(false);


+ 6
- 0
tests/docroot/Feed/Fetching/RedirectionDuplicate.php View File

@@ -0,0 +1,6 @@
<?php return [
'code' => 302,
'fields' => [
"Location: http://localhost:8000/Feed/Parsing/Valid",
],
];

Loading…
Cancel
Save