commit | 2e38263961190298d7198058ed91649dee106257 | [log] [tgz] |
---|---|---|
author | mukesh agrawal <quiche@chromium.org> | Thu Aug 01 13:41:15 2013 -0700 |
committer | ChromeBot <chrome-bot@google.com> | Tue Aug 06 10:44:24 2013 -0700 |
tree | a994dfbfcfe6e556d6b94f0e98ad98150ffaa7ad | |
parent | df5699618d61c301aa8436c8f50076e3b646aa75 [diff] |
shill: L2TPIPSecDriver: add test for another Notify case If we already have a |device_| when L2TPIPsecDriver::Notify is called, we don't create a new PPPDevice. This case didn't have test coverage before, so this CL adds it. While there: refactor some code, to reduce duplication across test cases. BUG=chromium:246826 TEST=unit tests (old+new) Change-Id: Ic7325cabfd28f9db4efd90c0f7b765c1f48d4901 Reviewed-on: https://gerrit.chromium.org/gerrit/64263 Reviewed-by: Ben Chan <benchan@chromium.org> Commit-Queue: mukesh agrawal <quiche@chromium.org> Tested-by: mukesh agrawal <quiche@chromium.org>