Love Fuel?    Donate

FuelPHP Forums

Ask your question about FuelPHP in the appropriate forum, or help others by answering their questions.
Test failures in fresh project
  • phpatrickphpatrick
    After creating a new project (fuel 1.8.0.1) with oil create I get 3 testing failures:

    1) Fuel\Core\Test_Debug::test_debug_dump_normally
    2) Fuel\Core\Test_Debug::test_debug_dump_by_call_user_func_array
    3) Fuel\Core\Test_Debug::test_debug_dump_by_call_fuel_func_array

    Is there something I can do about it?
  • HarroHarro
    Does your PHP installation have xdebug installed? What platform / operating system do you use? Which PHP version?

    Can you also post the expected/actual values of these failed tests?

    It is nothing serious btw, it is probably some layout issue with the output of Debug::dump() on your specific platform.
  • phpatrickphpatrick
    I tried with and without xdebug. Seems that it does not matter. I'm on Ubuntu Gnome 17.04.

    The first diff (the others seem to be of similar nature):

    1) Fuel\Core\Test_Debug::test_debug_dump_normally
    Failed asserting that two strings are equal.
    --- Expected
    +++ Actual
    @@ @@
    -' <script type="text/javascript">function fuel_debug_toggle(a){if(document.getElementById){if(document.getElementById(a).style.display=="none"){document.getElementById(a).style.display="block"}else{document.getElementById(a).style.display="none"}}else{if(document.layers){if(document.id.display=="none"){document.id.display="block"}else{document.id.display="none"}}else{if(document.all.id.style.display=="none"){document.all.id.style.display="block"}else{document.all.id.style.display="none"}}}};</script><div class="fuelphp-dump" style="font-size: 13px;background: #EEE !important; border:1px solid #666; color: #000 !important; padding:10px;"><h1 style="border-bottom: 1px solid #CCC; padding: 0 0 5px 0; margin: 0 0 5px 0; font: bold 120% sans-serif;">COREPATH/tests/debug.php @ line: 41</h1><pre style="overflow:auto;font-size:100%;"><strong>Variable #1:</strong>
    +' <script type="text/javascript">function fuel_debug_toggle(a){if(document.getElementById){if(document.getElementById(a).style.display=="none"){document.getElementById(a).style.display="block"}else{document.getElementById(a).style.display="none"}}else{if(document.layers){if(document.id.display=="none"){document.id.display="block"}else{document.id.display="none"}}else{if(document.all.id.style.display=="none"){document.all.id.style.display="block"}else{document.all.id.style.display="none"}}}};</script><div class="fuelphp-dump" style="font-size: 13px;background: #EEE !important; border:1px solid #666; color: #000 !important; padding:10px;"><h1 style="border-bottom: 1px solid #CCC; padding: 0 0 5px 0; margin: 0 0 5px 0; font: bold 120% sans-serif;">DOCROOT/fuel/core/tests/debug.php @ line: 41</h1><pre style="overflow:auto;font-size:100%;"><strong>Variable #1:</strong>
     <i></i> <strong></strong> (Integer): 1
     
     
     <strong>Variable #2:</strong>
     <i></i> <strong></strong> (Integer): 2
     
     
     <strong>Variable #3:</strong>
     <i></i> <strong></strong> (Integer): 3
     
     
     </pre></div>'
  • HarroHarro
    Accepted Answer
    It fails because it reports: 
        DOCROOT/fuel/core/tests/debug.php @ line: 41
    instead of
        COREPATH/tests/debug.php @ line: 41

    Which is fixed here https://github.com/fuel/core/commit/c28a83a2e1cfa32a88ed5c1a93a79de3547a873e in development, but not released yet.

    So no worries, it will automatically disappear with the next Fuel update (not planned yet).
  • nickstingernickstinger
    Can we backport this to 1.8?

    Although most people runs tests like this:

        php oil test --group=App

    Its a tad embarrassing to see it.


  • HarroHarro
    There is an 1.8 update on the way with will incorporate all fixes in develop.
Add a Comment

Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

Sign In Apply for Membership

Categories

In this Discussion