Skip to content

Conversation

bonjourmauko
Copy link
Member

@bonjourmauko bonjourmauko commented Sep 24, 2024

Fixes #1316
Dependas on #1309

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
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.

assert_near behaves inconsistenly with str values

3 participants