i think if they work in react forms then it should be fine - elemental editor v4 is just a series of FormBuilderLoaders


up until recently SapphireTest was always flushing in SS4 unit tests 😛

robbieaverill for reference

Show 1 attachment(s)

It's massively slowing down test execution,
particularly for rapid execution through test-driven-development (TDD)

Regression since 4.0.0-beta1, see|3873e4b#diff-e2f3520673dab1cd0127a469fe3a2a29R925

It replaced prior behaviour which respected the flush parameter:|3873e4b#diff-e2f3520673dab1cd0127a469fe3a2a29R925

You could argue this is an API change, but I think it's not going to affect anyone negatively in production. CI based tests are quite unlikely to change files or YAML config between test runs which would require a manifest rebuild. And developers executing tests should follow our existing documentation which always said to use flush between test runs when you change stuff.

Hide attachment content

there's a sort of hack you can do by extending each page type with a test only class and hiding its ancestor e.g. class HiddenVirtualPage extends VirtualPage implements TestOnly { private static $hide_ancestor = VirtualPage::class; } (i think)


If forking the module is the solution- that’s not optimal


reminds me: take a look at admin/client/src/bundles/bundle.js