Skip to content

Add test coverage for rulemaking load#6582

Merged
hcaofec merged 1 commit into
developfrom
feature/add_autotest_load_rulemaking
May 12, 2026
Merged

Add test coverage for rulemaking load#6582
hcaofec merged 1 commit into
developfrom
feature/add_autotest_load_rulemaking

Conversation

@fec-jli
Copy link
Copy Markdown
Contributor

@fec-jli fec-jli commented Apr 29, 2026

Summary (required)

Add test coverage for rulemaking load function in /webservices/legal/rulemaking_docs/rulemaking.py

Required reviewers

1-2 dev

Impacted areas of the application

Load rulemaking to OpenSearch

How to test

1)Checkout branch
2)pytest

@fec-jli fec-jli force-pushed the feature/add_autotest_load_rulemaking branch from 1e88ab7 to 4237ecb Compare April 29, 2026 16:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.51%. Comparing base (e4ea7d9) to head (4237ecb).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #6582      +/-   ##
===========================================
+ Coverage    85.50%   86.51%   +1.01%     
===========================================
  Files           99       99              
  Lines        10484    10484              
===========================================
+ Hits          8964     9070     +106     
+ Misses        1520     1414     -106     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@pkfec pkfec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, thanks @fec-jli

Copy link
Copy Markdown
Contributor

@hcaofec hcaofec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks for adding the test coverage. @fec-jli

@hcaofec hcaofec merged commit 32cd542 into develop May 12, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Add rulemaking load function test

3 participants