Re: [GSoC][PATCH v2] t: migrate t0110-urlmatch-normalization to the new framework
- From
Ghanshyam Thakkar <shyamthakkar001@gmail.com>
- Date
- Aug 14, 2024, 14:24 UTC
- Message-ID
- <D3FPBTED647O.7JKO1P410ZEE@gmail.com>
- In-Reply-To
- <xmqqh6bo448j.fsf@gitster.g>
Junio C Hamano <gitster@pobox.com> wrote:
Show 23 quoted lines
> Ghanshyam Thakkar <shyamthakkar001@gmail.com> writes: > > > With the addition of this unit test, we impose a new restriction of > > running the unit tests from either 't/' or 't/unit-tests/bin/' > > directories. This is to construct the path to files which contain some > > input urls under the 't/t-urlmatch-normalization' directory. This > > restriction is similar to one we have for end-to-end tests, where they > > can be ran from only 't/'. > > > > Addition of 't/unit-tests/bin/' is to allow > > for running individual tests which is not currently possible via any > > 'make' targets and also 'unit-tests-test-tool' target is also ran from > > the 't/unit-tests/bin' directory. > > Sorry, but I do not quite follow. The above makes it sound as if > the 'bin' subdirectory is something that never existed before this > patch and this patch introduces the use of that directory, but that > is hardly the case. What does that "Addition of" really refer to? > > Do you mean "we cannot run the tests from arbitrary places, and we > allow them to be run from t/, just like the normal tests" followed > by "in addition, we also allow them to be run from t/unit-tests/bin > directory because ..."?
Yes, I meant that. I sent a v3 which embeds those URLs from files into the code itself, which should not require such restriction.
Link: https://lore.kernel.org/git/20240814142057.94671-1-shyamthakkar001@gmail.com/
Thanks.