entity_get_form_display() is deprecated and should be removed

Looks good, thank you.

+++ b/tests/src/FunctionalJavascript/ContentEntity/ContentEntityAutosaveFormTestBase.php
@@ -691,7 +692,7 @@ abstract class ContentEntityAutosaveFormTestBase extends AutosaveFormTestBase {
-      entity_get_form_display($this->entityType, $this->bundle, 'default')
+      EntityFormDisplay::load($this->entityType . '.' . $this->bundle . '.default')

This assumes that the form display already exists. I think the reason #2 passed tests is that the only entity type being tested is Node, and the test setup's drupalCreateContentType() calls node_add_body_field(), which has the side effect of creating and saving a form display. I'm not sure that would be the case if this test is subclassed by something that tests a different entity type.

https://www.drupal.org/node/2835616 says to use \Drupal::service('entity_display.repository')->getFormDisplay() instead, which creates a new display entity if it doesn't already exist. However, that's an API that's only available in Drupal 8.8, and this module might still want to be working with Drupal 8.7.

@effulgentsia, thank you! This is something I wanted to check, but unfortunately I just forgot. I apologize for the delay.

And yes, I would prefer supporting as many Drupal versions as possible. It might not be a clean code, but maybe we could check whether the 8.7 method exists and if not fallback to the old one?

Like this?

Exactly, thank you!

Committed and pushed to 8.x-1.x.

Verified on latest 8.8.0 release.

autosave_form git:(8.x-1.x) ../../../../vendor/bin/drupal-check -d .
 22/22 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

                                                                                                                        
 [OK] No errors