Ticket #8336 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Write a better unit test for RemoveLogs

Reported by: Samuel Jackson Owned by: Dan Nixon
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

The remove logs test is a over complicated copy of the LoadLogs unit test. We should just create a workspace and add some logs, then run the algorithm to remove them rather than loading files.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

bulk move to assigned at the into of the triage step

comment:2 Changed 6 years ago by Dan Nixon

  • Owner changed from Anyone to Dan Nixon

comment:3 Changed 6 years ago by Dan Nixon

  • Milestone changed from Backlog to Release 3.3

comment:4 Changed 6 years ago by Dan Nixon

  • Status changed from assigned to inprogress

Replaced old unit test for RemoveLogs

Refs #8336

Changeset: 6d3f330975773fe66ea93248f10d7d60a21abfa5

comment:5 Changed 6 years ago by Dan Nixon

  • Status changed from inprogress to verify
  • Resolution set to fixed

To test:

  • Review changes to unit test and ensure it has suitable coverage
  • Ensure test is passing

comment:6 Changed 6 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:7 Changed 6 years ago by Karl Palmen

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8336_better_unit_test_for_removelogs'

Full changeset: ​fb323e4e07c07d6e6daf5ab5c5f4b2d897dbe6e0

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9181

Note: See TracTickets for help on using tickets.