From patchwork Mon Apr 25 14:46:16 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hanlin Chen X-Patchwork-Id: 15706 Return-Path: X-Original-To: parsemail@patchwork.libcamera.org Delivered-To: parsemail@patchwork.libcamera.org Received: from lancelot.ideasonboard.com (lancelot.ideasonboard.com [92.243.16.209]) by patchwork.libcamera.org (Postfix) with ESMTPS id 6ED2FC3260 for ; Mon, 25 Apr 2022 14:46:30 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id A2E6E65648; Mon, 25 Apr 2022 16:46:29 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org; s=mail; t=1650897989; bh=y+6V4O/0su9kVaHW6PFoGQdl7/IFfzHY4YnAqPaov8A=; h=To:Date:In-Reply-To:References:Subject:List-Id:List-Unsubscribe: List-Archive:List-Post:List-Help:List-Subscribe:From:Reply-To: From; b=worlXHsRcsFWwuNHaUhgmEeWjp1lHGe6C77is67eHEgszfl2HzvQRmBzh61fsRMiu K4EMr6boJ3sFzaQfeBLSI5pBekHo337Z9ziS8KLgGXsl6tK6Z00eHYHC5pyek+7iBF /lYBSCEWSNaWrgqPRTDxIEnDmOfH3E4pUalHiilx7YC02Fy33JDDz1s26rwssDXfpp bwjMK6kLetDn3SA1B+0XpE+fEyBl4rftm0Nlg+XWyfM57WiVVdsmfX1mwvFC2rUQ5/ 5cpqZ4DuOC5ifKhiAkOQS4QseOOQnFPHIYXFJEHYM18kGzQDNSFqvTAlTb6pOO8Hca 56LUpXyM/nG2Q== Received: from mail-pg1-x534.google.com (mail-pg1-x534.google.com [IPv6:2607:f8b0:4864:20::534]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id 2C750604A9 for ; Mon, 25 Apr 2022 16:46:28 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=chromium.org header.i=@chromium.org header.b="OYcoorok"; dkim-atps=neutral Received: by mail-pg1-x534.google.com with SMTP id z21so5961818pgj.1 for ; Mon, 25 Apr 2022 07:46:28 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=doSB8OplibwubXrB7WWu5izCTnk9bKZaDlqwCHPNx8c=; b=OYcoorokiZtEnihLpl8u5++xF2SxvpvNjqAt2t503/9D0wo1Ls0mwdpxoYCgwAOPVg mkBJX2KPqR3s+ETvJV+YHForAX5LA/2tuwPAw5w/osaQl78vcRzFiKn13iu/cREu3z2q TPJm2+i1uJ17IPtNObAcvruqLKqaMNeFPmk5g= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=doSB8OplibwubXrB7WWu5izCTnk9bKZaDlqwCHPNx8c=; b=kkg8QvbETWj6JMyeaeTaTN4lyaIXOyipkfWKn+XHEIfIXwO2R9Yfo6qjbm4OWxpS2g UPYd3ERpaWBFl10FMPNUqmn4B0OolaCF7NlXC4QFhM2mzKIV0/qxOfAyu26aSotsNpUl lvyA8Np3ISxdM4ztb8ukd1RQaCjxthdOl8FbGgPJVVOVelRaSMu5RfU7oEu6w7SMx1xH 8XzevUA5yoTD3hvB2tVlZt53GcVyzHm+un3EV6jws+Oyt45ijUYhzbsEA4NH6KBJiIY7 0KrV9WXFEJFlcg/RbRE6mcib/VUSWOj0JOtCuS2s8mFgBAKM58tgK5rOw7nokPS0j9JQ XUmw== X-Gm-Message-State: AOAM531ve9ZQpjxcEJhn/6hoZ1iVq6RohIQpsNeHb39+sB6oKGWPR8Er H61oCNnK5vnhELJD/oPNO+nR4IVzFq0LlQ== X-Google-Smtp-Source: ABdhPJzmEFHP3te/m7uCP4gOuqeWpVpQW/k8ylSR06FyPtpb4K5NWJ/ja6Vxr4DVD2i189b3O70nYA== X-Received: by 2002:a05:6a00:24cb:b0:50a:8151:9abc with SMTP id d11-20020a056a0024cb00b0050a81519abcmr19128222pfv.57.1650897986479; Mon, 25 Apr 2022 07:46:26 -0700 (PDT) Received: from localhost ([2401:fa00:1:17:38a2:b0a:67fa:9b05]) by smtp.gmail.com with UTF8SMTPSA id 189-20020a6217c6000000b0050aca6473e0sm11851025pfx.192.2022.04.25.07.46.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 25 Apr 2022 07:46:26 -0700 (PDT) To: libcamera-devel@lists.libcamera.org Date: Mon, 25 Apr 2022 22:46:16 +0800 Message-Id: <20220425144617.2549778-3-hanlinchen@chromium.org> X-Mailer: git-send-email 2.36.0.rc2.479.g8af0fa9b8e-goog In-Reply-To: <20220425144617.2549778-1-hanlinchen@chromium.org> References: <20220425144617.2549778-1-hanlinchen@chromium.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v5 2/3] test: Add YamlParser test X-BeenThere: libcamera-devel@lists.libcamera.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-Patchwork-Original-From: Han-Lin Chen via libcamera-devel From: Hanlin Chen Reply-To: Han-Lin Chen Errors-To: libcamera-devel-bounces@lists.libcamera.org Sender: "libcamera-devel" Add a unit test to exercise the API of the YamlParser class. Signed-off-by: Han-Lin Chen Reviewed-by: Laurent Pinchart --- test/meson.build | 1 + test/yaml-parser.cpp | 496 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 497 insertions(+) create mode 100644 test/yaml-parser.cpp diff --git a/test/meson.build b/test/meson.build index fd4c5ca0..9c68fae1 100644 --- a/test/meson.build +++ b/test/meson.build @@ -54,6 +54,7 @@ internal_tests = [ ['timer-thread', 'timer-thread.cpp'], ['unique-fd', 'unique-fd.cpp'], ['utils', 'utils.cpp'], + ['yaml-parser', 'yaml-parser.cpp'], ] internal_non_parallel_tests = [ diff --git a/test/yaml-parser.cpp b/test/yaml-parser.cpp new file mode 100644 index 00000000..b9122287 --- /dev/null +++ b/test/yaml-parser.cpp @@ -0,0 +1,496 @@ +/* SPDX-License-Identifier: GPL-2.0-or-later */ +/* + * Copyright (C) 2022, Google Inc. + * + * yaml-parser.cpp - YAML parser operations tests + */ + +#include +#include +#include + +#include + +#include "test.h" + +using namespace std; +using namespace libcamera; + +static const string testYaml = + "string: libcamera\n" + "double: 3.14159\n" + "uint32_t: 100\n" + "int32_t: -100\n" + "size: [1920, 1080]\n" + "list:\n" + " - James\n" + " - Mary\n" + "dictionary:\n" + " a: 1\n" + " b: 2\n" + " c: 3\n" + "level1:\n" + " level2:\n" + " - [1, 2]\n" + " - {one: 1, two: 2}\n"; + +static const string invalidYaml = + "Invalid : - YAML : - Content"; + +class YamlParserTest : public Test +{ +protected: + bool createFile(const string &content, string &filename) + { + filename = "/tmp/libcamera.test.XXXXXX"; + int fd = mkstemp(&filename.front()); + if (fd == -1) + return false; + + FILE *fh = fdopen(fd, "w"); + fputs(content.c_str(), fh); + + fclose(fh); + + return true; + } + + int init() + { + if (!createFile(testYaml, testYamlFile_)) + return TestFail; + + if (!createFile(invalidYaml, invalidYamlFile_)) + return TestFail; + + return TestPass; + } + + int run() + { + /* Test invalid YAML file */ + FILE *fh = fopen(invalidYamlFile_.c_str(), "r"); + if (!fh) { + cerr << "Fail to open invalid YAML file"; + return TestFail; + } + + std::unique_ptr root = YamlParser::parse(fh); + + if (root) { + cerr << "Invalid YAML file parse successfully"; + return TestFail; + } + + fclose(fh); + + /* Test YAML file */ + fh = fopen(testYamlFile_.c_str(), "r"); + if (!fh) { + cerr << "Fail to open test YAML file"; + return TestFail; + } + + root = YamlParser::parse(fh); + + if (!root) { + cerr << "Fail to parse test YAML file: "; + return TestFail; + } + + fclose(fh); + + if (!(*root).isDictionary()) { + cerr << "YAML root is not dictionary"; + return TestFail; + } + + if (!(*root).contains("string")) { + cerr << "Missing string object in YAML root"; + return TestFail; + } + + if (!(*root).contains("double")) { + cerr << "Missing double object in YAML root"; + return TestFail; + } + + if (!(*root).contains("int32_t")) { + cerr << "Missing int32_t object in YAML root"; + return TestFail; + } + + if (!(*root).contains("uint32_t")) { + cerr << "Missing uint32_t object in YAML root"; + return TestFail; + } + + if (!(*root).contains("size")) { + cerr << "Missing Size object in YAML root"; + return TestFail; + } + + if (!(*root).contains("list")) { + cerr << "Missing list object in YAML root"; + return TestFail; + } + + if (!(*root).contains("dictionary")) { + cerr << "Missing dictionary object in YAML root"; + return TestFail; + } + + if (!(*root).contains("level1")) { + cerr << "Missing leveled object in YAML root"; + return TestFail; + } + + /* Test string object */ + bool ok; + auto &strObj = (*root)["string"]; + + if (strObj.isDictionary()) { + cerr << "String object parse as Dictionary"; + return TestFail; + } + + if (strObj.isList()) { + cerr << "String object parse as List"; + return TestFail; + } + + if (strObj.get("", &ok) != "libcamera" || !ok) { + cerr << "String object parse as wrong content"; + return TestFail; + } + + if (strObj.get(-1, &ok) != -1 || ok) { + cerr << "String object parse as integer"; + return TestFail; + } + + if (strObj.get(1, &ok) != 1 || ok) { + cerr << "String object parse as unsigned integer"; + return TestFail; + } + + if (strObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "String object parse as double"; + return TestFail; + } + + if (strObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "String object parse as Size"; + return TestFail; + } + + /* Test int32_t object */ + auto &int32Obj = (*root)["int32_t"]; + + if (int32Obj.isDictionary()) { + cerr << "Integer object parse as Dictionary"; + return TestFail; + } + + if (int32Obj.isList()) { + cerr << "Integer object parse as Integer"; + return TestFail; + } + + if (int32Obj.get(-100, &ok) != -100 || !ok) { + cerr << "Integer object parse as wrong value"; + return TestFail; + } + + if (int32Obj.get("", &ok) != "-100" || !ok) { + cerr << "Integer object fail to parse as string"; + return TestFail; + } + + if (int32Obj.get(1.0, &ok) != -100.0 || !ok) { + cerr << "Integer object fail to parse as double"; + return TestFail; + } + + if (int32Obj.get(1, &ok) != 1 || ok) { + cerr << "Negative integer object parse as unsigned integer"; + return TestFail; + } + + if (int32Obj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Integer object parse as Size"; + return TestFail; + } + + /* Test uint32_t object */ + auto &uint32Obj = (*root)["uint32_t"]; + + if (uint32Obj.isDictionary()) { + cerr << "Unsigned integer object parse as Dictionary"; + return TestFail; + } + + if (uint32Obj.isList()) { + cerr << "Unsigned integer object parse as List"; + return TestFail; + } + + if (uint32Obj.get(-1, &ok) != 100 || !ok) { + cerr << "Unsigned integer object fail to parse as integer"; + return TestFail; + } + + if (uint32Obj.get("", &ok) != "100" || !ok) { + cerr << "Unsigned integer object fail to parse as string"; + return TestFail; + } + + if (uint32Obj.get(1.0, &ok) != 100.0 || !ok) { + cerr << "Unsigned integer object fail to parse as double"; + return TestFail; + } + + if (uint32Obj.get(100, &ok) != 100 || !ok) { + cerr << "Unsigned integer object parsed as wrong value"; + return TestFail; + } + + if (uint32Obj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Unsigned integer object parsed as Size"; + return TestFail; + } + + /* Test double value */ + auto &doubleObj = (*root)["double"]; + + if (doubleObj.isDictionary()) { + cerr << "Double object parse as Dictionary"; + return TestFail; + } + + if (doubleObj.isList()) { + cerr << "Double object parse as List"; + return TestFail; + } + + if (doubleObj.get("", &ok) != "3.14159" || !ok) { + cerr << "Double object fail to parse as string"; + return TestFail; + } + + if (doubleObj.get(1.0, &ok) != 3.14159 || !ok) { + cerr << "Double object parse as wrong value"; + return TestFail; + } + + if (doubleObj.get(-1, &ok) != -1 || ok) { + cerr << "Double object parse as integer"; + return TestFail; + } + + if (doubleObj.get(1, &ok) != 1 || ok) { + cerr << "Double object parse as unsigned integer"; + return TestFail; + } + + if (doubleObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Double object parse as Size"; + return TestFail; + } + + /* Test Size value */ + auto &sizeObj = (*root)["size"]; + + if (sizeObj.isDictionary()) { + cerr << "Size object parse as Dictionary"; + return TestFail; + } + + if (!sizeObj.isList()) { + cerr << "Size object parse as List"; + return TestFail; + } + + if (sizeObj.get("", &ok) != "" || ok) { + cerr << "Size object parse as string"; + return TestFail; + } + + if (sizeObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "Size object parse as double"; + return TestFail; + } + + if (sizeObj.get(-1, &ok) != -1 || ok) { + cerr << "Size object parse as integer"; + return TestFail; + } + + if (sizeObj.get(1, &ok) != 1 || ok) { + cerr << "Size object parse as unsigned integer"; + return TestFail; + } + + if (sizeObj.get(Size(0, 0), &ok) != Size(1920, 1080) || !ok) { + cerr << "Size object parse as wrong value"; + return TestFail; + } + + /* Test list object */ + auto &listObj = (*root)["list"]; + + if (listObj.isDictionary()) { + cerr << "List object parse as Dictionary"; + return TestFail; + } + + if (!listObj.isList()) { + cerr << "List object fail to parse as List"; + return TestFail; + } + + if (listObj.get("", &ok) != "" || ok) { + cerr << "List object parse as string"; + return TestFail; + } + + if (listObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "List object parse as double"; + return TestFail; + } + + if (listObj.get(-1, &ok) != -1 || ok) { + cerr << "List object parse as integer"; + return TestFail; + } + + if (listObj.get(1, &ok) != 1 || ok) { + cerr << "List object parse as unsigne integer"; + return TestFail; + } + + if (listObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "String list object parse as Size"; + return TestFail; + } + + if (listObj.size() > 2) { + cerr << "List object parse with wrong size"; + return TestFail; + } + + if (listObj[0].get("") != "James" || + listObj[1].get("") != "Mary") { + cerr << "List object parse as wrong value"; + return TestFail; + } + + /* Test dictionary object */ + auto &dictObj = (*root)["dictionary"]; + + if (!dictObj.isDictionary()) { + cerr << "Dictionary object fail to parse as Dictionary"; + return TestFail; + } + + if (dictObj.isList()) { + cerr << "Dictionary object parse as List"; + return TestFail; + } + + if (dictObj.get("", &ok) != "" || ok) { + cerr << "Dictionary object parse as string"; + return TestFail; + } + + if (dictObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "Dictionary object parse as double"; + return TestFail; + } + + if (dictObj.get(-1, &ok) != -1 || ok) { + cerr << "Dictionary object parse as integer"; + return TestFail; + } + + if (dictObj.get(1, &ok) != 1 || ok) { + cerr << "Dictionary object parse as unsigned integer"; + return TestFail; + } + + if (dictObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Dictionary object parse as Size"; + return TestFail; + } + + auto memeberNames = dictObj.getMemberNames(); + sort(memeberNames.begin(), memeberNames.end()); + + if (memeberNames.size() != 3) { + cerr << "Dictionary object fail to extra member names"; + return TestFail; + } + + if (memeberNames[0] != "a" || + memeberNames[1] != "b" || + memeberNames[2] != "c") { + cerr << "Dictionary object fail to parse member names"; + return TestFail; + } + + if (dictObj["a"].get(0) != 1 || + dictObj["b"].get(0) != 2 || + dictObj["c"].get(0) != 3) { + cerr << "Dictionary object fail to parse member value"; + return TestFail; + } + + /* Test leveled objects */ + auto &level1Obj = (*root)["level1"]; + + if (!level1Obj.isDictionary()) { + cerr << "level1 object fail to parse as Dictionary"; + return TestFail; + } + + auto &level2Obj = level1Obj["level2"]; + + if (!level2Obj.isList() || level2Obj.size() != 2) { + cerr << "level2 object should be 2 element list"; + return TestFail; + } + + auto &firstElement = level2Obj[0]; + if (!firstElement.isList() || + firstElement.size() != 2 || + firstElement[0].get(0) != 1 || + firstElement[1].get(0) != 2) { + cerr << "The first element of level2 object fail to pars as integer list"; + return TestFail; + } + + auto &secondElement = level2Obj[1]; + if (!secondElement.isDictionary() || + !secondElement.contains("one") || + !secondElement.contains("two") || + secondElement["one"].get(0) != 1 || + secondElement["two"].get(0) != 2) { + cerr << "The second element of level2 object fail to pases as dictionary"; + return TestFail; + } + + return TestPass; + } + + void cleanup() + { + unlink(testYamlFile_.c_str()); + unlink(invalidYamlFile_.c_str()); + } + +private: + std::string testYamlFile_; + std::string invalidYamlFile_; +}; + +TEST_REGISTER(YamlParserTest)