{"id":221,"url":"https://patchwork.libcamera.org/api/1.1/covers/221/?format=json","web_url":"https://patchwork.libcamera.org/cover/221/","project":{"id":1,"url":"https://patchwork.libcamera.org/api/1.1/projects/1/?format=json","name":"libcamera","link_name":"libcamera","list_id":"libcamera_core","list_email":"libcamera-devel@lists.libcamera.org","web_url":"","scm_url":"","webscm_url":""},"msgid":"<20190114095417.16473-1-kieran.bingham@ideasonboard.com>","date":"2019-01-14T09:54:15","name":"[libcamera-devel,RFC,0/2] Test Status Refactoring","submitter":{"id":4,"url":"https://patchwork.libcamera.org/api/1.1/people/4/?format=json","name":"Kieran Bingham","email":"kieran.bingham@ideasonboard.com"},"mbox":"https://patchwork.libcamera.org/cover/221/mbox/","series":[{"id":76,"url":"https://patchwork.libcamera.org/api/1.1/series/76/?format=json","web_url":"https://patchwork.libcamera.org/project/libcamera/list/?series=76","date":"2019-01-14T09:54:15","name":"Test Status Refactoring","version":1,"mbox":"https://patchwork.libcamera.org/series/76/mbox/"}],"comments":"https://patchwork.libcamera.org/api/covers/221/comments/","headers":{"Return-Path":"<kieran.bingham@ideasonboard.com>","Received":["from perceval.ideasonboard.com (perceval.ideasonboard.com\n\t[213.167.242.64])\n\tby lancelot.ideasonboard.com (Postfix) with ESMTPS id A54F960C6A\n\tfor <libcamera-devel@lists.libcamera.org>;\n\tMon, 14 Jan 2019 10:54:21 +0100 (CET)","from localhost.localdomain\n\t(cpc89242-aztw30-2-0-cust488.18-1.cable.virginm.net [86.31.129.233])\n\tby perceval.ideasonboard.com (Postfix) with ESMTPSA id 22046505;\n\tMon, 14 Jan 2019 10:54:21 +0100 (CET)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com;\n\ts=mail; t=1547459661;\n\tbh=BctrM7PRb44fNlEFmXvIomVPrIpNp5XVH8mdWRGWqDc=;\n\th=From:To:Cc:Subject:Date:From;\n\tb=Wr3s+RKRJSyhUp+ra+APZzyk+U22CWbUB4gAAooNIsD6udVYbfJouwCaPlrMcJg6O\n\twBtTB8f/pDTBkowp0gaLLysSfxBI0zOYD90U7ei6YAhIxq3REprP1aHBQcySTTeHcV\n\t8O44d4mEtSPDnUNyOlFDOhPbLi62Waf7Erv84sA0=","From":"Kieran Bingham <kieran.bingham@ideasonboard.com>","To":"LibCamera Devel <libcamera-devel@lists.libcamera.org>","Date":"Mon, 14 Jan 2019 09:54:15 +0000","Message-Id":"<20190114095417.16473-1-kieran.bingham@ideasonboard.com>","X-Mailer":"git-send-email 2.17.1","Subject":"[libcamera-devel] [PATCH RFC 0/2] Test Status Refactoring","X-BeenThere":"libcamera-devel@lists.libcamera.org","X-Mailman-Version":"2.1.23","Precedence":"list","List-Id":"<libcamera-devel.lists.libcamera.org>","List-Unsubscribe":"<https://lists.libcamera.org/options/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=unsubscribe>","List-Archive":"<https://lists.libcamera.org/pipermail/libcamera-devel/>","List-Post":"<mailto:libcamera-devel@lists.libcamera.org>","List-Help":"<mailto:libcamera-devel-request@lists.libcamera.org?subject=help>","List-Subscribe":"<https://lists.libcamera.org/listinfo/libcamera-devel>,\n\t<mailto:libcamera-devel-request@lists.libcamera.org?subject=subscribe>","X-List-Received-Date":"Mon, 14 Jan 2019 09:54:21 -0000"},"content":"To start moving towards using a TAP (Test Anything Protocol) [0],\nadd a TestStatus class and enumerations for our current test return\nvaluese.\n\nThe TestStatus class is subclassed to provide each return status value\nand is currently set as:\n  TestStatusFail,\n  TestStatusSkip,\n  TestStatusPass,\n\nTo prevent collision with the existing TestFail, TestSkip, TestPass.\n\nIf this series progresses and is accepted, I would prefer to use our\ncurrent names (TestPass, not TestStatusPass) and would thus rename the\nTestStatus subclasses.\n\nAnyway, the goal is that every test status should have a description\nalong with it's status so that a reason for any fail or skip can be\nrecorded.\n\n\nTwo helpers are shown \"is()\" and \"isnt()\" based on the naming from libtap++ [1]\n\nI'm not tied to these names, they were just a starting point for adding\n'value check' tests.\n\n\n[0] https://testanything.org/\n[1] https://github.com/cbab/libtappp\n\nKieran Bingham (2):\n  test: Add TestStatus classes\n  test: Provide TestStatus validation tests\n\n test/libtest/meson.build     |  1 +\n test/libtest/test.h          |  2 ++\n test/libtest/test_status.cpp | 42 +++++++++++++++++++++++++++\n test/libtest/test_status.h   | 55 ++++++++++++++++++++++++++++++++++++\n test/meson.build             |  1 +\n test/test_status.cpp         | 52 ++++++++++++++++++++++++++++++++++\n 6 files changed, 153 insertions(+)\n create mode 100644 test/libtest/test_status.cpp\n create mode 100644 test/libtest/test_status.h\n create mode 100644 test/test_status.cpp"}