Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:121963
MIME-Version: 1.0
References: <CAEwRQKbHFD_BBV1PDgj8RmdUXwGzXcCvem=GgAeiNVgPzjC3cg@mail.gmail.com>
 <CAG=Eydhgyh4pcWbJr-RgyJv_iyaf9TG+n5uqvY5Ozz0x5HJuLA@mail.gmail.com>
In-Reply-To: <CAG=Eydhgyh4pcWbJr-RgyJv_iyaf9TG+n5uqvY5Ozz0x5HJuLA@mail.gmail.com>
Date: Fri, 8 Dec 2023 13:04:09 -0300
Message-ID: <CAEwRQKYNZkXdd-kmpiRr0sJ3Yw4tixUo80QqZjRAfghfqYwj8A@mail.gmail.com>
To: Florian Engelhardt <florian@engelhardt.tc>
Cc: PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] PHP test coverage
From: carlosv775@gmail.com (Vinicius Dias)

> The code coverage report can be found at https://app.codecov.io/github/php/php-src

Ah, perfect. Thank you for sharing. I will take a closer look after
work, but something seems weird to me. Here[1] you see that
ctype_alpha seems to be uncovered, but here[2] we can see that it's
tested.

[1]: https://app.codecov.io/github/php/php-src/blob/master/ext%2Fctype%2Fctype.c#L112
[2]: https://github.com/php/php-src/blob/master/ext/ctype/tests/ctype_alpha_basic.phpt


> I will have a look, the page overall looks like it could need some love ;-)
> In the meantime if you are interested in writing tests, I once wrote a blog post about that topic at https://dev.to/realflowcontrol/growing-the-php-core-one-test-at-a-time-4g4k
Ah, that's great. Thank you for that! :-D