[Workspace][Bug] Check if workspaces exists when creating saved objects

[yubonluo]

Description

When calling create/bulkCreate save object with API, it needs to check if the passed workspaces exist.

Issues Resolved

Screenshot

No ui change

Testing the changes

Create a saved object

• option 1

curl -X POST -H "osd-xsrf: true" -H "Content-Type: application/json" \
-d '{
  "attributes": {
    "title": "Revenue Dashboard",
    "description": "Revenue dashboard",
    "panelsJSON": "[{\"version\":\"2.9.0\",\"gridData\":{\"x\":0,\"y\":0,\"w\":24,\"h\":15,\"i\":\"5db1d75d-f680-4869-a0e8-0f2b8b05b99c\"},\"panelIndex\":\"5db1d75d-f680-4869-a0e8-0f2b8b05b99c\",\"embeddableConfig\":{},\"panelRefName\":\"panel_0\"}]",
    "optionsJSON": "{\"hidePanelTitles\":false,\"useMargins\":true}",
    "version": 1,
    "timeRestore": true,
    "kibanaSavedObjectMeta": {
      "searchSourceJSON": "{\"query\":{\"language\":\"kuery\",\"query\":\"\"},\"filter\":[]}"
    }
  },
  "references": [
    {
      "id": "37cc8650-b882-11e8-a6d9-e546fe2bba5f",
      "name": "panel_0",
      "type": "visualization"
    }
  ]
}' \
"http://localhost:5601/w/non-exist-workspace/api/saved_objects/dashboard" -u 'admin:myStrongPassword123!' --insecure

{"statusCode":400,"error":"Bad Request","message":"Exist invalid workspaces"}%

• option 2

curl -X POST -H "osd-xsrf: true" -H "Content-Type: application/json" \
-d '{
  "attributes": {
    "title": "Revenue Dashboard",
    "description": "Revenue dashboard",
    "panelsJSON": "[{\"version\":\"2.9.0\",\"gridData\":{\"x\":0,\"y\":0,\"w\":24,\"h\":15,\"i\":\"5db1d75d-f680-4869-a0e8-0f2b8b05b99c\"},\"panelIndex\":\"5db1d75d-f680-4869-a0e8-0f2b8b05b99c\",\"embeddableConfig\":{},\"panelRefName\":\"panel_0\"}]",
    "optionsJSON": "{\"hidePanelTitles\":false,\"useMargins\":true}",
    "version": 1,
    "timeRestore": true,
    "kibanaSavedObjectMeta": {
      "searchSourceJSON": "{\"query\":{\"language\":\"kuery\",\"query\":\"\"},\"filter\":[]}"
    }
  },
"workspaces": ["non-exist-workspace1", "non-exist-workspace2"],
  "references": [
    {
      "id": "37cc8650-b882-11e8-a6d9-e546fe2bba5f",
      "name": "panel_0",
      "type": "visualization"
    }
  ]
}' \
"http://localhost:5601/api/saved_objects/dashboard" -u 'admin:myStrongPassword123!' --insecure
{"statusCode":400,"error":"Bad Request","message":"Exist invalid workspaces"}% 

bulk create saved objects

option 1

curl -X POST -H "osd-xsrf: true" -H "Content-Type: application/json" \
-d '[
  {
    "id": "67a9021c-c97e-4499-8150-9722ab44edd4",
    "type": "visualization",
    "attributes": {
      "title": "vega-visualization",
      "fieldFormatMap": "{\"hour_of_day\":{}}",
      "fields": "[{\"name\":\"@timestamp\",\"type\":\"date\",\"esTypes\":[\"date\"],\"count\":0,\"scripted\":false,\"searchable\":true,\"aggregatable\":true,\"readFromDocValues\":true}]"
    },
    "version": "1",
    "migrationVersion": {},
    "references": [
      {
        "id": "ef71d6c1-8e6b-418d-9f7c-e5d9bbde9cf7",
        "type": "data-source",
        "name": "dataSource"
      }
    ],
    "initialNamespaces": [
      "default"
    ]
  }
]' \
"http://localhost:5601/w/non-exist-workspace/api/saved_objects/_bulk_create" -u 'admin:myStrongPassword123!' --insecure

{"statusCode":400,"error":"Bad Request","message":"Exist invalid workspaces"}% 

option 2

curl -X POST -H "osd-xsrf: true" -H "Content-Type: application/json" \
-d '[
  {
    "id": "67a9021c-c97e-4499-8150-9722ab44edd4",
    "type": "visualization",
    "attributes": {
      "title": "vega-visualization",
      "fieldFormatMap": "{\"hour_of_day\":{}}",
      "fields": "[{\"name\":\"@timestamp\",\"type\":\"date\",\"esTypes\":[\"date\"],\"count\":0,\"scripted\":false,\"searchable\":true,\"aggregatable\":true,\"readFromDocValues\":true}]"
    },
    "version": "1",
    "migrationVersion": {},
    "references": [
      {
        "id": "ef71d6c1-8e6b-418d-9f7c-e5d9bbde9cf7",
        "type": "data-source",
        "name": "dataSource"
      }
    ],
    "initialNamespaces": [
      "default"
    ]
  }
]' \
"http://localhost:5601/api/saved_objects/_bulk_create?workspaces=non-exist-workspace" -u 'admin:myStrongPassword123!' --insecure
{"statusCode":400,"error":"Bad Request","message":"Exist invalid workspaces"}%  

Changelog

• fix: [Workspace] [Bug] Check if workspaces exists when creating saved objects.

Check List

• All tests pass
  • yarn test:jest
  • yarn test:jest_integration
• New functionality includes testing.
• New functionality has been documented.
• Update CHANGELOG.md
• Commits are signed per the DCO using --signoff

[codecov]

Codecov Report

Attention: Patch coverage is 90.90909% with 2 lines in your changes missing coverage. Please review.

Project coverage is 60.88%. Comparing base (080b0db) to head (1fd3501).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
...ver/saved_objects/workspace_id_consumer_wrapper.ts	90.90%	0 Missing and 2 partials ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8739   +/-   ##
=======================================
  Coverage   60.88%   60.88%           
=======================================
  Files        3802     3802           
  Lines       91083    91087    +4     
  Branches    14382    14382           
=======================================
+ Hits        55456    55461    +5     
  Misses      32086    32086           
+ Partials     3541     3540    -1     

Flag	Coverage Δ
Linux_1	29.03% <90.90%> (+<0.01%)	⬆️
Linux_2	56.38% <ø> (ø)
Linux_3	37.90% <ø> (ø)
Linux_4	29.00% <ø> (ø)
Windows_1	29.04% <90.90%> (+<0.01%)	⬆️
Windows_2	56.34% <ø> (ø)
Windows_3	37.90% <ø> (ø)
Windows_4	29.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ananzh]

I think we also need to fix the tests. Then I could help to approve.

[ruanyl]

I'm good with the current implementation, but perhaps we should introduce a new workspaceClient.bulkGet API, can we add a TODO item if it makes sense? @yubonluo @SuZhou-Joe