Allow to add feeds from the local file system

Bug #1248737 reported by Leo Arias on 2013-11-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Shorts App
High
Unassigned

Bug Description

Currently, autopilot tests add a feed from www.canonical.com. That's bad because it makes the test depend on external resources that we don't control, and it also make the tests slow because they have to access the internet.

It would be great if we could add also a feed from the local file system, so in the Add feeds screen, instead of entering http://www.canonical.com/rss.xml we would enter file:///tmp/rss.xml

This will simplify the tests a lot. There might be tools to resolve http:// and file:// addresses, so this might not be hard to do implement, but I'm not sure about that.

And, there might be people keeping their rss feeds in the local system too. I suppose this is not common, but I'm sure this won't be just a testability feature, some people will find it useful.

Related branches

Leo Arias (elopio) on 2013-11-06
tags: added: elopio
Nicholas Skaggs (nskaggs) wrote :

This is a key piece to getting the rss reader tests in good shape -- running reliably on the test system

Changed in ubuntu-rssreader-app:
importance: Undecided → High

I am RSS Reader dev. Currently we use Google Feed API for parsing feeds.
This service requires only url to feed, parses it itself and so on.
Seems that there is no way for adding feeds locally.

BR, Roman.

2013/11/7 Nicholas Skaggs <email address hidden>

> This is a key piece to getting the rss reader tests in good shape --
> running reliably on the test system
>
> ** Changed in: ubuntu-rssreader-app
> Importance: Undecided => High
>
> --
> You received this bug notification because you are a member of Ubuntu
> RSS Feed Reader Developers, which is subscribed to Ubuntu RSS Feed
> Reader App.
> Matching subscriptions: Ubuntu RSS Reader Bugmail
> https://bugs.launchpad.net/bugs/1248737
>
> Title:
> Allow to add feeds from the local file system
>
> Status in RSS Feed Reader application for Ubuntu devices:
> Confirmed
>
> Bug description:
> Currently, autopilot tests add a feed from www.canonical.com. That's
> bad because it makes the test depend on external resources that we
> don't control, and it also make the tests slow because they have to
> access the internet.
>
> It would be great if we could add also a feed from the local file
> system, so in the Add feeds screen, instead of entering
> http://www.canonical.com/rss.xml we would enter file:///tmp/rss.xml
>
> This will simplify the tests a lot. There might be tools to resolve
> http:// and file:// addresses, so this might not be hard to do
> implement, but I'm not sure about that.
>
> And, there might be people keeping their rss feeds in the local system
> too. I suppose this is not common, but I'm sure this won't be just a
> testability feature, some people will find it useful.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu-rssreader-app/+bug/1248737/+subscriptions
>

Leo Arias (elopio) wrote :

If we can't use a local rss file, then I think the options that are left
are:
-mock the google api
-or set up a fake http server.

Joey Chan (qqworini) wrote :

It is hard to make such a decision, to add some fake functions just for testing.

According to this bug's title, add feed from local file system may become a new feature sooner or later, but it depends on the Canonical design team.

So I'm afraid this bug should be changed to "wishlist"

Roman Shchekin (mrqtros) wrote :

Agree with Joey.
For tests you can make your own U1DB with one feed inside in format of JSON
object.
But app concept will be broken, we assume that in our "add feed" we are
always search in web.

2013/11/7 Joey Chan <email address hidden>

> It is hard to make such a decision, to add some fake functions just for
> testing.
>
> According to this bug's title, add feed from local file system may
> become a new feature sooner or later, but it depends on the Canonical
> design team.
>
> So I'm afraid this bug should be changed to "wishlist"
>
> --
> You received this bug notification because you are a member of Ubuntu
> RSS Feed Reader Developers, which is subscribed to Ubuntu RSS Feed
> Reader App.
> Matching subscriptions: Ubuntu RSS Reader Bugmail
> https://bugs.launchpad.net/bugs/1248737
>
> Title:
> Allow to add feeds from the local file system
>
> Status in RSS Feed Reader application for Ubuntu devices:
> Confirmed
>
> Bug description:
> Currently, autopilot tests add a feed from www.canonical.com. That's
> bad because it makes the test depend on external resources that we
> don't control, and it also make the tests slow because they have to
> access the internet.
>
> It would be great if we could add also a feed from the local file
> system, so in the Add feeds screen, instead of entering
> http://www.canonical.com/rss.xml we would enter file:///tmp/rss.xml
>
> This will simplify the tests a lot. There might be tools to resolve
> http:// and file:// addresses, so this might not be hard to do
> implement, but I'm not sure about that.
>
> And, there might be people keeping their rss feeds in the local system
> too. I suppose this is not common, but I'm sure this won't be just a
> testability feature, some people will find it useful.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/ubuntu-rssreader-app/+bug/1248737/+subscriptions
>

Nicholas Skaggs (nskaggs) wrote :

Hmm, that does present a problem. You could get creative with hosts file routing..

Roman Shchekin (mrqtros) wrote :

Yes, testers should do this trick :)))

08.11.13 0:52 Nicholas Skaggs написал(а):

Hmm, that does present a problem. You could get creative with hosts file
routing..

--

You received this bug notification because you are a member of Ubuntu
RSS Feed Reader Developers, which is subscribed to Ubuntu RSS Feed
Reader App.
Matching subscriptions: Ubuntu RSS Reader Bugmail

https://bugs.launchpad.net/bugs/1248737

Title:
Allow to add feeds from the local file system

Status in RSS Feed Reader application for Ubuntu devices:
Confirmed

Bug description:
Currently, autopilot tests add a feed from www.canonical.com. That's
bad because it makes the test depend on external resources that we
don't control, and it also make the tests slow because they have to
access the internet.

It would be great if we could add also a feed from the local file
system, so in the Add feeds screen, instead of entering
https://bugs.launchpad.net/bugs/1248737 we would enter https://bugs.launchpad.net/bugs/1248737

This will simplify the tests a lot. There might be tools to resolve
http:// and file:// addresses, so this might not be hard to do
implement, but I'm not sure about that.

And, there might be people keeping their rss feeds in the local system
too. I suppose this is not common, but I'm sure this won't be just a
testability feature, some people will find it useful.

To manage notifications about this bug go to:

https://bugs.launchpad.net/bugs/1248737

Fix committed into lp:ubuntu-rssreader-app at revision 417, scheduled for release in ubuntu-rssreader-app, milestone Unknown

Changed in ubuntu-rssreader-app:
status: Confirmed → Fix Committed
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers