Where communities thrive


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

    sebastianbergmann on master

    Rename method Fix (compare)

  • Jan 27 04:27

    sebastianbergmann on master

    Leftover from 5e09cdc815ad9e0a3… (compare)

  • Jan 27 03:05
    Krinkle commented on 5e09cdc
  • Jan 26 11:37
    williamdes commented #3906
  • Jan 26 09:33
    someniatko commented #1494
  • Jan 26 08:51
    ratatatKE commented #1494
  • Jan 26 08:29
    nezaboravi commented #3983
  • 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)

Ewout Pieter den Ouden
@epdenouden
maybe we need a phpunit slack
Markus Podar
@mfn
What's so bad about gitter :)
FWIF also running slack for other stuff but couldn't care less
Ewout Pieter den Ouden
@epdenouden
ah morning :)
I'll just assume then everything works and everybody is testing all-green at 100% coverage all day every day ;-)
Markus Podar
@mfn
Ewout Pieter den Ouden
@epdenouden
as long as the data providers just crash to shell, there's no red test right ;-)
Markus Podar
@mfn
image.png
Ewout Pieter den Ouden
@epdenouden
@mfn oooo graphql that's a nice toy
Markus Podar
@mfn
👍
Ewout Pieter den Ouden
@epdenouden
you're having fun :)
was just looking at the Code Sprint in sept in Mannheim
"[...] because, if PHPUnit is not well-tested, then what will be?"
Ewout Pieter den Ouden
@epdenouden
man this euro heatwave
"it's fine" :fire: :fire: :fire: :fire: :fire: :fire: :fire:
Yoanndp
@Yoanndp
Hello, someone is there?
Yoanndp
@Yoanndp
Nevermind, I fixed it :) Have a nice day
Ghost
@ghost~5d6f7af9d73408ce4fc9f5f7
Hi there, how is it possible to mock several methods on one stub? I tried with several calls to ->method(), but I get the error 'Method name matcher is already defined, cannot redefine'
Nicolas Hohm
@nickel715
@bezin Usually you do it like this https://github.com/sebastianbergmann/phpunit/blob/master/tests/unit/Framework/TestBuilderTest.php#L28-L38
If you need further help please provide some example code.
Ghost
@ghost~5d6f7af9d73408ce4fc9f5f7
@nickel715 Thanks for your hint, I found out about that a couple of hours after my question. I chained all my calls, but did not separate them per method :thumbsup:
Sebastian Bergmann
@sebastianbergmann
Andreas Möller
@localheinz
@epdenouden https://bear.app
re: eval-stdin security risk
Ewout Pieter den Ouden
@epdenouden
also, kein Code Sprint chat? :)
ich werde die '@depends on class' mal fixen 🛠
Marco Pivetta
@Ocramius

Mine: sebastianbergmann/phpunit#3120

Feel free to come over and tells me your strong opinions if I'm supposed to do something specific :P

Marco Pivetta
@Ocramius
@epdenouden can you assign the above to the sprint, please?
Ewout Pieter den Ouden
@epdenouden
@Ocramius done!
Marco Pivetta
@Ocramius
thx
Matthias Schobner
@schobner
i try this issue: Class name of expected function call #3676
sebastianbergmann/phpunit#3676
Sebastian Bergmann
@sebastianbergmann
This might be interested for someone in to performance profiling: sebastianbergmann/phpunit#3745
Daniel Ruf
@DanielRuf
gave feedback on sebastianbergmann/php-code-coverage#694 which should be fixed with the UMD version of Popper
Oliver Klee
@oliverklee
In PhpStorm, the .yml schema validation complains about this line: https://github.com/sebastianbergmann/phpunit/blob/master/.travis.yml#L55
env: php-cs-fixer
It expects the environment variable having a value. Would it make sense to give it a value (e.g., 1) to get rid of this validation warning?
Sebastian Bergmann
@sebastianbergmann
As I understand it, "env: php-cs-fixer" defines an environment named php-cs-fixer. I would ignore this warning since Travis CI works fine with the configuration.
Daniel Ruf
@DanielRuf
@epdenouden do you know what we have to change / fix there? sebastianbergmann/phpunit#3693
Ewout Pieter den Ouden
@epdenouden
@DanielRuf moment...
ahja! ich vermute: RegEx Elend
the whole @annotation parsing could use a nice cleanup
Sebastian Bergmann
@sebastianbergmann
Starting to wrap my head around sebastianbergmann/phpunit#3803
Ewout Pieter den Ouden
@epdenouden
but this is just a weird one, I think it's just a strip/trim/regex thingy
Sebastian Bergmann
@sebastianbergmann
the whole @annotation parsing could use a nice cleanup
Indeed.
Ewout Pieter den Ouden
@epdenouden
The annotations are high on my private $wishlist as it can be refactored to use much fewer resources
while digging around @dataProvider I found some lookups being done twice, for example
Marco Pivetta
@Ocramius
@epdenouden do we haz issue for annotations? I can look into it.
Ewout Pieter den Ouden
@epdenouden
@Ocramius I am not sure; it gets touched on by bugs like the one @DanielRuf just mentioned
it's one of those older parts that 'just kinda works'