learningequality / learningequality/kolibri

exam ordering is ambiguous

Open
#3,956 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

APP: Coach APP: Learn TAG: tech update / debt TAG: unit tests
Dominant language
Python
Stars
1.1k
Forks
972
Avg merge
7d 16h
Merged PRs (30d)
33

Description

Observed behavior

In #3954 we rely on the default behavior of the API to order the exams reverse-chronologically. However there is no guarantee in the API that this will remain the behavior.

Expected behavior

We should add 'created' (or 'modified'?) timestamps to the exams so that they can be sorted predictably.

Or, we should also add tests to verify proper sorting

User-facing consequences

sort order may break in the future

Context

0.10.0

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the exam API behavior described in #3954 and checking how exams are currently ordered. Determine whether timestamps or explicit sorting is the intended direction, then add coverage that verifies predictable ordering. Done means the ordering is defined and protected by tests.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.