Skip to content

Conversation

@Dilhasha
Copy link
Contributor

Purpose

Add basic test cases and workflows for the Moesif extension

@anuruddhal anuruddhal requested a review from Copilot August 27, 2025 04:57
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds comprehensive test coverage for the Moesif extension by introducing unit tests for Java components and Ballerina integration tests. The tests validate the core functionality of the extension including tracer provider behavior, rate limiting sampler, and configuration handling.

  • Adds Java unit tests for RateLimitingSampler and MoesifTracerProvider classes
  • Creates Ballerina test suites for tracer provider functionality with mock external method calls
  • Introduces GitHub Actions workflows for CI/CD including pull request validation, daily builds, and release publishing

Reviewed Changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/src/test/java/io/ballerina/observe/trace/moesif/sampler/RateLimitingSamplerTest.java Comprehensive unit tests for the RateLimitingSampler class covering constructor validation, sampling behavior, and parameter handling
tests/src/test/java/io/ballerina/observe/trace/moesif/MoesifTracerProviderTest.java Unit tests for MoesifTracerProvider including name validation, initialization behavior, and propagator configuration
tests/build.gradle Gradle configuration for running Java unit tests with proper dependencies and coverage reporting
native/build.gradle Updates native module build to include JUnit 5 test dependencies and configuration
ballerina/tests/tracer_provider_test.bal Ballerina test suite for tracer provider logic with mock external method calls and configuration validation
ballerina/Dependencies.toml Updated dependencies to include test framework support
ballerina-tests/ Complete integration test project structure with server tests, configuration files, and build setup
.github/workflows/ GitHub Actions workflows for CI/CD including pull request validation, builds, and publishing

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Aug 27, 2025

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@Dilhasha Dilhasha merged commit ad4d0c9 into main Aug 27, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants