Fix incorrect 'test' statement in unit test
authorChip Turner <chip@fb.com>
Tue, 23 Dec 2014 18:40:17 +0000 (10:40 -0800)
committerDave Watson <davejwatson@fb.com>
Mon, 29 Dec 2014 18:40:18 +0000 (10:40 -0800)
commitb61dfbebfb3c125bc5b7e4cc235ab2ea8bfb5086
tree02b078c2df6c06addbd5d4cad4cc593abb911892
parent2799131d44876a12fbbc150afa688ad997ffb723
Fix incorrect 'test' statement in unit test

Summary:
The 'test' command uses one equal sign for string equality, not
two.  bash apparently is okay with two, but zsh isn't, causing this test
to pass if your shell was bash and fail if it was zsh.  The test now
passes with bash, bash in sh mode, zsh, and even dash.

Test Plan:
runtests, also, these pass (zsh failed before):

SHELL=/bin/bash _build/dbg/folly/test/subprocess_test
SHELL=/bin/sh _build/dbg/folly/test/subprocess_test
SHELL=/bin/zsh _build/dbg/folly/test/subprocess_test
SHELL=/bin/dash _build/dbg/folly/test/subprocess_test

Reviewed By: njormrod@fb.com

Subscribers: lins, anca, folly-diffs@

FB internal diff: D1756090

Signature: t1:1756090:1419360674:8576e61a6e0ee102612c5eae0e1fbd79cc397bfa
folly/test/SubprocessTest.cpp