Skip to content

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

Merged
CarolineDenis merged 6 commits into
mainfrom
issue-8326
Jul 28, 2026
Merged

[test]: Create report & label based on an existing query#8327
CarolineDenis merged 6 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 stored queries.
    • Added integration coverage for creating labels from newly created and renamed queries.
    • Verified successful creation, query and user associations, content types, and generated resource data.

@github-actions

github-actions Bot commented Jul 20, 2026

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=-89

--- 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=826

--- 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=-112

--- 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: 487b533d-8a0a-4ee2-a84d-b4623fad4667

📥 Commits

Reviewing files that changed from the base of the PR and between df3b8cb and 11eb1cb.

📒 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
Adds Django integration tests for report creation from an existing stored query and label creation from a renamed query, validating HTTP status, Spreport relationships, mimetypes, and decoded appresource content.
🚥 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 summarizes the main change: tests for creating a report and label from an existing query.
Linked Issues check ✅ Passed The added tests match issue #8326 by covering report and label creation from an existing query.
Out of Scope Changes check ✅ Passed The changes are limited to tests directly supporting the stated report and label creation workflow.
Automatic Tests ✅ Passed The PR adds automated tests in specifyweb/backend/report_runner/tests.py for /report_runner/create/ covering report and label creation.
Testing Instructions ✅ Passed The new backend tests clearly cover /report_runner/create/ for both jrxml/report and jrxml/label, matching the affected report/label creation flow.
✨ 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
@CarolineDenis
CarolineDenis merged commit c77009c into main Jul 28, 2026
20 checks passed
@CarolineDenis
CarolineDenis deleted the issue-8326 branch July 28, 2026 17:35
@github-project-automation github-project-automation Bot moved this from 📋Back Log to ✅Done in General Tester Board Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅Done

Development

Successfully merging this pull request may close these issues.

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

2 participants