Skip to content

test: cover admin issue E2E workflows - #758

Merged
DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
HasToBeJames:test/admin-issue-e2e
Oct 22, 2026
Merged

DeFiVC merged 1 commit into
ChainLearnOfficial:mainfrom
HasToBeJames:test/admin-issue-e2e

Conversation

@HasToBeJames

@HasToBeJames HasToBeJames commented Oct 22, 2026 •

Copy link
Copy Markdown

Description

Adds requested E2E coverage for the four assigned admin API issues and secures auth-block administration behind authentication and admin authorization.

Closes #658
Closes #659
Closes #660
Closes #661

Changes proposed

What were you told to do?

  • Cover administrator auth-block listing, clearing, and enforcement for blocked authentication requests.
  • Cover saving and publishing admin course drafts.
  • Cover enrollment trend date ranges, aggregation, and cache behavior.
  • Cover quiz history aggregation, superseded attempts, and cache behavior.

What did I do?

Admin access and course drafts

  • Added E2E cases for authenticated admin-only auth-block routes and clear/list behavior.
  • Added E2E cases for draft save visibility and publishing a completed course.

Course analytics

  • Added E2E cases for enrollment trend filtering/grouping and cached results.
  • Added E2E cases for quiz history calculations, excluding superseded submissions, and cached results.

Check List

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title and description of the PR explain the approach.
  • I am making a pull request against the main branch.
  • My commit message style matches the requested structure.
  • The implementation was reviewed and confirmed to work as intended.
  • I am only making changes relevant to the requested issues.

Screenshots / Validation Evidence

  • npm run typecheck failed because of existing repository-wide TypeScript errors in config, quiz, rewards, and other modules; no clean type-check is claimed.
  • npm run lint completed with 0 errors and existing warnings.
  • E2E suites could not start because local Postgres/Redis and the required test environment values are not configured.

@drips-wave

drips-wave Bot commented Oct 22, 2026

Copy link
Copy Markdown

@HasToBeJames Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@HasToBeJames HasToBeJames changed the title test(admin): cover admin issue E2E workflows test: cover admin issue E2E workflows Oct 22, 2026
@DeFiVC
DeFiVC merged commit c4b251c into ChainLearnOfficial:main Oct 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants