My favorites | Sign in
Logo
                
New issue | Search
for
| Advanced search | Search tips
Issue 123: Remove code related to using test data with URLs
  Back to list
Status:  Done
Owner:  pekka.klarck
Closed:  Mar 2009
Type-Enhancement
Target-2.1
Priority-Medium


Sign in to add a comment
 
Reported by laukpe, Sep 30, 2008
At some point we thought it would be a good idea to be able to run tests like 

    pybot http://my.server/tests.html

Even though we haven't "advertised" it, we still have this support. The
biggest reason why it isn't really useful are:

- You can only execute single test case files, not directories. We could,
of course, add support for following links from index.html files but that
wouldn't be trivial.
- You need to download needed libraries anyway. We could, of course, find
libraries that are imported using physical paths, but that would not be
enough as a general case.
- It is easy to create a separate tool for downloading test case files and
everything else needed. This kind of orchestration capabilities are better
kept out of the core framework. 

Supporting test cases files, resource files, and variable files that use
URLs makes many parts of the code really complicated, because we always
need to take into account that the needed file might need to be downloaded
first. Removing this not-really-useful functionality makes the code easier
to maintain and makes further refactoring easier.
Comment 1 by pekka.klarck, Mar 20, 2009
(No comment was entered for this change.)
Labels: -Type-Refactoring Type-Enhancement
Comment 2 by pekka.klarck, Mar 20, 2009
Started this work by removed all acceptance tests in r1444.
Owner: pekka.klarck
Comment 3 by pekka.klarck, Mar 20, 2009
(No comment was entered for this change.)
Status: Started
Comment 4 by pekka.klarck, Mar 20, 2009
Removed all code that used utils.is_url in r1447. There might still be some
supporting code related to this somewhere (like utils.download that I just removed too).
Status: Done
Comment 5 by pekka.klarck, Apr 19, 2009
Set priority to match other 2.1 issues.
Comment 6 by pekka.klarck, Apr 19, 2009
Set priority to match other 2.1 issues. And this time also reset the
priority....
Labels: Priority-Low
Comment 7 by pekka.klarck, Apr 19, 2009
Was supposed to set this to medium earlier....
Labels: -Priority-Low Priority-Medium
Sign in to add a comment

Hosted by Google Code