From patchwork Wed Apr 27 14:09:28 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Hanlin Chen X-Patchwork-Id: 15731 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 64CE6C3256 for ; Wed, 27 Apr 2022 14:09:41 +0000 (UTC) Received: from lancelot.ideasonboard.com (localhost [IPv6:::1]) by lancelot.ideasonboard.com (Postfix) with ESMTP id 2091A6564B; Wed, 27 Apr 2022 16:09:41 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=libcamera.org; s=mail; t=1651068581; bh=7ppwUTApfsRk05zIklJeAUfhKKZ6Ti5B1xLNzfCjAec=; 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=c3ZxWV2uMQ/hG4fVFpkb+5jggqjqD6KDrS0pjmx5yF8GrTRXU5S+2Mbwh4MVXVQ2r KDDg96x6m/srAjBA/oRAtozc41fxEk1qzgK6TrvrOs7/L4g+SMMCxLEhflk6KmHhOb rEEhV3W2UCSovK2qtkfnLvsvdUnBOlQXZQC5v/kjDflxcxJmSjVydLa9x+2RFxssRc ILHQmUQVeWenXCCDBdnesmwl+jtNtjXW3EFdL2WVBtaFrJ6LNvJ4TIUoFRQxa5ZbkI ewk6Lc8UVnVJj7E9/x3h8xGP/1FfM1XsEORU9KHo/eWZH+ZSuJeniehiJP3BRdDtAh QfPhxhPKxJY5Q== Received: from mail-pg1-x529.google.com (mail-pg1-x529.google.com [IPv6:2607:f8b0:4864:20::529]) by lancelot.ideasonboard.com (Postfix) with ESMTPS id BD37265647 for ; Wed, 27 Apr 2022 16:09:38 +0200 (CEST) Authentication-Results: lancelot.ideasonboard.com; dkim=pass (1024-bit key; unprotected) header.d=chromium.org header.i=@chromium.org header.b="ENLpp7f3"; dkim-atps=neutral Received: by mail-pg1-x529.google.com with SMTP id z21so1561600pgj.1 for ; Wed, 27 Apr 2022 07:09:38 -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=ov2nrY9RN7eTkFBmoTdK9AcqT3YhKHYcAfF+RuBVOXM=; b=ENLpp7f3AQzfClppkThUPPP//nQqLESquL1CYrVAB+B6ZGY3sDv+tQIxte2ZV7QfUA YL/ezj6HwPXvWKBlWG89dZFM3uVsbGFioCunJeDR/StNaV6FfTJAtL4rzOQkLNGiFZ/e o8McHPJR15sBGE+RASMR324WlK6bheOSVgLW4= 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=ov2nrY9RN7eTkFBmoTdK9AcqT3YhKHYcAfF+RuBVOXM=; b=TI9I5tpP1m6YnQvqHfKFcvk5war/7ZXuvP1HGCL9YRrjYsS2Ss6NY3HbbrcUvMGxKr wPOdco6X7q/gHxstOKPOcu9aNH9K2lAACPoePAbSlrB2Ss11A/s5bDOq58wBwdtSIObX lyoTd2usXRHdTeYUr+CmH1t39+dN+U4l5HrSDyqL2m0FMIJM5JSPdkICKNaprVvGWLVY jAibTPYfwcWODphkAVu4ei5/Pvtt6NwgQ7uIE+h1MzNn2pZBzffunkkr2xVPxAMA+nX6 cSBETLBibYjVtpMwU9uYZIdIC70WIEdd2koTXlXoBjAkJzzC+8QAYKols7VKchuHW5OO AfhQ== X-Gm-Message-State: AOAM530VFaKM5ef6MlEncDux/AWIuGKQKbFS1Nh9wLKR7Qddd+b38wVN OFohJ0XNO2YULnBPjpFYLGc0kD9uDsfmZA== X-Google-Smtp-Source: ABdhPJyCx1NpBufqg8i2Z7cI/VdfQTA2UvuH/mehSEeR3t157G0R0wPSyI1L0+eQM7geDDXbAPI2Mw== X-Received: by 2002:a63:2b45:0:b0:3ab:971b:be5e with SMTP id r66-20020a632b45000000b003ab971bbe5emr6200914pgr.265.1651068576829; Wed, 27 Apr 2022 07:09:36 -0700 (PDT) Received: from localhost ([2401:fa00:1:17:47e:a9de:8ea1:3876]) by smtp.gmail.com with UTF8SMTPSA id s4-20020a056a00194400b004fb358ffe84sm19372612pfk.104.2022.04.27.07.09.35 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 27 Apr 2022 07:09:36 -0700 (PDT) To: libcamera-devel@lists.libcamera.org Date: Wed, 27 Apr 2022 22:09:28 +0800 Message-Id: <20220427140929.429977-2-hanlinchen@chromium.org> X-Mailer: git-send-email 2.36.0.rc2.479.g8af0fa9b8e-goog In-Reply-To: <20220427140929.429977-1-hanlinchen@chromium.org> References: <20220427140929.429977-1-hanlinchen@chromium.org> MIME-Version: 1.0 Subject: [libcamera-devel] [PATCH v6 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 | 494 +++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 495 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..e00489db --- /dev/null +++ b/test/yaml-parser.cpp @@ -0,0 +1,494 @@ +/* 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" << std::endl; + return TestFail; + } + + std::unique_ptr root = YamlParser::parse(fh); + fclose(fh); + + if (root) { + cerr << "Invalid YAML file parse successfully" << std::endl; + return TestFail; + } + + /* Test YAML file */ + fh = fopen(testYamlFile_.c_str(), "r"); + if (!fh) { + cerr << "Fail to open test YAML file" << std::endl; + return TestFail; + } + + root = YamlParser::parse(fh); + fclose(fh); + + if (!root) { + cerr << "Fail to parse test YAML file: " << std::endl; + return TestFail; + } + + if (!root->isDictionary()) { + cerr << "YAML root is not dictionary" << std::endl; + return TestFail; + } + + if (!root->contains("string")) { + cerr << "Missing string object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("double")) { + cerr << "Missing double object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("int32_t")) { + cerr << "Missing int32_t object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("uint32_t")) { + cerr << "Missing uint32_t object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("size")) { + cerr << "Missing Size object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("list")) { + cerr << "Missing list object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("dictionary")) { + cerr << "Missing dictionary object in YAML root" << std::endl; + return TestFail; + } + + if (!root->contains("level1")) { + cerr << "Missing leveled object in YAML root" << std::endl; + return TestFail; + } + + /* Test string object */ + bool ok; + auto &strObj = (*root)["string"]; + + if (strObj.isDictionary()) { + cerr << "String object parse as Dictionary" << std::endl; + return TestFail; + } + + if (strObj.isList()) { + cerr << "String object parse as List" << std::endl; + return TestFail; + } + + if (strObj.get("", &ok) != "libcamera" || !ok) { + cerr << "String object parse as wrong content" << std::endl; + return TestFail; + } + + if (strObj.get(-1, &ok) != -1 || ok) { + cerr << "String object parse as integer" << std::endl; + return TestFail; + } + + if (strObj.get(1, &ok) != 1 || ok) { + cerr << "String object parse as unsigned integer" << std::endl; + return TestFail; + } + + if (strObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "String object parse as double" << std::endl; + return TestFail; + } + + if (strObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "String object parse as Size" << std::endl; + return TestFail; + } + + /* Test int32_t object */ + auto &int32Obj = (*root)["int32_t"]; + + if (int32Obj.isDictionary()) { + cerr << "Integer object parse as Dictionary" << std::endl; + return TestFail; + } + + if (int32Obj.isList()) { + cerr << "Integer object parse as Integer" << std::endl; + return TestFail; + } + + if (int32Obj.get(-100, &ok) != -100 || !ok) { + cerr << "Integer object parse as wrong value" << std::endl; + return TestFail; + } + + if (int32Obj.get("", &ok) != "-100" || !ok) { + cerr << "Integer object fail to parse as string" << std::endl; + return TestFail; + } + + if (int32Obj.get(1.0, &ok) != -100.0 || !ok) { + cerr << "Integer object fail to parse as double" << std::endl; + return TestFail; + } + + if (int32Obj.get(1, &ok) != 1 || ok) { + cerr << "Negative integer object parse as unsigned integer" << std::endl; + return TestFail; + } + + if (int32Obj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Integer object parse as Size" << std::endl; + return TestFail; + } + + /* Test uint32_t object */ + auto &uint32Obj = (*root)["uint32_t"]; + + if (uint32Obj.isDictionary()) { + cerr << "Unsigned integer object parse as Dictionary" << std::endl; + return TestFail; + } + + if (uint32Obj.isList()) { + cerr << "Unsigned integer object parse as List" << std::endl; + return TestFail; + } + + if (uint32Obj.get(-1, &ok) != 100 || !ok) { + cerr << "Unsigned integer object fail to parse as integer" << std::endl; + return TestFail; + } + + if (uint32Obj.get("", &ok) != "100" || !ok) { + cerr << "Unsigned integer object fail to parse as string" << std::endl; + return TestFail; + } + + if (uint32Obj.get(1.0, &ok) != 100.0 || !ok) { + cerr << "Unsigned integer object fail to parse as double" << std::endl; + return TestFail; + } + + if (uint32Obj.get(100, &ok) != 100 || !ok) { + cerr << "Unsigned integer object parsed as wrong value" << std::endl; + return TestFail; + } + + if (uint32Obj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Unsigned integer object parsed as Size" << std::endl; + return TestFail; + } + + /* Test double value */ + auto &doubleObj = (*root)["double"]; + + if (doubleObj.isDictionary()) { + cerr << "Double object parse as Dictionary" << std::endl; + return TestFail; + } + + if (doubleObj.isList()) { + cerr << "Double object parse as List" << std::endl; + return TestFail; + } + + if (doubleObj.get("", &ok) != "3.14159" || !ok) { + cerr << "Double object fail to parse as string" << std::endl; + return TestFail; + } + + if (doubleObj.get(1.0, &ok) != 3.14159 || !ok) { + cerr << "Double object parse as wrong value" << std::endl; + return TestFail; + } + + if (doubleObj.get(-1, &ok) != -1 || ok) { + cerr << "Double object parse as integer" << std::endl; + return TestFail; + } + + if (doubleObj.get(1, &ok) != 1 || ok) { + cerr << "Double object parse as unsigned integer" << std::endl; + return TestFail; + } + + if (doubleObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Double object parse as Size" << std::endl; + return TestFail; + } + + /* Test Size value */ + auto &sizeObj = (*root)["size"]; + + if (sizeObj.isDictionary()) { + cerr << "Size object parse as Dictionary" << std::endl; + return TestFail; + } + + if (!sizeObj.isList()) { + cerr << "Size object parse as List" << std::endl; + return TestFail; + } + + if (sizeObj.get("", &ok) != "" || ok) { + cerr << "Size object parse as string" << std::endl; + return TestFail; + } + + if (sizeObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "Size object parse as double" << std::endl; + return TestFail; + } + + if (sizeObj.get(-1, &ok) != -1 || ok) { + cerr << "Size object parse as integer" << std::endl; + return TestFail; + } + + if (sizeObj.get(1, &ok) != 1 || ok) { + cerr << "Size object parse as unsigned integer" << std::endl; + return TestFail; + } + + if (sizeObj.get(Size(0, 0), &ok) != Size(1920, 1080) || !ok) { + cerr << "Size object parse as wrong value" << std::endl; + return TestFail; + } + + /* Test list object */ + auto &listObj = (*root)["list"]; + + if (listObj.isDictionary()) { + cerr << "List object parse as Dictionary" << std::endl; + return TestFail; + } + + if (!listObj.isList()) { + cerr << "List object fail to parse as List" << std::endl; + return TestFail; + } + + if (listObj.get("", &ok) != "" || ok) { + cerr << "List object parse as string" << std::endl; + return TestFail; + } + + if (listObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "List object parse as double" << std::endl; + return TestFail; + } + + if (listObj.get(-1, &ok) != -1 || ok) { + cerr << "List object parse as integer" << std::endl; + return TestFail; + } + + if (listObj.get(1, &ok) != 1 || ok) { + cerr << "List object parse as unsigne integer" << std::endl; + return TestFail; + } + + if (listObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "String list object parse as Size" << std::endl; + return TestFail; + } + + if (listObj.size() > 2) { + cerr << "List object parse with wrong size" << std::endl; + return TestFail; + } + + if (listObj[0].get("") != "James" || + listObj[1].get("") != "Mary") { + cerr << "List object parse as wrong value" << std::endl; + return TestFail; + } + + /* Test dictionary object */ + auto &dictObj = (*root)["dictionary"]; + + if (!dictObj.isDictionary()) { + cerr << "Dictionary object fail to parse as Dictionary" << std::endl; + return TestFail; + } + + if (dictObj.isList()) { + cerr << "Dictionary object parse as List" << std::endl; + return TestFail; + } + + if (dictObj.get("", &ok) != "" || ok) { + cerr << "Dictionary object parse as string" << std::endl; + return TestFail; + } + + if (dictObj.get(1.0, &ok) != 1.0 || ok) { + cerr << "Dictionary object parse as double" << std::endl; + return TestFail; + } + + if (dictObj.get(-1, &ok) != -1 || ok) { + cerr << "Dictionary object parse as integer" << std::endl; + return TestFail; + } + + if (dictObj.get(1, &ok) != 1 || ok) { + cerr << "Dictionary object parse as unsigned integer" << std::endl; + return TestFail; + } + + if (dictObj.get(Size(0, 0), &ok) != Size(0, 0) || ok) { + cerr << "Dictionary object parse as Size" << std::endl; + return TestFail; + } + + auto memeberNames = dictObj.memberNames(); + sort(memeberNames.begin(), memeberNames.end()); + + if (memeberNames.size() != 3) { + cerr << "Dictionary object fail to extra member names" << std::endl; + return TestFail; + } + + if (memeberNames[0] != "a" || + memeberNames[1] != "b" || + memeberNames[2] != "c") { + cerr << "Dictionary object fail to parse member names" << std::endl; + 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" << std::endl; + return TestFail; + } + + /* Test leveled objects */ + auto &level1Obj = (*root)["level1"]; + + if (!level1Obj.isDictionary()) { + cerr << "level1 object fail to parse as Dictionary" << std::endl; + return TestFail; + } + + auto &level2Obj = level1Obj["level2"]; + + if (!level2Obj.isList() || level2Obj.size() != 2) { + cerr << "level2 object should be 2 element list" << std::endl; + 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 parse as integer list" << std::endl; + 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 parse as dictionary" << std::endl; + return TestFail; + } + + return TestPass; + } + + void cleanup() + { + unlink(testYamlFile_.c_str()); + unlink(invalidYamlFile_.c_str()); + } + +private: + std::string testYamlFile_; + std::string invalidYamlFile_; +}; + +TEST_REGISTER(YamlParserTest)