Ticket #727 (closed defect: fixed)

Opened 4 years ago

Last modified 3 years ago

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:

Description

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

comment:1 Changed 4 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 4 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 3 years ago by Sewi

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

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.