git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Christian Couder <christian.couder@gmail.com>
To: Ghanshyam Thakkar <shyamthakkar001@gmail.com>
Cc: git@vger.kernel.org, kaartic.sivaraam@gmail.com, ps@pks.im,
	 karthik.188@gmail.com
Subject: Re: [GSoC][RFC][Proposal v2] Move existing tests to a unit testing framework
Date: Tue, 26 Mar 2024 10:32:47 +0100	[thread overview]
Message-ID: <CAP8UFD3xiGH72BRdQ=76npUKp8pCF+cLXoNYkUFUNKyejWDSAg@mail.gmail.com> (raw)
In-Reply-To: <20240321202218.13648-1-shyamthakkar001@gmail.com>

Hi Ghanshyam,

On Thu, Mar 21, 2024 at 9:22 PM Ghanshyam Thakkar
<shyamthakkar001@gmail.com> wrote:

> @@ -294,8 +325,28 @@ have not setup the blog yet, it will be at:
>  ----Availability----
>
>  My current semester ends on 30th April and my exam/viva is tentatively on 7th
> -May. After that I have freed my summer for GSoC, therefore I will be able to
> -give a minimum of 35-40 hours per week.
> +Mayand I will graduate after the results are out in June. Therefore I will be

s/Mayand/May and/

> +able to give a minimum of 35-40 hours per week after 7th May as I have not taken
> +up any responsibilities in the summer for the purpose of GSoC.


> +----Stretch Goals----
> +
> +These goals may not be achieved during the GSoC period, however, these may come
> +in handy if the migration is done ahead of the schedule or it is determined that
> +the number of tests that need to be migrated are not sufficient to cover the
> +allocated GSoC period further down the line. These can also be done after the
> +GSoC period. (Note that these are just personal observations/ideas and may not
> +be accepted by mentors, hence '_Stretch_ Goals'.)
> +- Enhancement of the unit testing library. An example of the need of doing this
> +  would be t-ctype, which bypasses top-level macros such as TEST(), in favor of
> +  custom logic with the use of internal helper functions.
> +- Add more helper functions in the unit testing library such as handling and use
> +  of parameters defined by environment variables (which I personally faced in
> +  implementing t-oidarray.) I am sure there are other such utilities which might
> +  be worthwhile.
> +- Adding support for performance related testing through unit tests.
> +- Working in other areas of the codebase not related to unit testing or doing
> +  other tasks as specified by mentors.

This makes me think that it would be nice if you or others interested
in the "Move existing tests to a unit testing framework" project could
review Luma's patches that haven't been reviewed and merged yet. See
the "Key Achievements" in
https://lumap.gitlab.io/posts/outreachy-internship-conclusion for
pointers to the patches.

Thanks for updating your proposal anyway. Except for the typo above,
all the changes LGTM.


      reply	other threads:[~2024-03-26  9:41 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-19 22:54 [GSoC][RFC][Proposal] Move existing tests to a unit testing framework Ghanshyam Thakkar
2024-03-20 13:58 ` Christian Couder
2024-03-21 20:21   ` [GSoC][RFC][Proposal v2] " Ghanshyam Thakkar
2024-03-26  9:32     ` Christian Couder [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAP8UFD3xiGH72BRdQ=76npUKp8pCF+cLXoNYkUFUNKyejWDSAg@mail.gmail.com' \
    --to=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=kaartic.sivaraam@gmail.com \
    --cc=karthik.188@gmail.com \
    --cc=ps@pks.im \
    --cc=shyamthakkar001@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).