Opened 11 years ago

Closed 11 years ago

#727 closed defect (fixed)

do not run test that requires networking without the permission of the user

Reported by: szabgab Owned by:
Priority: major Milestone:
Component: editor Version: 0.48
Keywords: tests Cc:


In t/92-padre-file.t we have a test that is trying to access the Internet. We should not attempt to do so without asking the permission of the user as we might trigger some alarms.

Change History (3)

comment:1 Changed 11 years ago by szabgab

In r9002 the test were changed to be skipped unless in AUTOMATED testing but this should be probably controlled by a prompt.

comment:2 Changed 11 years ago by Sewi

They are more-or-less optional tests. Developers should be able to use them to test the modules, but if they fail during Padre installation, the installation should still continue (Padre will run without networked file access, maybe some features are missing).

A better place for this test (and some others) would be a developer-test - directory containing tests which should pass on the trunk but shouldn't kill an installation.

comment:3 Changed 11 years ago by Sewi

  • Resolution set to fixed
  • Status changed from new to closed

Network tests require the environment variable PADRE_NETWORK_T to be true for running.
(Changed some time ago.)

Note: See TracTickets for help on using tickets.