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

Issue 340890043: [plaso] Skip tests that don't work on Windows Subsystem for Linux (Closed)

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

Description

[plaso] Skip tests that don't work on Windows Subsystem for Linux

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M tests/cli/log2timeline_tool.py View 2 chunks +6 lines, -0 lines 1 comment Download

Messages

Total messages: 4
onager
6 years, 1 month ago (2018-03-20 20:09:49 UTC) #1
Joachim Metz
https://codereview.appspot.com/340890043/diff/1/tests/cli/log2timeline_tool.py File tests/cli/log2timeline_tool.py (right): https://codereview.appspot.com/340890043/diff/1/tests/cli/log2timeline_tool.py#newcode345 tests/cli/log2timeline_tool.py:345: @unittest.skipIf(platform.system() == 'Windows', 'not supported on Windows') I opt ...
6 years, 1 month ago (2018-03-20 20:14:43 UTC) #2
Joachim Metz
per discussion LGTM
6 years, 1 month ago (2018-03-20 20:16:16 UTC) #3
onager
6 years, 1 month ago (2018-03-20 20:30:34 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 wsl
Sign in to reply to this message.

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