Skip to content

Commit

Permalink
Fixed SDK tests to use new API overloads
Browse files Browse the repository at this point in the history
  • Loading branch information
MarkSeufert committed Dec 16, 2020
1 parent 1646f19 commit d10b065
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions sdk/test/logs/logger_sdk_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ TEST(LoggerSDK, LogToNullProcessor)
auto logger = lp->GetLogger("logger");

// Log a sample log record to a nullptr processor
logger->Log("Test log");
logger->Debug("Test log");
}

class MockProcessor final : public LogProcessor
Expand All @@ -56,11 +56,11 @@ class MockProcessor final : public LogProcessor
// Cast the recordable received into a concrete LogRecord type
auto copy = std::shared_ptr<LogRecord>(static_cast<LogRecord *>(record.release()));

// Copy over the received log record's name, body, timestamp fields over to the recordable
// Copy over the received log record's severity, name, and body fields over to the recordable
// passed in the constructor
record_received_->SetSeverity(copy->GetSeverity());
record_received_->SetName(copy->GetName());
record_received_->SetBody(copy->GetBody());
record_received_->SetTimestamp(copy->GetTimestamp());
}
bool ForceFlush(std::chrono::microseconds timeout = std::chrono::microseconds(0)) noexcept
{
Expand Down Expand Up @@ -92,10 +92,9 @@ TEST(LoggerSDK, LogToAProcessor)
ASSERT_EQ(processor, lp->GetProcessor());

// Check that the recordable created by the Log() statement is set properly
opentelemetry::core::SystemTimestamp now(std::chrono::system_clock::now());
logger->Log(opentelemetry::logs::Severity::kWarn, "Message", now);
logger->Log(opentelemetry::logs::Severity::kWarn, "Log Name", "Log Message");

ASSERT_EQ(shared_recordable->GetSeverity(), opentelemetry::logs::Severity::kWarn);
ASSERT_EQ(shared_recordable->GetBody(), "Message");
ASSERT_EQ(shared_recordable->GetTimestamp().time_since_epoch(), now.time_since_epoch());
ASSERT_EQ(shared_recordable->GetName(), "Log Name");
ASSERT_EQ(shared_recordable->GetBody(), "Log Message");
}

0 comments on commit d10b065

Please sign in to comment.