With new people joining Council, these next couple of months are definitely timely to review and update our standard operating testing procedures. Trish and Syd, thanks for working on this just now! 

Elisa

Elisa Beshero-Bondar, PhD (she/they)
Chair, TEI Technical Council
Program Chair of Digital Media, Arts, and Technology | Professor of Digital Humanities |  Director of the Digital Humanities Lab at Penn State Erie, The Behrend College

Typeset by hand on my iPhone

On Oct 18, 2023, at 8:18 AM, Elisa Beshero-Bondar <ebbondar@gmail.com> wrote:

I like solution 3 and “Legacy_Tests” as well, but only if we are sure we’re ready to demote the original test as legacy. Does Test2 cover everything we care about now? Are we missing any tests there that matter and that the old tests cover?

Elisa

Elisa Beshero-Bondar, PhD (she/they)
Chair, TEI Technical Council
Program Chair of Digital Media, Arts, and Technology | Professor of Digital Humanities |  Director of the Digital Humanities Lab at Penn State Erie, The Behrend College

Typeset by hand on my iPhone

On Oct 17, 2023, at 5:42 PM, Bauman, Syd <s.bauman@northeastern.edu> wrote:


Oooh. I like the idea of  Test/ → Legacy_Test/  and  Test2/ → Test/.



I lean towards 3), though perhaps Test/ can be kept with a new name, such as additional_tests or... Test2/

Trish & I are doing the p5subset → Stylesheets process. Most of our time has been spent handling Test/. I think it is just too cumbersome — the amount of time we (Council, collectively) spend wrangling with Test/ is likely by now much greater than the amount of time it saves by catching bugs early.

I think we need to do one of three things, toot sweet:
  1. Re-work the Test/Makefile so that the process is much easier: e.g., when comparisons are deferred, a shell script to perform them needs to be generated; this might be hard to impossible.
  2. Stop using Test/, it is just not helpful enough to be worth it. Leave the directory in the repo so that we can use it when there is a particularly gnarly case, or when a human is not waiting for it, but remove it from the general test procedure we use, e.g. when generating p5subset for the Stylesheets repo.
  3. Remove Test/ from the repo (and various instructions) entirely (and rename Test2/ to Test/ :-)
My vote is for (2). I could be perhaps be talked into (3) reasonably easily; it would take some serious convincing to get me to vote for (1).

P.S. BTW, Martin & I intended that we would do (3) when we developed Test2/. (I wanted to hang onto Test/, i.e. do (2), for awhile to ensure that everything tested by Test/ was also tested by Test2/.)

_______________________________________________
Tei-council mailing list
Tei-council@lists.tei-c.org
http://lists.lists.tei-c.org/mailman/listinfo/tei-council