Phpunit: إزالة السمة $ maxDepth والمعلمة وما إلى ذلك.

تم إنشاؤها على ١٠ أكتوبر ٢٠١٨  ·  4تعليقات  ·  مصدر: sebastianbergmann/phpunit

typbackward-compatibility

التعليق الأكثر فائدة

ستؤدي إزالة المعلمة $maxDepth من assertEquals() وما إلى ذلك إلى كسر جميع الاختبارات التي تستخدم assertEquals() مع المعلمات الاختيارية التي تلي $maxDepth .

الطريقة الوحيدة لتنظيف assertEquals() التي {نأمل | ربما} تسبب مشاكل أقل ستكون التنظيف بشكل أكثر جذرية وليس فقط إزالة $maxDepth ولكن أيضًا جميع المعلمات الاختيارية التي تأتي بعد ذلك. هذا ، أيضًا ، لا يروق لك حقًا.

كل هذه المعلمات الاختيارية كان يجب ألا تتم إضافتها مطلقًا ، فهي تصنع لواجهة برمجة تطبيقات رهيبة. لست متأكدًا ، على الرغم من ذلك ، من فائدة تنظيف assertEquals() وما إلى ذلك عن طريق إزالة هذه المعلمات الاختيارية وإضافة طرق تأكيد أكثر تحديدًا بدلاً منها تفوق آلام كسر التوافق.

ال 4 كومينتر

يستبدل أيضًا برقم # 3180.

ستؤدي إزالة المعلمة $maxDepth من assertEquals() وما إلى ذلك إلى كسر جميع الاختبارات التي تستخدم assertEquals() مع المعلمات الاختيارية التي تلي $maxDepth .

الطريقة الوحيدة لتنظيف assertEquals() التي {نأمل | ربما} تسبب مشاكل أقل ستكون التنظيف بشكل أكثر جذرية وليس فقط إزالة $maxDepth ولكن أيضًا جميع المعلمات الاختيارية التي تأتي بعد ذلك. هذا ، أيضًا ، لا يروق لك حقًا.

كل هذه المعلمات الاختيارية كان يجب ألا تتم إضافتها مطلقًا ، فهي تصنع لواجهة برمجة تطبيقات رهيبة. لست متأكدًا ، على الرغم من ذلك ، من فائدة تنظيف assertEquals() وما إلى ذلك عن طريق إزالة هذه المعلمات الاختيارية وإضافة طرق تأكيد أكثر تحديدًا بدلاً منها تفوق آلام كسر التوافق.

المعلمة الوحيدة التي استخدمتها أحيانًا هي حالة التجاهل. لم تستخدم أي شيء آخر.

يبدو أن الترحيل إلى تأكيد مخصص لذلك هو مسار ترقية معقول بالنسبة لي.

حلت محلها # 3341.

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

القضايا ذات الصلة

sebastianbergmann picture sebastianbergmann  ·  3تعليقات

gellis picture gellis  ·  4تعليقات

keradus picture keradus  ·  3تعليقات

stephen-leavitt-sonyatv-com picture stephen-leavitt-sonyatv-com  ·  4تعليقات

rentalhost picture rentalhost  ·  4تعليقات