انتقل إلى هنا من https://github.com/sebastianbergmann/phpunit/pull/3335#issuecomment -428570649.
يستبدل أيضًا برقم # 3180.
ستؤدي إزالة المعلمة $maxDepth
من assertEquals()
وما إلى ذلك إلى كسر جميع الاختبارات التي تستخدم assertEquals()
مع المعلمات الاختيارية التي تلي $maxDepth
.
الطريقة الوحيدة لتنظيف assertEquals()
التي {نأمل | ربما} تسبب مشاكل أقل ستكون التنظيف بشكل أكثر جذرية وليس فقط إزالة $maxDepth
ولكن أيضًا جميع المعلمات الاختيارية التي تأتي بعد ذلك. هذا ، أيضًا ، لا يروق لك حقًا.
كل هذه المعلمات الاختيارية كان يجب ألا تتم إضافتها مطلقًا ، فهي تصنع لواجهة برمجة تطبيقات رهيبة. لست متأكدًا ، على الرغم من ذلك ، من فائدة تنظيف assertEquals()
وما إلى ذلك عن طريق إزالة هذه المعلمات الاختيارية وإضافة طرق تأكيد أكثر تحديدًا بدلاً منها تفوق آلام كسر التوافق.
المعلمة الوحيدة التي استخدمتها أحيانًا هي حالة التجاهل. لم تستخدم أي شيء آخر.
يبدو أن الترحيل إلى تأكيد مخصص لذلك هو مسار ترقية معقول بالنسبة لي.
حلت محلها # 3341.
التعليق الأكثر فائدة
ستؤدي إزالة المعلمة
$maxDepth
منassertEquals()
وما إلى ذلك إلى كسر جميع الاختبارات التي تستخدمassertEquals()
مع المعلمات الاختيارية التي تلي$maxDepth
.الطريقة الوحيدة لتنظيف
assertEquals()
التي {نأمل | ربما} تسبب مشاكل أقل ستكون التنظيف بشكل أكثر جذرية وليس فقط إزالة$maxDepth
ولكن أيضًا جميع المعلمات الاختيارية التي تأتي بعد ذلك. هذا ، أيضًا ، لا يروق لك حقًا.كل هذه المعلمات الاختيارية كان يجب ألا تتم إضافتها مطلقًا ، فهي تصنع لواجهة برمجة تطبيقات رهيبة. لست متأكدًا ، على الرغم من ذلك ، من فائدة تنظيف
assertEquals()
وما إلى ذلك عن طريق إزالة هذه المعلمات الاختيارية وإضافة طرق تأكيد أكثر تحديدًا بدلاً منها تفوق آلام كسر التوافق.