Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Jan 25 07:12

    sebastianbergmann on 8.4

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 9 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.0

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… (compare)

  • Jan 25 07:12

    sebastianbergmann on master

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 10 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 8.5

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 10 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 8.2

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 7 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.5

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 4 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 8.0

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 5 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.4

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 3 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 8.1

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 6 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.1

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 (compare)

  • Jan 25 07:12

    sebastianbergmann on 8.3

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 8 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.2

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 1 more (compare)

  • Jan 25 07:12

    sebastianbergmann on 7.3

    Fixed no explanation of how to … "phpunit tests/CurrencyTest" is… Merge branch '7.0' into 7.1 and 2 more (compare)

  • Jan 25 07:12
    sebastianbergmann closed #166
  • Jan 25 07:12
    sebastianbergmann commented #166
  • Jan 24 12:24
    sebastianbergmann milestoned #4027
  • Jan 24 12:24
    sebastianbergmann labeled #4027
  • Jan 24 12:24
    sebastianbergmann opened #4027
  • Jan 24 12:24
    sebastianbergmann assigned #4027
  • Jan 24 11:18

    sebastianbergmann on master

    This is now immutable (compare)

Ewout Pieter den Ouden
@epdenouden
I'm fiddling around with @depends anyway at the moment
Marco Pivetta
@Ocramius
biutifuru
Ewout Pieter den Ouden
@epdenouden
I can have a look later today or tomorrow; it needs some work for the whole lazy loading data providers anyway
bofalke
@bofalke
Should the methods 'expectWarning' and 'expectNotice' be added?
sebastianbergmann/phpunit#3758
Marco Pivetta
@Ocramius
Poking again: need review on sebastianbergmann/phpunit#3810 - it has one discussion point, but otherwise done-ish
@bofalke seems like a bad idea: can expect one of PHPUnit's converted exception types
->expectException(\PHPUnit\yaddayadda\WarningThingy) works just fine
Sebastian Bergmann
@sebastianbergmann
@Ocramius Will be with you shortly.
bofalke
@bofalke
@Ocramius ok thx
Sebastian Bergmann
@sebastianbergmann
I just commented, too. And closed it.
Florian Engelhardt
@flow-control
honest-work.jpg
Marco Pivetta
@Ocramius
@epdenouden got a second to discuss annotations?
Daniel Ruf
@DanielRuf
@SebastianKull see https://www.php.net/manual/en/language.types.float.php so it makes sense in the validation of equality (precision depends on the system, use bcmath, gmp)
Sebastian Bergmann
@sebastianbergmann
Oliver Klee
@oliverklee
Anyone else having problems running the basic tests? sebastianbergmann/phpunit#3815
Marco Pivetta
@Ocramius
Same failures
Sebastian Bergmann
@sebastianbergmann
@Ocramius @oliverklee Those tests are "special" and have their own README. @epdenouden can explain :)
Marco Pivetta
@Ocramius
@sebastianbergmann sebastianbergmann/phpunit#3810 mergeable
Ewout Pieter den Ouden
@epdenouden
@oliverklee ups! README.md werde ich updaten
Oliver Klee
@oliverklee
Danke! :heart:
Oliver Klee
@oliverklee
Has anyone already gotten the end-to-end tests to run in PhpStorm (only the end-to-end tests, not all tests)?
Sebastian Kull
@SebastianKull
@sebastianbergmann can you please give feedback on sebastianbergmann/phpunit#3817 ?
Sebastian Bergmann
@sebastianbergmann
@oliverklee Sorry, I do not run tests in PhpStorm. They are in a separate testsuite named end-to-end. So on the CLI it can be filtered with "--testsuite end-to-end". That should also be possible, somehow, through PhpStorm's UI.
Sebastian Bergmann
@sebastianbergmann
@SebastianKull Other than "floats are hard" I cannot provide feedback right now, sorry.
Florian Engelhardt
@flow-control
If it's ok for everyone, i will start looking into sebastianbergmann/phpunit#2253 anyone already into this?
Andreas Möller
@localheinz
@sebastianbergmann What do you think about giving sebastianbergmann/phpunit#3809 a try? We could still revert if it doesn’t work as expected.
Marco Pivetta
@Ocramius
pardon the wording, but is there an un-fucked version of array_merge_recursive()? This is borked^2: https://3v4l.org/Wm2O6
Sebastian Bergmann
@sebastianbergmann
We break for lunch in ~ 5 minutes.
Marco Pivetta
@Ocramius
DON'T BREAK THE LUNCH!
Andreas Möller
@localheinz
@sebastianbergmann vimeo/psalm laeuft nun auch auf GitHub actions: https://github.com/sebastianbergmann/phpunit/pull/3820!
Ewout Pieter den Ouden
@epdenouden
🥁
Daniel Ruf
@DanielRuf
@epdenouden if you have time, can you review sebastianbergmann/phpunit#3822 Would be great =) I guess it should not be a greedy selector.
Ewout Pieter den Ouden
@epdenouden
thx! saw the alert
are you familiar enough with PHPT to guard the change with a test?
Marco Pivetta
@Ocramius

\PHPUnit\Util\Test::resolveElementToReflectionObjects

What...?!

Ewout Pieter den Ouden
@epdenouden
my brain is starting to skip tests
Sebastian Bergmann
@sebastianbergmann
Pending reservation: "Little Saigon", Seckenheimer Str. 81, 68165 Mannheim
Marco Pivetta
@Ocramius
PHP Fatal error:  Uncaught Error: Internal error: Failed to retrieve the reflection object in /home/ocramius/Documents/sebastianbergmann/phpunit/src/Annotation/DocBlock.php:273
GRRRRRR
Sebastian Bergmann
@sebastianbergmann
Dinner at 20:00 at "Little Saigon", Seckenheimer Str. 81, 68165 Mannheim
Ewout Pieter den Ouden
@epdenouden
@Ocramius are you trying to MockBlock?!
Marco Pivetta
@Ocramius
$this->reflector->getTraits() :S
I hate traits.
Ewout Pieter den Ouden
@epdenouden
gg no re
Sebastian Bergmann
@sebastianbergmann
That hate is a trait you should work on ;)
Marco Pivetta
@Ocramius
>.<
Oliver Klee
@oliverklee
While trying to fix some types, I discovered some bugs in TestSuiteIterator and discovered that it did not have any tests at all. So here they are: sebastianbergmann/phpunit#3825