Phpunit: تم إزالة توقعExceptionMessageRegExp () في PHPUnit 9 بدون تحذير الإهمال المعطى في PHPUnit 8
| س | أ
| -------------------- | ---------------
| إصدار PHPUnit | 9
| إصدار PHP | 7.3
| طريقة التثبيت | ملحن
ملخص
الالتزام بتقديم expectExceptionMessageMatches يقول أن expectExceptionMessageRegExp سيؤدي إلى تحذير في 9 وستتم إزالته في 10.
لكن 9.0 أزال الطريقة تمامًا بدلاً من إضافة تحذير فيها. هذا يعني أنه تمت إزالة الطريقة دون الإبلاغ عن تحذير الإهمال عند إجراء الاختبارات باستخدامها.
ولا يوجد سجل تغيير يقول إن هذه الطريقة قد تم إهمالها (لا يتحدث سجل التغيير في PHPUnit 8 عن إهمال expectExceptionMessageRegExp ويقول سجل التغيير PHPUnit 9 أنه تمت إزالته).
stof
ال 3 كومينتر
تمت إضافة expectExceptionMessageMatches في 8.4 فقط ، وهذا هو السبب في أن إعلان PHPUnit 8 لا يتحدث عن ذلك على الإطلاق.
stof
في ٦ مارس ٢٠٢٠
تم إهمال expectExceptionMessageRegExp() في PHPUnit 8.4. حقيقة أن هذا الإهمال لا يتم الإبلاغ عنه عند استخدام الطريقة بشكل مزعج ، أوافق ، ولكن هل هناك قيمة فعلية لإضافة هذا السلوك في PHPUnit 8.5؟
sebastianbergmann
في ٦ مارس ٢٠٢٠
حسنًا ، يهدف الالتزام الأولي إلى إضافة تحذير الإيقاف في 9 (راجع الالتزام المرتبط) وإزالة في 10. لهذا السبب لا يوجد تحذير في 8.4 و 8.5.
تكمن المشكلة في أن الإزالة تمت دون اتباع الخطة الأولية ، ودون إضافة تحذير الإهمال في 8.x ، مما يعني أنه لا يوجد تحذير على الإطلاق.
تتمثل قيمة التحذير في أنه يمكنني العمل على الإهمال عند رؤية التحذير (وكل ذلك مرة واحدة) ، بدلاً من الاضطرار إلى إصلاح الخطأ الفادح بسبب الطريقة التي تمت إزالتها ، وتكرار هذا لكل مكان استخدمه حتى لا أقوم بذلك تلقيت خطأ فادحًا بعد الآن (أو يمكنني البحث عن expectExceptionMessageRegExp باستخدام IDE الخاص بي بالفعل ، ولكن بعد ذلك لا تساعدني PHPUnit في الترقية وقد يبرر ذلك عدم استخدام تحذيرات الإيقاف على الإطلاق ...).
إذا أمكن ، أعتقد أنه سيكون من المنطقي إضافة التحذير في 8.5 ، نعم.
stof
في ٦ مارس ٢٠٢٠
القضايا ذات الصلة
AnmSaiful
·
4تعليقات
sebastianbergmann
·
4تعليقات
dkarlovi
·
3تعليقات
rentalhost
·
4تعليقات
edyan
·
4تعليقات
التعليق الأكثر فائدة
حسنًا ، يهدف الالتزام الأولي إلى إضافة تحذير الإيقاف في 9 (راجع الالتزام المرتبط) وإزالة في 10. لهذا السبب لا يوجد تحذير في 8.4 و 8.5.
تكمن المشكلة في أن الإزالة تمت دون اتباع الخطة الأولية ، ودون إضافة تحذير الإهمال في 8.x ، مما يعني أنه لا يوجد تحذير على الإطلاق.
تتمثل قيمة التحذير في أنه يمكنني العمل على الإهمال عند رؤية التحذير (وكل ذلك مرة واحدة) ، بدلاً من الاضطرار إلى إصلاح الخطأ الفادح بسبب الطريقة التي تمت إزالتها ، وتكرار هذا لكل مكان استخدمه حتى لا أقوم بذلك تلقيت خطأ فادحًا بعد الآن (أو يمكنني البحث عن
expectExceptionMessageRegExpباستخدام IDE الخاص بي بالفعل ، ولكن بعد ذلك لا تساعدني PHPUnit في الترقية وقد يبرر ذلك عدم استخدام تحذيرات الإيقاف على الإطلاق ...).إذا أمكن ، أعتقد أنه سيكون من المنطقي إضافة التحذير في 8.5 ، نعم.