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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: dynamic Fiscal Year creation in tests #28667

Merged
merged 1 commit into from
Dec 1, 2021

Conversation

ankush
Copy link
Member

@ankush ankush commented Dec 1, 2021

Few tests are failing because FY 2021-2022 doesn't exist and reports are trying to fetch FY of 1 month from today.

Root cause: FY up to 2021 were hardcoded in test records 馃ゲ so they were scheduled to fail today.
Fix: Dynamic FY for test records, from 2011 to (today + 4 years)

@gavindsouza
Copy link
Collaborator

Update .gitignore too? 馃

@ankush
Copy link
Member Author

ankush commented Dec 1, 2021

Update .gitignore too? 馃

update what exactly? 馃 馃槙

@gavindsouza
Copy link
Collaborator

Oops. My bad

@ankush ankush merged commit fdffa03 into frappe:develop Dec 1, 2021
@ankush ankush deleted the fiscal_years branch December 1, 2021 12:01
mergify bot pushed a commit that referenced this pull request Dec 1, 2021
mergify bot pushed a commit that referenced this pull request Dec 1, 2021
ankush added a commit that referenced this pull request Dec 1, 2021
(cherry picked from commit fdffa03)

Co-authored-by: Ankush Menat <ankush@frappe.io>
ankush added a commit that referenced this pull request Dec 1, 2021
(cherry picked from commit fdffa03)

Co-authored-by: Ankush Menat <ankush@frappe.io>
mergify bot pushed a commit that referenced this pull request Dec 14, 2021
(cherry picked from commit fdffa03)

# Conflicts:
#	erpnext/accounts/doctype/fiscal_year/test_fiscal_year.py
#	erpnext/accounts/doctype/fiscal_year/test_records.json
ankush added a commit that referenced this pull request Dec 14, 2021
(cherry picked from commit fdffa03)

fix: resolve conflict

fix
ankush added a commit that referenced this pull request Dec 14, 2021
(cherry picked from commit fdffa03)

fix: resolve conflict

fix
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
conncampbell pushed a commit to conncampbell/erpnext that referenced this pull request Jan 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants