Project

General

Profile

Bug #6806

WhisperBack test suite for log sanitization does not test what it should

Added by intrigeri over 5 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
03/02/2014
Due date:
% Done:

100%

Feature Branch:
whisperback:bugfix/filter_more_serial_nos
Type of work:
Code
Blueprint:
Starter:
No
Affected tool:
WhisperBack

Description

It's testing that [$something REMOVED] can be found after sanitizing, but it does not test that the info that should be removed cannot be found anymore, which is what we should really test.


Related issues

Blocks Tails - Bug #6798: Whisperback should sanitize more serial numbers Resolved 03/01/2014
Blocks Tails - Bug #6797: Whisperback should sanitize DMI Resolved 03/01/2014
Blocks Tails - Bug #6804: Sanitize expanded IPv6 addresses in WhisperBack Resolved 03/02/2014

History

#1 Updated by alant over 5 years ago

  • Status changed from Confirmed to In Progress
  • QA Check set to Ready for QA
  • Feature Branch set to whisperback:bugfix/filter_more_serial_nos

This is now implemented in the feature branch.

#2 Updated by intrigeri over 5 years ago

  • Target version set to Tails_0.23

#3 Updated by intrigeri over 5 years ago

  • Assignee changed from alant to intrigeri

I'll give anonym a hand on that one.

#4 Updated by intrigeri over 5 years ago

  • Assignee changed from intrigeri to alant
  • QA Check changed from Ready for QA to Dev Needed

I've documented how to run the test suite in that branch.

The test suite does not pass for me.

#5 Updated by intrigeri over 5 years ago

  • Target version deleted (Tails_0.23)

#6 Updated by alant over 5 years ago

  • Assignee deleted (alant)
  • % Done changed from 0 to 50
  • QA Check changed from Dev Needed to Ready for QA

Marked the test expected to failed as such.

#7 Updated by alant over 5 years ago

  • Assignee set to anonym

#8 Updated by anonym over 5 years ago

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • QA Check changed from Ready for QA to Pass

#9 Updated by intrigeri over 5 years ago

  • Target version set to Tails_1.1
  • % Done changed from 50 to 100

#10 Updated by BitingBird about 5 years ago

  • Status changed from Fix committed to Resolved

Also available in: Atom PDF