Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: additional tests recommendation for sub contracting order (backport #2783) #2813

Merged
merged 3 commits into from
Dec 5, 2024

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Dec 5, 2024


This is an automatic backport of pull request #2783 done by [Mergify](https://mergify.com).

* test: coverage for onload method

* test: validate doc references obtained from function call

* fix: minor refactor

* test: validate workflow for unregistered company

* test: validate stock entry for different purpose and stock entry validations

* fix: validate standalone stock entries

* fix: better error strings

* fix: minor fix

* fix: company gstin instead of bill from gstin

* fix: validate e-Waybill expiry

(cherry picked from commit f10e559)

# Conflicts:
#	india_compliance/gst_india/overrides/test_subcontracting_transaction.py
@mergify mergify bot added the conflicts label Dec 5, 2024
Copy link
Contributor Author

mergify bot commented Dec 5, 2024

Cherry-pick of f10e559 has failed:

On branch mergify/bp/version-15-hotfix/pr-2783
Your branch is up to date with 'origin/version-15-hotfix'.

You are currently cherry-picking commit f10e5595.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   india_compliance/gst_india/utils/test_e_waybill.py
	modified:   india_compliance/gst_india/utils/tests.py

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   india_compliance/gst_india/overrides/test_subcontracting_transaction.py

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@Ninad1306 Ninad1306 enabled auto-merge December 5, 2024 11:41
@Ninad1306 Ninad1306 force-pushed the mergify/bp/version-15-hotfix/pr-2783 branch from 229c93c to f5ae997 Compare December 5, 2024 12:36
@Ninad1306 Ninad1306 force-pushed the mergify/bp/version-15-hotfix/pr-2783 branch from f5ae997 to 368a8f3 Compare December 5, 2024 12:37
@Ninad1306 Ninad1306 disabled auto-merge December 5, 2024 12:37
Copy link

codecov bot commented Dec 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.94%. Comparing base (1573f40) to head (368a8f3).
Report is 17 commits behind head on version-15-hotfix.

Additional details and impacted files

Impacted file tree graph

@@                  Coverage Diff                  @@
##           version-15-hotfix    #2813      +/-   ##
=====================================================
- Coverage              59.97%   59.94%   -0.04%     
=====================================================
  Files                    123      122       -1     
  Lines                  12116    12039      -77     
=====================================================
- Hits                    7267     7217      -50     
+ Misses                  4849     4822      -27     
Files with missing lines Coverage Δ
india_compliance/gst_india/utils/tests.py 97.22% <ø> (ø)

... and 5 files with indirect coverage changes

Impacted file tree graph

@mergify mergify bot merged commit 2fc426d into version-15-hotfix Dec 5, 2024
12 checks passed
@mergify mergify bot deleted the mergify/bp/version-15-hotfix/pr-2783 branch December 5, 2024 12:50
@india-compliance-bot
Copy link
Collaborator

🎉 This PR is included in version 15.14.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants