git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Patrick Steinhardt <ps@pks.im>
To: Christian Couder <christian.couder@gmail.com>
Cc: Chandra Pratap <chandrapratap3519@gmail.com>,
	git@vger.kernel.org, karthik.188@gmail.com,
	kaartic.sivaraam@gmail.com
Subject: Re: [RFC][GSoC] Proposal: Move reftable and other tests to the unit testing framework
Date: Thu, 21 Mar 2024 13:27:24 +0100	[thread overview]
Message-ID: <ZfwnrL6Zl_lcV09y@tanuki> (raw)
In-Reply-To: <CAP8UFD3Rb0kN=g=EPcSqYBb3U=OgW_06jqwafF=vRYhetZ9hdQ@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2180 bytes --]

On Wed, Mar 20, 2024 at 02:17:23PM +0100, Christian Couder wrote:
> On Tue, Mar 19, 2024 at 6:11 PM Chandra Pratap
> <chandrapratap3519@gmail.com> wrote:
[snip]
> > A new unit testing framework was introduced to the Git mailing list last
> > year with the aim of simplifying testing and improving maintainability.
> > The idea was accepted and merged into master on 09/11/2023. This project
> > aims to extend that work by moving more tests from the current setup to
> > the new unit testing framework.
> >
> > The SoC 2024 Ideas page (link: https://git.github.io/SoC-2019-Ideas/)
> > mentions reftable unit tests migration as a separate project from the
> > general unit test migration project, however, I propose migrating other
> > tests alongside the reftable unit tests as a part of this proposal.
> 
> It means that if we select your proposal, we cannot select someone
> else to work on either the "Move existing tests to a unit testing
> framework" project or the "Convert reftable unit tests to use the unit
> testing framework" project.
> 
> I am not sure but I think that, after migrating all the reftable unit
> tests, I would prefer you working on other reftable related tasks
> rather than on more unit test migrations.

I agree, I'd also like to keep these projects separate from each other.
Also, the reftable tests could certainly use some polishing and a lot
more documentation than they currently have. Their coding style does not
really match the rest of the project, and if we're busy migrating the
code then I think we should also take the chance and touch it up.

That will require the student to understand the reftable code a whole
lot more deeply than a mere conversion would require though. Thus, I
would like to caution any student who wants to pick up this project to
not underestimate the effort required for this project. The reftable
code itself is certainly non-trivial.

As Chris mentioned, if this project would be finished early I'm quite
sure that the efforts would uncover bugs, test gaps or similar things in
the vicinity of reftables that the student could certainly continue to
work on.

Patrick

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

  parent reply	other threads:[~2024-03-21 12:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-03-19 17:11 [RFC][GSoC] Proposal: Move reftable and other tests to the unit testing framework Chandra Pratap
2024-03-20 13:17 ` Christian Couder
2024-03-20 19:35   ` Chandra Pratap
2024-03-21 12:27   ` Patrick Steinhardt [this message]
2024-03-29  6:54     ` [RFC][GSoC] Proposal v2: Move more " Chandra Pratap
2024-03-29 17:51       ` Karthik Nayak
2024-03-29  7:08     ` [RFC][GSoC] Proposal v2: Move and improve reftable tests in " Chandra Pratap
2024-04-02 10:01       ` Christian Couder

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=ZfwnrL6Zl_lcV09y@tanuki \
    --to=ps@pks.im \
    --cc=chandrapratap3519@gmail.com \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=kaartic.sivaraam@gmail.com \
    --cc=karthik.188@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).