Skip to content

Add CI that validates against real activity code #69

Description

@dannystaple

Problem

This repo has no tests (see #62/#63) — CI only lints (a single trailing-whitespace pre-commit hook) and checks that the package installs/builds. Since coder_dojo_common is really support glue for Coder Dojo activities (game programming, stats, maths visualisations), the most meaningful validation is running it against real activity code, not abstract unit tests.

Blocked by: #68 (need the list of activity sheets/repos before we know what to run in CI)

Goal

Once #68 has produced a reference list of activity sources, add a CI job that pulls in (or vendors copies of) representative activity code and runs it against the current coder_dojo_common, so breaking changes to the library get caught before they break a dojo session.

Approach (rough)

  • Start small: pick one or two representative activities and get them running in CI as a smoke test
  • Extend coverage incrementally per activity category, as sub-tickets:
    • Game programming activities (pgzero / games extra)
    • Stats/data activities (data extra: pandas, numpy, matplotlib, plotly)
    • Maths visualisation activities
    • Kenney asset CLI (kenney-assets command) — smoke test against a real asset pack
  • Decide how activity code gets into CI: vendored copy in this repo vs. checkout of another repo vs. script fetched from Google Docs export
  • Wire the smoke tests into on-pr-lint.yml (or a new workflow) so they run on every PR

This ticket is intentionally a placeholder/parent — split each checklist item into its own issue once #68 lands and it's clear what's actually available to test against.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions