Replace Flask-BabelEx with Flask-Babel #2460
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
samuelhwilliams
force-pushed
the
switch-to-flask-babel
branch
from
179bc94
to
097275c
Compare
This allows us to have (almost) a single place where we have to think about Babel (in the `babel`) fixture. This should make it easier for us to, in the future, run tests without babel or flask-babel installed, to validate flask-admin for users without that package. This is important because the `flask_admin.babel` module has two clearly different codepaths depending on whether the package is installed. Importantly, because of the fixture chaining (`admin` requires `babel`), it also means that `babel` (if available) is always configured for every flask app instance for every test.
samuelhwilliams
force-pushed
the
switch-to-flask-babel
branch
from
097275c
to
4c723b2
Compare
Adds a `flask_babel` mark to any tests that explicitly require `Flask-Babel` to be installed. Configures tox and GitHub Actions to run a test suite with Flask-Babel removed. This validates a codepath through `flask_admin.babel`, which should essentially be passthrough/noop behaviour if Flask-Babel is unavailable.
samuelhwilliams
force-pushed
the
switch-to-flask-babel
branch
from
dbca159
to
3153466
Compare
16 tasks
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Flask-BabelEx has been deprecated and archived. The original Flask-Babel is now alive and active again, so it makes sense to convert Flask-Admin back to using that library instead from v2.0.0.
Although Flask-Admin has claimed to support Flask-Babel at the same time as Flask-BabelEx, there was one semi-significant issue that I came across with only Flask-Babel: if the package is installed, but Flask-Babel is never initialised against the Flask app, then Flask-Admin will crash. Flask-BabelEx would silently fall back to the untranslated strings, but Flask-Admin does a hard lookup on
app.extensions['babel']and dies.This led to a series of test changes in order to make sure that Flask-Babel is correctly set up for all tests. If
Flask-Babelis not installed, this is a noop, which should also make it easier to run tests as a user that does not use Flask-Babel - something that does not currently happen, leaving a bit of a gap in our test coverage asflask_admin.babelhas two very distinct code paths pivoted on whether the package is installed (note installed, not initialised against a Flask app)closes: #2447