hardcoresushi/libcryfs
1
0
Fork 0
Code Issues Pull Requests Releases Activity

Remove --version test case because it doesn't work

Browse Source
This commit is contained in:
Sebastian Messmer 2018-02-01 11:24:55 -08:00
parent 5fc0b7cae8
commit 5463c14872
2 changed files with 1 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/cryfs-cli/program_options/Parser.cpp
View File

@ -182,5 +182,5 @@ void Parser::_addPositionalOptionForBaseDir(po::options_description *desc, po::p
[[noreturn]] void Parser::_showVersionAndExit() { [[noreturn]] void Parser::_showVersionAndExit() {
// no need to show version because it was already shown in the CryFS header before parsing program options // no need to show version because it was already shown in the CryFS header before parsing program options
exit(1); exit(0);
} }

10
test/cryfs-cli/program_options/ParserTest.cpp
View File

@ -55,16 +55,6 @@ TEST_F(ProgramOptionsParserTest, ShowCiphers) {
); );
} }
TEST_F(ProgramOptionsParserTest, Version) {
string expected = "CryFS Version " + gitversion::VersionString();
EXPECT_EXIT(
parse({"./myExecutable", "--version"}),
::testing::ExitedWithCode(0),
expected.c_str()
);
}
TEST_F(ProgramOptionsParserTest, BaseDir_Absolute) { TEST_F(ProgramOptionsParserTest, BaseDir_Absolute) {
ProgramOptions options = parse({"./myExecutable", "/home/user/baseDir", "/home/user/mountDir"}); ProgramOptions options = parse({"./myExecutable", "/home/user/baseDir", "/home/user/mountDir"});
EXPECT_EQ("/home/user/baseDir", options.baseDir()); EXPECT_EQ("/home/user/baseDir", options.baseDir());