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

fix: Employee Transfer and Project Profitability test cases #28633

Merged
merged 3 commits into from
Nov 30, 2021

Conversation

ruchamahabal
Copy link
Member

@ruchamahabal ruchamahabal commented Nov 30, 2021

image

image

@ruchamahabal ruchamahabal changed the title fix: Employee Transfer test cases fix: Employee Transfer and Project Profitability test cases Nov 30, 2021
@codecov
Copy link

codecov bot commented Nov 30, 2021

Codecov Report

Merging #28633 (8338c43) into develop (aeb62af) will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #28633      +/-   ##
===========================================
+ Coverage    55.25%   55.28%   +0.02%     
===========================================
  Files         1121     1121              
  Lines        66746    66746              
===========================================
+ Hits         36883    36900      +17     
+ Misses       29863    29846      -17     
Impacted Files Coverage Δ
...work_order_stock_report/work_order_stock_report.py 94.11% <0.00%> (-5.89%) ⬇️
erpnext/education/doctype/student/student.py 73.68% <0.00%> (-3.16%) ⬇️
...ion/doctype/course_enrollment/course_enrollment.py 44.00% <0.00%> (-2.00%) ⬇️
...eorder_level/itemwise_recommended_reorder_level.py 92.30% <0.00%> (-1.93%) ⬇️
...ctype/accounting_dimension/accounting_dimension.py 64.06% <0.00%> (-1.57%) ⬇️
...e/period_closing_voucher/period_closing_voucher.py 88.23% <0.00%> (-1.48%) ⬇️
.../report/stock_projected_qty/stock_projected_qty.py 86.84% <0.00%> (-1.32%) ⬇️
erpnext/projects/doctype/timesheet/timesheet.py 78.45% <0.00%> (-0.82%) ⬇️
...xt/accounts/doctype/sales_invoice/sales_invoice.py 78.09% <0.00%> (-0.70%) ⬇️
...next/accounts/doctype/subscription/subscription.py 81.74% <0.00%> (-0.57%) ⬇️
... and 16 more

@ruchamahabal ruchamahabal merged commit d0f4f03 into frappe:develop Nov 30, 2021
@ruchamahabal
Copy link
Member Author

@Mergifyio backport version-13-hotfix version-13-pre-release

mergify bot pushed a commit that referenced this pull request Nov 30, 2021
* fix: Employee Transfer testcases

* fix: Project Profitability test case

(cherry picked from commit d0f4f03)
mergify bot pushed a commit that referenced this pull request Nov 30, 2021
* fix: Employee Transfer testcases

* fix: Project Profitability test case

(cherry picked from commit d0f4f03)
@mergify
Copy link
Contributor

mergify bot commented Nov 30, 2021

backport version-13-hotfix version-13-pre-release

✅ Backports have been created

ruchamahabal added a commit that referenced this pull request Nov 30, 2021
…28639)

* fix: Employee Transfer testcases

* fix: Project Profitability test case

(cherry picked from commit d0f4f03)

Co-authored-by: Rucha Mahabal <ruchamahabal2@gmail.com>
ruchamahabal added a commit that referenced this pull request Nov 30, 2021
…28640)

* fix: Employee Transfer testcases

* fix: Project Profitability test case

(cherry picked from commit d0f4f03)

Co-authored-by: Rucha Mahabal <ruchamahabal2@gmail.com>
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
…8633)

* fix: Employee Transfer testcases

* fix: Project Profitability test case
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
…8633)

* fix: Employee Transfer testcases

* fix: Project Profitability test case
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant