[LTP] [PATCH] doc: Add basic shell test description
Cyril Hrubis
chrubis@suse.cz
Thu Nov 13 11:21:30 CET 2025
Hi!
> > > And here would have subtitle "shell loader test library". Why? It'd be good to
> > > admit we have plain shell (which uses tst_test.sh) and shell loader. Of course,
> > > shell loader could be a mentioned first, then old API:
>
> > Generally I would like to avoid mixing documentation for different APIs.
> > I'm afraid that would make things unnecessarily confusing.
>
> OK, no problem (I did not get you want to document only Shell Loader). But
> hiding the fact that vast majority of the shell tests use tst_test.h is not
> optimal (it might be confusing for new users).
I'm not trying to hide it. I'm trying to write at least some
documentation for the shell loader, that is independed task from
converting the documentation we have for the older shell library into
rst and putting it in it's own menu item.
> Also, Shell Loader does not have useful functions from tst_test.h, e.g.
> ROD(), EXPECT_PASS(). But ok, nobody needs them so far (although ROD() could be
> useful) and I would also ask people to use C API if possible.
Indeed the shell loader is currently under construction, but it allows
us to write a simple tests already and should be good enough for simple
enough tests.
> Could we decide that tst_test.sh becomes "legacy/old" shell API which could be
> on a separate page doc/developers/legacy_api_shell_tests.rst which would just
> mention it's deprecated and linked:
> https://github.com/linux-test-project/ltp/blob/master/doc/old/Shell-Test-API.asciidoc
> (AI would convert it quickly to RST, but I'd be ok to keep just a link.)
Yes, that is what I think would be ideal solution, much better than
mixing the two APIs in a single page.
> And test.h becomes undocumented "legacy legacy" shell API (test.h is still used
> by few tests).
You mean 'test.sh' I suppose, yes there are a few tests using it. We
should clean that up eventually.
--
Cyril Hrubis
chrubis@suse.cz
More information about the ltp
mailing list