cmake: add LIBOCPP_ prefix to library-own options

.github/CODEOWNERS

@@ -0,0 +1,7 @@
+* @pietfried @hikinggrass @marcemmers @maaikez
+
+/config/v16/ @pietfried @hikinggrass
+/include/ocpp/v16/ @pietfried @hikinggrass
+/lib/ocpp/v16/ @pietfried @hikinggrass
+
+/.github/ @pietfried @hikinggrass

.github/ISSUE_TEMPLATE/bug_report.yaml

@@ -0,0 +1,36 @@
+name: Bug Report
+description: Thanks for taking the time to fill out this bug report!
+
+body:
+  - type: dropdown
+    id: affected-ocpp-version
+    attributes:
+      label: OCPP Version
+      description: Which OCPP is affected? Mark both if applicable.
+      options:
+        - OCPP1.6
+        - OCPP2.0.1
+      multiple: true
+    validations:
+      required: true
+
+  - type: textarea
+    id: description
+    attributes:
+      label: Describe the bug
+      description: A clear and concise description of what the bug is. If applicable, add screenshots or logfiles to help explain your problem.
+    validations:
+      required: true
+
+  - type: textarea
+    id: reproduction
+    attributes:
+      label: To Reproduce
+      description: |
+        If applicable describe the steps to and additional information to reproduce the behavior like message communication between CSMS, used configuration(s), compile options and your system information
+
+  - type: textarea
+    id: other
+    attributes:
+      label: Anything else?
+      placeholder: Add any other context about the bug report here.

.github/ISSUE_TEMPLATE/feature_request.yaml

@@ -0,0 +1,39 @@
+name: Feature Request
+description: Thanks for taking the time to fill out this feature request!
+
+body:
+  - type: dropdown
+    id: affected-ocpp-version
+    attributes:
+      label: OCPP Version
+      description: For which OCPP do you suggest this feature? Mark both if applicable.
+      options:
+        - OCPP1.6
+        - OCPP2.0.1
+      multiple: true
+    validations:
+      required: true
+
+  - type: textarea
+    id: description
+    attributes:
+      label: Describe the problem
+      description: What problem is your feature request targeting and why is it a problem? Please describe.
+      placeholder: |
+        A clear and concise description of what the problem is.
+    validations:
+      required: true
+
+  - type: textarea
+    id: solution
+    attributes:
+      label: Describe your solution
+      description: Describe the solution you'd like
+      placeholder: |
+        A clear and concise description of what you want to happen.
+
+  - type: textarea
+    id: other
+    attributes:
+      label: Additional context
+      placeholder: Add any other context about the feature request here.

.github/pull_request_template.md

@@ -0,0 +1,9 @@
+## Describe your changes
+
+## Issue ticket number and link
+
+## Checklist before requesting a review
+- [ ] I have performed a self-review of my code
+- [ ] I have made corresponding changes to the documentation
+- [ ] I read the [contribution documentation](https://github.com/EVerest/EVerest/blob/main/CONTRIBUTING.md) and made sure that my changes meet its requirements
+

CMakeLists.txt

@@ -1,7 +1,7 @@
 cmake_minimum_required(VERSION 3.14)
 
 project(ocpp
-    VERSION 0.9.7
+    VERSION 0.9.8
     DESCRIPTION "A C++ implementation of the Open Charge Point Protocol"
     LANGUAGES CXX
 )
@@ -11,10 +11,16 @@ find_package(everest-cmake 0.1 REQUIRED
 )
 
 # options
-option(BUILD_TESTING "Run unit tests" OFF)
+
 option(CMAKE_RUN_CLANG_TIDY "Run clang-tidy" OFF)
-option(LIBOCPP_BUILD_EXAMPLES "Build charge_point and central_system binaries." OFF)
-option(OCPP_INSTALL "Install the library (shared data might be installed anyway)" ${EVC_MAIN_PROJECT})
+option(${PROJECT_NAME}_BUILD_EXAMPLES "Build charge_point and central_system binaries." OFF)
+option(${PROJECT_NAME}_INSTALL "Install the library (shared data might be installed anyway)" ${EVC_MAIN_PROJECT})
+option(${PROJECT_NAME}_BUILD_TESTING "Build unit tests, used if included as dependency" OFF)
+option(BUILD_TESTING "Build unit tests, used if standalone project" OFF)
+
+if((${CMAKE_PROJECT_NAME} STREQUAL ${PROJECT_NAME} OR ${PROJECT_NAME}_BUILD_TESTING) AND BUILD_TESTING)
+    set(LIBOCPP_BUILD_TESTING ON)
+endif()
 
 # dependencies
 find_package(Boost COMPONENTS program_options regex system thread REQUIRED)
@@ -25,7 +31,7 @@ if(NOT DISABLE_EDM)
     evc_setup_edm()
 
     # In EDM mode, we can't install exports (because the dependencies usually do not install their exports)
-    set(OCPP_INSTALL OFF)
+    set(${PROJECT_NAME}_INSTALL OFF)
 
     # FIXME (aw): websocketpp doesn't play well with EDM/CPM
     add_library(websocketpp::websocketpp INTERFACE IMPORTED)
@@ -51,7 +57,7 @@ add_subdirectory(config)
 
 
 # packaging
-if (OCPP_INSTALL)
+if (${PROJECT_NAME}_INSTALL)
     install(
         TARGETS ocpp
         EXPORT ocpp-targets
@@ -103,8 +109,9 @@ if(CMAKE_RUN_CLANG_TIDY)
     -export-fixes=clang-tidy-fixes.yaml)
 endif()
 
-if(BUILD_TESTING)
+if(LIBOCPP_BUILD_TESTING)
     include(CTest)
+    enable_testing()
     add_subdirectory(tests)
 endif()
 

README.md

@@ -1,12 +1,20 @@
 # C++ implementation of OCPP
 ![Github Actions](https://github.com/EVerest/libocpp/actions/workflows/build_and_test.yaml/badge.svg)
 
-This is a C++ library implementation of OCPP for version 1.6 (https://www.openchargealliance.org/protocols/ocpp-16/) and 2.0.1 (https://www.openchargealliance.org/protocols/ocpp-201/). It enables charging stations to communicate with cloud backends for remote control, monitoring and billing of charging processes.
+This is a C++ library implementation of OCPP for version 1.6 and 2.0.1
+(see [OCPP protocols at OCA website](https://openchargealliance.org/protocols/open-charge-point-protocol/)).
+
+It enables charging stations to communicate with cloud backends for remote
+control, monitoring and billing of charging processes.
 
 Libocpp can be used for the communication of one charging station and multiple EVSE using a single websocket connection.
 
 Libocpp provides a complete implementation of OCPP 1.6. The implementation of OCPP 2.0.1 is currently under development.
 
+## Get Involved
+
+See the [COMMUNITY.md](https://github.com/EVerest/EVerest/blob/main/COMMUNITY.md) and [CONTRIBUTING.md](https://github.com/EVerest/EVerest/blob/main/CONTRIBUTING.md) of the EVerest project to get involved.
+
 ## Table of contents
 - [C++ implementation of OCPP](#c-implementation-of-ocpp)
   - [Table of contents](#table-of-contents)
@@ -523,10 +531,20 @@
 
 ## Unit testing
 
-GTest is required for building the test cases target.
-To build the target and run the tests you can reference the script `.ci/build-kit/install_and_test.sh`.
-The script allows the GitHub Actions runner to execute.
-Local testing is still in progress.
+If you want to run the unit tests in the tests subdirectory: install the needed dependencies.
+For Debian GNU/Linux 11 you can install it like this:
+
+```bash
+sudo apt install libgtest-dev lcov
+python3 -m pip install gcovr
+```
+
+Run the unit tests
+
+```bash
+  cmake .. -DBUILD_TESTING=ON
+  make test
+```
 
 ## Building with FetchContent instead of EDM
 In [doc/build-with-fetchcontent](doc/build-with-fetchcontent) you can find an example how to build libocpp with FetchContent instead of EDM.

config/v201/component_schemas/custom/Connector_1_1.json

@@ -35,7 +35,7 @@
       ],
       "description": "Component exists",
       "type": "boolean",
-      "default": "0"
+      "default": false
     },
     "ChargeProtocol": {
       "variable_name": "ChargeProtocol",

config/v201/component_schemas/custom/Connector_2_1.json

@@ -35,7 +35,7 @@
       ],
       "description": "Component exists",
       "type": "boolean",
-      "default": "0"
+      "default": false
     },
     "ChargeProtocol": {
       "variable_name": "ChargeProtocol",

config/v201/component_schemas/custom/EVSE_1.json

@@ -51,7 +51,7 @@
       ],
       "description": "Component exists",
       "type": "boolean",
-      "default": "0"
+      "default": false
     },
     "EvseId": {
       "variable_name": "EvseId",

config/v201/component_schemas/custom/EVSE_2.json

@@ -51,7 +51,7 @@
       ],
       "description": "Component exists",
       "type": "boolean",
-      "default": "0"
+      "default": false
     },
     "EvseId": {
       "variable_name": "EvseId",

config/v201/component_schemas/standardized/AlignedDataCtrlr.json

@@ -16,7 +16,7 @@
           "mutability": "ReadWrite"
         }
       ],
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "AlignedDataCtrlrAvailable": {
@@ -32,7 +32,7 @@
         }
       ],
       "description": "If this variable reports a value of true, Clock-Aligned Data is supported.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "AlignedDataInterval": {
@@ -97,7 +97,7 @@
         }
       ],
       "description": "If set to true, the Charging Station SHALL include signed meter values in the SampledValueType in the MeterValuesRequest to the CSMS.",
-      "default": "0",
+      "default": false,
       "type": "boolean"
     },
     "AlignedDataTxEndedInterval": {

config/v201/component_schemas/standardized/AuthCacheCtrlr.json

@@ -17,7 +17,7 @@
         }
       ],
       "description": "Authorization caching is available, but not necessarily enabled.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "AuthCacheCtrlrEnabled": {
@@ -33,7 +33,7 @@
         }
       ],
       "description": "If set to true, Authorization caching is enabled.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "AuthCacheLifeTime": {

config/v201/component_schemas/standardized/AuthCtrlr.json

@@ -16,7 +16,7 @@
           "mutability": "ReadWrite"
         }
       ],
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "AdditionalInfoItemsPerMessage": {
@@ -47,7 +47,7 @@
         }
       ],
       "description": "Whether a remote request to start a transaction in the form of RequestStartTransactionRequest message should be authorized beforehand like a local action to start a transaction.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "LocalAuthorizeOffline": {
@@ -63,7 +63,7 @@
         }
       ],
       "description": "Whether the Charging Station, when Offline, will start a transaction for locally-authorized identifiers,",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "LocalPreAuthorize": {
@@ -79,7 +79,7 @@
         }
       ],
       "description": "Whether the Charging Station, when online, will start a transaction for locally-authorized identifiers without waiting for or requesting an AuthorizeResponse from the CSMS.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "MasterPassGroupId": {
@@ -110,7 +110,7 @@
         }
       ],
       "description": "Support for unknown offline transactions.",
-      "default": "1",
+      "default": true,
       "type": "boolean"
     },
     "DisableRemoteAuthorization": {

config/v201/component_schemas/standardized/InternalCtrlr.json

@@ -16,7 +16,7 @@
                   "mutability": "ReadWrite"
               }
           ],
-          "default": "1",
+          "default": true,
           "type": "boolean"
       },
       "ChargePointId": {
@@ -240,7 +240,7 @@
                   "mutability": "ReadOnly"
               }
           ],
-          "default": "1",
+          "default": true,
           "type": "boolean"
       },
       "LogMessages": {
@@ -255,7 +255,7 @@
                   "mutability": "ReadOnly"
               }
           ],
-          "default": "1",
+          "default": true,
           "type": "boolean"
       },
       "LogMessagesFormat": {
@@ -338,7 +338,7 @@
               }
           ],
           "description": "Use default verify paths for validating CSMS server certificate",
-          "default": "1",
+          "default": true,
           "type": "boolean"
       },
       "OcspRequestInterval": {
@@ -483,7 +483,7 @@
                 "mutability": "ReadOnly"
             }
         ],
-        "default": "0",
+        "default": false,
         "type": "boolean"
       },
       "MessageQueueSizeThreshold": {
@@ -537,6 +537,22 @@
           "description": "List of criteria supported for a get custom report. Enabled,Active,Problem,Available",
           "default": "Enabled,Active,Problem,Available",
           "type": "string"
+      },
+      "RoundClockAlignedTimestamps": {
+          "variable_name": "RoundClockAlignedTimestamps",
+          "characteristics": {
+              "supportsMonitoring": false,
+              "dataType": "boolean"
+          },
+          "attributes": [
+              {
+                  "type": "Actual",
+                  "mutability": "ReadOnly"
+              }
+          ],
+          "description": "If enabled the metervalues configured with the AlignedDataCtrlr will be rounded to the exact time intervals",
+          "default": false,
+          "type": "boolean"
       }
   },
   "required": [