Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(6251)

Issue 345770043: [plaso] Improved engine test coverage and clean up (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years ago by Joachim Metz
Modified:
6 years ago
Reviewers:
jberggren
CC:
kiddi, log2timeline-dev_googlegroups.com, romaing, onager
Visibility:
Public.

Description

[plaso] Improved engine test coverage and clean up

Patch Set 1 #

Patch Set 2 : Changes after merge #

Unified diffs Side-by-side diffs Delta from patch set Stats (+136 lines, -89 lines) Patch
M plaso/engine/engine.py View 1 2 chunks +85 lines, -84 lines 0 comments Download
M tests/engine/engine.py View 3 chunks +47 lines, -4 lines 0 comments Download
M tests/multi_processing/worker_process.py View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 4
Joachim Metz
6 years ago (2018-03-31 16:58:55 UTC) #1
jberggren
On 2018/03/31 16:58:55, Joachim Metz wrote: LGTM
6 years ago (2018-04-05 08:22:01 UTC) #2
Joachim Metz
Code updated.
6 years ago (2018-04-06 04:36:39 UTC) #3
Joachim Metz
6 years ago (2018-04-06 04:45:11 UTC) #4
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: review.py close testsnclean2
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b