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 assert_near #1228

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

Test assert_near #1228

wants to merge 6 commits into from

Conversation

bonjourmauko
Copy link
Member

New features

  • Introduce openfisca_test.
    • Provides assert_near and parse to prepare tests.

Technical changes

  • Add tests to assert_near.

@bonjourmauko bonjourmauko added level:intermediate Requires average OpenFisca experience kind:feat A feature request, a feature deprecation labels Sep 24, 2024
@bonjourmauko bonjourmauko added this to the Improved testing milestone Sep 24, 2024
@bonjourmauko bonjourmauko requested review from a team September 24, 2024 09:16
@bonjourmauko bonjourmauko self-assigned this Sep 24, 2024
@coveralls
Copy link

coveralls commented Sep 24, 2024

Coverage Status

coverage: 73.526% (-1.0%) from 74.538%
when pulling d0b53e5 on fix/openfisca-test
into eb154a0 on master.

Copy link
Member

@benjello benjello left a comment

Choose a reason for hiding this comment

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

IMHO it is definitely a major bump

CHANGELOG.md Outdated
@@ -1,5 +1,17 @@
# Changelog

## 41.6.0 [#1228](https://github.com/openfisca/openfisca-core/pull/1228)
Copy link
Member

Choose a reason for hiding this comment

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

assert_near is a public tool used by many other packages.
It is a major version bump.

setup.py Outdated
@@ -67,7 +67,7 @@

setup(
name="OpenFisca-Core",
version="41.5.5",
version="41.6.0",
Copy link
Member

Choose a reason for hiding this comment

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

IMHO it should be a major bump

@bonjourmauko
Copy link
Member Author

IMHO it is definitely a major bump

Could you check, in the _parsers and _assertions files, if that corresponds to the intended behaviour?

The string evaluation works, I fixed some bugs when dealing with non-evaluated strings.

@benjello
Copy link
Member

Could you check, in the _parsers and _assertions files, if that corresponds to the intended behaviour?

Looks good to me, but for safety check you should run this on the openfisca-france test suite.
It is definitely a good crash test.

@bonjourmauko bonjourmauko marked this pull request as draft September 27, 2024 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:feat A feature request, a feature deprecation level:intermediate Requires average OpenFisca experience
Projects
Status: Reviewing
Development

Successfully merging this pull request may close these issues.

3 participants