Fixing getDocLayerVersion command. (#180)

Author: apkar

src/ExtCmd.actor.cpp

@@ -876,8 +876,14 @@ struct GetDocLayerVersionCmd {
 	static Future<Reference<ExtMsgReply>> call(Reference<ExtConnection> ec,
 	                                           Reference<ExtMsgQuery> query,
 	                                           Reference<ExtMsgReply> reply) {
-		reply->addDocument(BSON("package version" << FDB_DOC_VT_VERSION << "source version" << getGitVersion()
-		                                          << "flow source version" << getFlowGitVersion()));
+		// clang-format off
+		reply->addDocument(BSON(
+			"ok" << 1 <<
+			"packageVersion" << FDB_DOC_VT_VERSION <<
+			"sourceVersion" << getGitVersion() <<
+			"flowSourceVersion" << getFlowGitVersion()
+			));
+		// clang-format on
 		return reply;
 	}
 };

test/correctness/smoke/test_basic_flow.py

@@ -75,3 +75,11 @@ def test_connection_status(fixture_db):
     assert 'authenticatedUserPrivileges' not in status['authInfo']
     assert status['ok'] == 1
 
+
+def test_doclayer_info(fixture_db):
+    db = fixture_db
+    info = db.command("getDocLayerVersion")
+    assert 'flowSourceVersion' in info
+    assert 'packageVersion' in info
+    assert 'sourceVersion' in info
+