- Issue created by @kevinquillen
- First commit to issue fork.
- Merge request !16Issue #3351967: Add kernel test for estimated read time β (Merged) created by simonbaese
- Status changed to Needs review
about 1 year ago 7:09pm 20 October 2023 - π©πͺGermany simonbaese Berlin
Added a kernel test for the basic functionality. Does not cover translations of nodes, yet. Can you please list some edge cases that were relevant in former issues?
- πΊπΈUnited States mtalt Maryland
@simonbaese Thanks again for your contributions.
The main case that caused issues was on creation of new nodes where the core links field was attempted to be rendered, but neither a node ID nor revision ID was available yet and the node couldn't be loaded. The issues involving translations was due to the same error.
Potential tests:
1. Creation of new node
2. Update the node
3. Create a new translation and test that both the translated estimated read time and source estimated read time are correct
4. Update a translation - Assigned to simonbaese
- Status changed to Needs work
about 1 year ago 4:12pm 21 October 2023 - Status changed to Postponed
about 1 year ago 1:48pm 22 October 2023 - π©πͺGermany simonbaese Berlin
The test is failing with the discussed cases, because there is another bug. Please see π Creating multi-language entities programmatically does not estimate read time for all translations Active . After resolving the issue, the tests will go green.
- Status changed to RTBC
9 months ago 5:22pm 22 February 2024 -
mtalt β
committed f0179faf on 1.0.x authored by
simonbaese β
Issue #3351967 by simonbaese, mtalt, kevinquillen: Add kernel tests
-
mtalt β
committed f0179faf on 1.0.x authored by
simonbaese β
- Status changed to Fixed
9 months ago 5:26pm 22 February 2024 - πΊπΈUnited States mtalt Maryland
Thanks @simonbaese and @kevinquillen! With the fix merged from https://www.drupal.org/project/estimated_read_time/issues/3395919 π Creating multi-language entities programmatically does not estimate read time for all translations Active , the tests passed.
Automatically closed - issue fixed for 2 weeks with no activity.