ftp test is not useful and should be removed

Bug #854222 reported by Daniel Manrique on 2011-09-19
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Medium
Brendan Donegan
checkbox (Ubuntu)
Undecided
Unassigned

Bug Description

Ubuntu Oneiric and Checkbox 0.12.7 (and also present in trunk).

The scripts/network_check script is used by the following jobs:
networking/wired_connection
networking/modem_connection
wireless/wireless_connection
external_usb_modem

network_check verifies ability to connect via http and ftp to ubuntu.com, and shows a dialog box with results for both protocols, such as:

HTTP: Success
FTP: Failure

The FTP portion of the test should be removed, since it confuses users (in the above example, was the test a success or a failure?), is more likely to be blocked by firewalls (which can result in a false negative) and is falling into disuse. Testing HTTP connectivity should be enough for the purposes of this test. The test asks the user for manual confirmation of results, which for now suffices, but should be noted as awkward.

Eventually, the best solution would be to replace it by some more automated and less intrusive way of testing high-level (Application layer) protocols (we already have reliable testing of lower layers via the internet_test script).

Related branches

Daniel Manrique (roadmr) on 2011-09-19
Changed in checkbox:
status: New → Triaged
importance: Undecided → Medium
milestone: none → 0.12.8
assignee: nobody → Canonical Hardware Certification (canonical-hw-cert)

I'll remove the part of the network_check script that concerns itself with FTP.

Changed in checkbox:
assignee: Canonical Hardware Certification (canonical-hw-cert) → Brendan Donegan (brendan-donegan)
status: Triaged → In Progress
Daniel Manrique (roadmr) on 2011-09-20
Changed in checkbox:
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package checkbox - 0.12.8

---------------
checkbox (0.12.8) oneiric; urgency=low

  New upstream release (LP: #862579):

  [Brendan Donegan]
  * Remove test for FTP connection from network_check script (LP: #854222)
  * Update a parameter in usb_test to have it run faster.
  * Remove record_playback_after_suspend from Ubuntu Friendly whitelist (LP: #855540)
  * Fix minor typo in multi-monitor friendly resolution_test script which caused
    minimum_resolution test to fail (LP: #855599)
  * Remove storage_devices_test from Ubuntu Friendly whitelist since bonnie++ (which it uses) is not installed by default (LP: #855841)
  * Changed description and name to reflect Ubuntu Friendly branding. Now when a user searches for Ubuntu Friendly in the lens, Checkbox will appear (LP: #852036)
  * Reset the selections at the test suite prompt if No is selected at the recover prompt (LP: #861208)
  * Save the connection name(s) instead of the interface name so that they can be reconnected to properly after the wireless before/after suspend tests have completed (LP: #861502)
  * Make connect_wireless use the UUID of the connection instead of the name for greater reliability (LP: #862190)

  [Daniel Manrique]
  * Restored _recover attribute, re-enabling welcome and test selection
    screens (LP: #852204)
  * Remove memory/test from the Ubuntu Friendly whitelist (LP: #853799)
  * Use diff instead of grep, better comparing of empty files (LP: #852014)
  * Apport integration: new mandatory "tag" value in ApportOptions (LP: #852201)
  * Add warning prior to starting the tests (LP: #855328)
  * Apport integration: Fix instantiation of Gtk.RadioButton, needed due
    to PyGI related API changes (LP: #805679)
  * Remove ping -R parameter that apparently caused ICMP packets to be dropped
    by some routers (LP: #861404)

  [ Evan Broder ]
  * Replace resolution_test with an implementation which uses GdkScreen to
    be multimonitor-aware (LP: #632987)

  [Jeff Lane]
  * Fix names of optical drive tests and remove a non-existing test from the
    whitelist (LP: #854808)
  * Fix wireless_*_suspend jobs so they recreate iface file instead of append
    each time (LP: #855845)
    (LP: #852201)
  * Clarify better the intend of the is_laptop question (LP: #861844)
  * Fixed dependencies for tests that depend on suspend/suspend_advanced
    (LP: #860651)

  [Tim Chen]
  * Fix cpu_scaling_test (LP: #811177)

  [Ara Pulido]
  * Avoid connect_wireless messing with AP with similar names (LP: #861538)
  * Remove bluetooth/file-transfer from the list of tests to run, since due to
    bug 834348 it always fails.

  [Marc Tardif]
  * Added support for wildcards when verifying the transport certificate.
  * Applying depends across suites (LP: #861218)
 -- Daniel Manrique <email address hidden> Thu, 29 Sep 2011 13:12:01 -0400

Changed in checkbox (Ubuntu):
status: New → Fix Released
Ara Pulido (ara) on 2011-09-30
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers