Skip to content

[FEATURE] [MER-5855] Adds core support for LearningObjectiveCoverage Data Source - #6785

Open
rgachuhi wants to merge 4 commits into
masterfrom
MER-5855-learning-objective-data-source
Open

[FEATURE] [MER-5855] Adds core support for LearningObjectiveCoverage Data Source#6785
rgachuhi wants to merge 4 commits into
masterfrom
MER-5855-learning-objective-data-source

Conversation

@rgachuhi

Copy link
Copy Markdown
Contributor

Summary

  • Added Oli.Authoring.ObjectiveCoverage, a read-only application-level data source for Learning Objective coverage.
  • Implemented a compact working-publication projection with project scoping and deleted-revision filtering.
  • Added normalized hierarchy, curriculum, page, activity, coverage, detail, and search indexes.
  • Added embedded-activity-only coverage with banked activity exclusion.
  • Added telemetry, malformed-data safeguards, cycle protection, and deterministic output.
  • Added focused ExUnit coverage and Harness traceability artifacts.

Scope

This PR covers the core data layer for the Objectives Editor epic. It does not include:

  • Workspace UI integration
  • Filters or CSV export
  • Proficiency aggregation
  • Activity-bank indexing
  • Database migrations or caching

Verification

  • mix format --check-formatted

  • mix compile --warnings-as-errors

  • mix test test/oli/authoring/objective_coverage_test.exs

    • 6 tests, 0 failures
  • Harness work-item validation passed

  • Requirements implementation traceability passed

@github-actions

github-actions Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor
Warnings
⚠️ PR is large (2036 LOC changed). Consider splitting.

Risk score: 5 → risk/medium

Generated by 🚫 dangerJS against 5cfb9a8

@github-actions

Copy link
Copy Markdown
Contributor

AI Review — elixir

No issues found

@github-actions

Copy link
Copy Markdown
Contributor

AI Review — security

No issues found

@github-actions

Copy link
Copy Markdown
Contributor

AI Review — performance

No issues found

Comment thread lib/oli/authoring/objective_coverage.ex Outdated
Comment thread lib/oli/authoring/objective_coverage.ex
Comment thread lib/oli/authoring/objective_coverage.ex
Comment thread lib/oli/authoring/objective_coverage.ex Outdated
Comment thread lib/oli/authoring/objective_coverage.ex Outdated
Comment thread test/oli/authoring/objective_coverage_test.exs Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants