Skip to content

An initial spec setup for getting things going.#9

Merged
russ merged 3 commits intomasterfrom
8-specs
Aug 14, 2025
Merged

An initial spec setup for getting things going.#9
russ merged 3 commits intomasterfrom
8-specs

Conversation

@russ
Copy link
Owner

@russ russ commented Aug 14, 2025

No assertions yet. I need to probably put a database in play to see if jobs are being ran in specs.

No assertions yet. I need to probably put a database in play to see if
jobs are being ran in specs.
@russ russ linked an issue Aug 14, 2025 that may be closed by this pull request
Closed
@russ russ requested a review from Copilot August 14, 2025 19:41
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 establishes an initial test specification setup for the TJLavin project, introducing basic testing infrastructure for a job queue system that uses AMQP for message queuing.

Key changes:

  • Added core dependencies (JSON and AMQP client) to the main module
  • Created a test worker helper class to facilitate testing of queued jobs
  • Set up basic test infrastructure with spec helper and initial job enqueueing test
  • Configured GitHub Actions CI workflow with formatting, linting, and testing jobs

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/tj_lavin.cr Adds JSON and AMQP client dependencies to support job queue functionality
spec/support/my_worker_helper.cr Creates a test worker class extending TJLavin::QueuedJob for testing purposes
spec/spec_helper.cr Configures test environment with logging, AMQP URL, and TJLavin setup
spec/job_spec.cr Implements basic job enqueueing test with placeholder for future assertions
.github/workflows/ci.yml Sets up CI pipeline with format checking, linting, and specs using LavinMQ service

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

@russ russ merged commit f94b490 into master Aug 14, 2025
3 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.

Specs

1 participant