Skip writing the HTTP actions to config file and log warning when REST Methods are configured for tables/views

[Shyam Sundar J (severussundar)]

Why make this change?

• Closes Make the REST Methods property nullable in EntityRestOptions class #1571 Need to unignore test once behavior of stored procedures is fixed #1570

• At the moment, CLI writes out all the 5 HTTP methods to the config file when adding an entity backed by table/view.
  

• However, for tables/views, the Methods property does not play any role and can be skipped writing to the config file.

What is this change?

• EntityRestOptions: Methods property is made nullable. References where it was initialized with an empty array is updated to null. Appropriate null checks are added.
• EntityRestOptionsConverter: Serialization logic is updated to write to the config file only when the Methods within REST element is not empty
• RuntimeConfigValidator.ValidatePresenceOfExtraRestProperties(): Helper method to log a warning message when Methods property is added manually to an entity backed by table/view.
• HTTP Actions are removed from the snapshot files.
• dab-config.* files - Methods property is removed from the reference config files

How was this tested?

• Integration Tests
• Unit Tests
• Manual Tests

Sample Request(s)

• CLI: REST actions are not written to the config file
  dab add books --source books --permissions "anonymous:*"

Config Json created:

"entities": {
    "books": {
  "source": {
    "object": "books",
    "type": "table"
  },
  "graphql": {
    "enabled": true,
    "type": {
      "singular": "books",
      "plural": "books"
    }
  },
  "rest": {
    "enabled": true
  },
  "permissions": [
    {
      "role": "anonymous",
      "actions": [
        {
          "action": "*"
        }
      ]
    }
  ]
}
  }

• When the config file is hand-edited to add methods property to a table/view, engine logs a warning message during start-up

Config:

"Book": {
      "source": {
        "object": "books",
        "type": "table"
      },
      "rest": {
        "enabled": true,
        "methods": [
          "get",
          "post"
        ]
      },
      "graphql": {
        "enabled": true,
        "type": {
          "singular": "book",
          "plural": "books"
        }
      },

Warning Logged:

[Sean Leonard (seantleonard)]

Some questions.

[Ayush Agarwal (ayush3797)]

Some questions and suggestions around the same. Looks good so far!

[Aniruddh Munde (Aniruddh25)]

Looks good so far, left few comments.

[Aniruddh Munde (Aniruddh25)]

Thanks for noticing this bug and fixing it for 0.8!

[Ayush Agarwal (ayush3797)]

Just one question. LGTM! Thanks for fixing this!