simpler test glob

[turadg]

Closes: #1913

Description

This set out to use the Ava default file glob (i.e. no custom setting) but ultimately doesn't for two reasons:

• requires all the non-test files to have a preceding underscore (this is both laborious to complete and not a unanimously well received outcome)
• would allow test files to have any file name, including foo.fixture.js. We like the .test.js consistency

Since I did move a bunch of files to use underscores, I left that change in here, along with the script that does it. I have no qualms about removing them at reviewer request.

This also includes a little CI change that saved me time, ci: no-bail so all packages run lint

Security Considerations

n/a

Scaling Considerations

n/a

Documentation Considerations

Consistent with existing practices. There is a lot of underscore now, but it's not required.

Testing Considerations

CI

Compatibility Considerations

Not exported

Upgrade Considerations

none

[kriskowal]

Love kicking off the legacy test path pattern. Head’s up to @boneskull with possibly-impacted work in flight.

[kriskowal]

--no-opportunity-for-parole

[kumavis]

some weird behavior around --no-bail

seeing this on lerna 8.1.8

with --no-bail:

lerna ERR! Received non-zero exit code 1 during execution
lerna success run Ran npm script 'test' in 39 packages in 42.6s:
lerna success - @endo/base64
lerna success - @endo/bundle-source
[...]
lerna success - ses
[...]

without --no-bail:

lerna ERR! yarn run test exited 1 in 'ses'

CI job https://github.com/endojs/endo/actions/runs/11153456684/job/31000992316?pr=2564

doc https://github.com/lerna/lerna/tree/main/libs/commands/run#--no-bail
note on inconsistency of default behavior? lerna/lerna#3827

[kumavis]

yarn workspaces foreach runs all without bailing #2565