Skip to content

[test]: Create report & label based on an existing query#8327

Open
rijulpoudel wants to merge 2 commits into
mainfrom
issue-8326
Open

[test]: Create report & label based on an existing query#8327
rijulpoudel wants to merge 2 commits into
mainfrom
issue-8326

Conversation

@rijulpoudel

@rijulpoudel rijulpoudel commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Fixes #8326

Summary by CodeRabbit

  • Tests
    • Added integration coverage for creating reports from existing queries.
    • Added coverage for creating labels from new queries.
    • Verified response status, report details, linked queries, formats, and generated content.

@github-actions

Copy link
Copy Markdown

Warning

One or more dependencies are approaching or past End-of-Life.
Please plan upgrades accordingly.

STATUS=WARNING
NODE_VERSION=20
NODE_CYCLE=20
EOL_DATE=2026-04-30
DAYS_REMAINING=-81

--- Node.js ---
Version: 20
EOL: 2026-04-30
Status: WARNING

STATUS=OK
PYTHON_VERSION=3.12
PYTHON_CYCLE=3.12
EOL_DATE=2028-10-31
DAYS_REMAINING=834

--- Python ---
Version: 3.12
EOL: 2028-10-31
Status: OK

STATUS=WARNING
DJANGO_VERSION=4.2
DJANGO_CYCLE=4.2
EOL_DATE=2026-04-07
DAYS_REMAINING=-104

--- Django ---
Version: 4.2
EOL: 2026-04-07
Status: WARNING


@coderabbitai

coderabbitai Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a382c45c-ffb0-4247-8c50-0b6007f8d920

📥 Commits

Reviewing files that changed from the base of the PR and between ff7a6b7 and 6e7c7aa.

📒 Files selected for processing (1)
  • specifyweb/backend/report_runner/tests.py

📝 Walkthrough

Walkthrough

Changes

Report runner creation tests

Layer / File(s) Summary
Create endpoint scenarios
specifyweb/backend/report_runner/tests.py
Replaces the arithmetic placeholder with Django integration tests for creating reports from existing queries and labels from new queries, validating HTTP responses, persisted links, mimetypes, and decoded appresource data.
🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly reflects the main change: adding tests for report and label creation from queries.
Linked Issues check ✅ Passed The new tests cover both report creation from an existing query and label creation from a new query as requested by #8326.
Out of Scope Changes check ✅ Passed The changes stay within the testing scope and do not introduce unrelated functionality.
Automatic Tests ✅ Passed PASS: The PR adds Django integration tests covering report and label creation in specifyweb/backend/report_runner/tests.py.
Testing Instructions ✅ Passed PASS — The new tests clearly target report_runner create, covering both report and label creation with the expected query, user, and mimetype assertions.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch issue-8326

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@rijulpoudel rijulpoudel added this to the 7.12.1 milestone Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📋Back Log

Development

Successfully merging this pull request may close these issues.

[test]: Create report & label based on an existing query

2 participants