TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int

[aspark21]

PHP8.1
Moodle 4.3 stable

New PHPUnit failures since Thursday. Seems it could be due to the core weekly release rather than the plugin update

qtype_gapfill.walkthrough_test/test_draggable_itemsqtype_gapfill\walkthrough_test::test_draggable_items 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int 
/var/www/html/question/classes/output/question_version_info.php:62 
/var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 
/var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 
/var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 
/var/www/html/question/engine/tests/helpers.php:986 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:61 
/var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_dropdownsqtype_gapfill\walkthrough_test::test_dropdowns TypeError: Cannot assign string to 
property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 
/var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 
/var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 
/var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:97 
/var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_deferred_feedback_unansweredqtype_gapfill\walkthrough_test::test_deferred_feedback_unanswered 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 
/var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 
/var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 
/var/www/html/question/engine/questionusage.php:483
/var/www/html/question/engine/tests/helpers.php:921 
/var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:123 
/var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_interactive_with_correctqtype_gapfill\walkthrough_test::test_interactive_with_correct 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 /var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:316 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_interactive_wildcard_with_correctqtype_gapfill\walkthrough_test::test_interactive_wildcard_with_correct 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 /var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:366 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_interactive_discard_duplicatesqtype_gapfill\walkthrough_test::test_interactive_discard_duplicates 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 /var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:480 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_interactive_grade_for_blankqtype_gapfill\walkthrough_test::test_interactive_grade_for_blank 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 /var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:570 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_matching_divsqtype_gapfill\walkthrough_test::test_matching_divs 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:616 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_get_aftergap_textqtype_gapfill\walkthrough_test::test_get_aftergap_text 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:642 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

qtype_gapfill.walkthrough_test/test_deferred_grade_for_blankqtype_gapfill\walkthrough_test::test_deferred_grade_for_blank 
TypeError: Cannot assign string to property core_question\output\question_version_info::$version of type int  
/var/www/html/question/classes/output/question_version_info.php:62 /var/www/html/question/engine/renderer.php:150 
/var/www/html/question/engine/renderer.php:102 /var/www/html/question/behaviour/behaviourbase.php:113 
/var/www/html/question/engine/questionattempt.php:912 /var/www/html/question/engine/questionusage.php:483 
/var/www/html/question/engine/tests/helpers.php:921 /var/www/html/question/engine/tests/helpers.php:1012 
/var/www/html/question/type/gapfill/tests/walkthrough_test.php:688 /var/www/html/lib/phpunit/classes/advanced_testcase.php:81

[marcusgreen]

Interesting, I will investigate

[aspark21]

Found this where it was fixed in core:
https://tracker.moodle.org/browse/MDL-79507
moodle/moodle@f462a48

[aspark21]

Quite likely to be due to this change
https://tracker.moodle.org/browse/MDL-80087
moodle/moodle@b7a7263

not that it helps with what a fix might look like

[marcusgreen]

I suspect it is a test failure rather than a plugin failure, but I am keen all my tests pass

[aspark21]

full test run has now completed with MDL-80087 reverted and this resolves those failing tests, so that's definitely where it's coming from

[marcusgreen]

Hi Alistair, I am not entirely sure why the unit tests ever worked under PHP 8.2. However as a result of his commit
69988aa
They now work as expected on the CI. Can you confirm it works for you

[aspark21]

Thanks Marcus!

Tests running now, results should be in in about 3.5h.