Skip to content

Conversation

@mickeyn
Copy link
Contributor

@mickeyn mickeyn commented May 4, 2025

Basic test data + initial test

@oalders oalders requested a review from Copilot October 10, 2025 17:58
Copy link
Contributor

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 basic test data and initial test infrastructure for the metacpan-ingest project. It includes setup for testing with fake CPAN data, Docker Compose configuration for testing environment, and continuous integration improvements.

  • Adds comprehensive test data including fake CPAN packages and coverage data for testing
  • Implements initial setup test that validates all major indexing operations
  • Introduces Docker Compose configuration for running tests with Elasticsearch

Reviewed Changes

Copilot reviewed 18 out of 52 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
wait-for-es.sh Shell script to wait for Elasticsearch to be available before running tests
test_data/ Fake CPAN data files for testing including package metadata and coverage data
t/00_setup.t Comprehensive setup test that validates indexing operations across all indices
metacpan_ingest.conf Adds test configuration for fake CPAN directory
log4perl_test.conf Test logging configuration
lib/MetaCPAN/Mapper.pm Enhanced to support test mode and index creation with mappings
lib/MetaCPAN/Ingest.pm Test environment detection and configuration improvements
lib/MetaCPAN/File.pm Removed deprecated payload field from suggest configuration
lib/MetaCPAN/External/Debian.pm Fixed ES constructor parameter
lib/MetaCPAN/ES.pm Test mode support and ES constructor improvements
lib/MetaCPAN/Contributor.pm ES instance management improvements
docker-compose.yml Docker Compose configuration for test environment
README.md Added build status badge
Dockerfile Added test files to container
.github/workflows/build-container.yml Updated to use Docker Compose for testing
.circleci/config.yml CircleCI configuration for comprehensive testing

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

@mickeyn mickeyn changed the title Draft: Added setup test Tests - setup + basic tests for all scripts Oct 18, 2025
@mickeyn mickeyn merged commit 2570dd0 into master Oct 19, 2025
1 check passed
@mickeyn mickeyn deleted the mickey/test branch October 19, 2025 13:03
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