Bug #11698

Test suite calls undefined save_pcap_file method in " the network device has its default MAC address configured"

Added by intrigeri 2016-08-23 06:20:31 . Updated 2016-11-16 14:06:38 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Test suite
Target version:
Tails_3.0
Start date:
2016-08-23
Due date:
% Done:

100%

Feature Branch:
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Deliverable for:

Description

calling as root: . /usr/local/lib/tails-shell-library/hardware.sh && get_all_ethernet_nics
call returned: [0, "eth0\n", ""]
    Then 1 network interface is enabled                                                             # features/step_definitions/mac_spoofing.rb:75
calling as root: . /usr/local/lib/tails-shell-library/hardware.sh && get_all_ethernet_nics
call returned: [0, "eth0\n", ""]
calling as root: . /usr/local/lib/tails-shell-library/hardware.sh && get_all_ethernet_nics
call returned: [0, "eth0\n", ""]
calling as root: . /usr/local/lib/tails-shell-library/hardware.sh && get_current_mac_of_nic eth0
call returned: [0, "50:54:00:ce:30:6d\n", ""]
    And the network device has its default MAC address configured                                   # features/step_definitions/mac_spoofing.rb:11
      undefined local variable or method `save_pcap_file' for #<Object:0x000000091d2e10> (NameError)
      ./features/step_definitions/mac_spoofing.rb:28:in `/^the network device has (its default|a spoofed) MAC address configured$/'
      features/mac_spoofing.feature:19:in `And the network device has its default MAC address configured'
    And the real MAC address was leaked                                                             # features/step_definitions/mac_spoofing.rb:34
      Scenario failed at time 02:27:30

Perhaps we should instead call:

@sniffer.save_pcap_file

… as we do in other parts of the test suite?

Subtasks

History

#1 Updated by intrigeri 2016-08-28 23:31:07

see commit:814cd3f8d179041505c501584b6c21a7f23c85d9 on feature/stretch

#2 Updated by intrigeri 2016-08-28 23:31:25

#3 Updated by intrigeri 2016-08-30 00:59:36

  • Status changed from Confirmed to In Progress
  • Assignee set to intrigeri
  • Target version set to Tails_2.6
  • Feature Branch set to test/11698-save-pcap-file

This still breaks on feature/stretch, simply because save_pcap_file is not defined anywhere in our code.

#4 Updated by intrigeri 2016-08-30 01:20:06

  • Status changed from In Progress to Confirmed
  • Assignee changed from intrigeri to anonym
  • Priority changed from Normal to Elevated
  • Target version changed from Tails_2.6 to Tails_3.0
  • Feature Branch deleted (test/11698-save-pcap-file)

This prevents me from having reliable results and useful output from the test suite wrt. potential MAC spoofing regressions in feature/stretch => please prioritize this quite high among your other test suite porting tasks.

#5 Updated by anonym 2016-11-15 15:45:04

  • Status changed from Confirmed to In Progress

Applied in changeset commit:3f62b2ec02aed705c31d2c02d74a322dca92b3e4.

#6 Updated by anonym 2016-11-15 16:26:34

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)

Fixed in

  • commit:3f62b2ec02aed705c31d2c02d74a322dca92b3e4
  • commit:c38f05606b94904ccbb662c86932de5f8b113a67 (fixup)

#7 Updated by intrigeri 2016-11-16 14:06:38

  • Status changed from Fix committed to Resolved
  • % Done changed from 0 to 100

(If there’s a reason not to close this ticket, please reopen and clarify.)