Skip to content

Commit 6177d0c

Browse files
martinzinkCopilot
andcommitted
Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 335d6eb commit 6177d0c

4 files changed

Lines changed: 6 additions & 5 deletions

File tree

extension-framework/cpp-extension-lib/mocklib/include/MockLogger.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ class MockLogger : public core::logging::Logger {
2727
public:
2828
void set_max_log_size(int) override {}
2929
void log_string(const core::logging::LOG_LEVEL level, std::string s) override { logs_[level].emplace_back(std::move(s)); }
30-
[[nodiscard]] bool should_log(const core::logging::LOG_LEVEL level) override { return level > log_level_; }
30+
[[nodiscard]] bool should_log(const core::logging::LOG_LEVEL level) override { return level >= log_level_; }
3131
[[nodiscard]] core::logging::LOG_LEVEL level() const override { return log_level_; }
3232

3333
core::logging::LOG_LEVEL log_level_ = core::logging::LOG_LEVEL::trace;

extension-framework/cpp-extension-lib/mocklib/include/MockProcessSession.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,8 @@ class MockProcessSession : public api::core::ProcessSession {
5151
MockProcessSession() = default;
5252
MockProcessSession(const MockProcessSession&) = delete;
5353
MockProcessSession& operator=(const MockProcessSession&) = delete;
54-
MockProcessSession(const MockProcessSession&&) = delete;
55-
MockProcessSession& operator=(const MockProcessSession&&) = delete;
54+
MockProcessSession(MockProcessSession&&) = delete;
55+
MockProcessSession& operator=(MockProcessSession&&) = delete;
5656

5757
~MockProcessSession() override;
5858

extension-framework/cpp-extension-lib/mocklib/include/MockStreams.h

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919

2020
#include <algorithm>
2121
#include <cstddef>
22+
#include <cstring>
2223
#include <span>
2324
#include <vector>
2425

extension-framework/cpp-extension-lib/mocklib/src/mock-minifi-c.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,13 +46,13 @@ MinifiBool MinifiProcessContextHasNonEmptyProperty(MinifiProcessContext*, Minifi
4646
MinifiStatus MinifiProcessContextGetControllerService(MinifiProcessContext*, MinifiStringView, MinifiStringView, MinifiControllerService**) {
4747
throw std::runtime_error("Not implemented");
4848
}
49-
void MinifiProcessContextGetDynamicProperties(MinifiProcessContext*,
49+
void MinifiProcessContextGetDynamicProperties(MinifiProcessContext*, MinifiFlowFile*,
5050
void (*)(void* user_ctx, MinifiStringView dynamic_property_name, MinifiStringView dynamic_property_value), void*) {
5151
throw std::runtime_error("Not implemented");
5252
}
5353

5454
MinifiStatus MinifiProcessContextGetSslData(MinifiProcessContext*, MinifiStringView,
55-
void (*)(void* user_ctx, const MinifiSslData& ssl_data), void*) {
55+
void (*)(void* user_ctx, const MinifiSslData* ssl_data), void*) {
5656
throw std::runtime_error("Not implemented");
5757
}
5858

0 commit comments

Comments
 (0)