Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport workspace]Patch/first pr (#194) #196

Merged

Conversation

SuZhou-Joe
Copy link
Collaborator

@SuZhou-Joe SuZhou-Joe commented Sep 22, 2023

  • temp: add unit test

  • feat: add function test for workspace CRUD routes

  • feat: use saved objects client instead of internal repository

  • feat: update CHANGELOG

  • feat: exclude permission check wrapper

  • feat: add integration test

  • feat: add configuration

  • feat: enable workspace flag when run workspace related test


Description

Issues Resolved

Screenshot

Testing the changes

Check List

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

@codecov-commenter
Copy link

codecov-commenter commented Sep 22, 2023

Codecov Report

Merging #196 (a42ccd2) into workspace (37171cf) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           workspace     #196      +/-   ##
=============================================
- Coverage      66.12%   66.12%   -0.01%     
=============================================
  Files           3420     3420              
  Lines          65753    65753              
  Branches       10590    10590              
=============================================
- Hits           43479    43477       -2     
- Misses         19633    19634       +1     
- Partials        2641     2642       +1     
Flag Coverage Δ
Linux_1 30.42% <ø> (ø)
Linux_2 55.12% <ø> (ø)
Linux_3 42.76% <ø> (-0.01%) ⬇️
Linux_4 34.52% <ø> (ø)

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

see 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@ruanyl
Copy link
Owner

ruanyl commented Sep 22, 2023

@SuZhou-Joe It looks some tests are failed

@SuZhou-Joe
Copy link
Collaborator Author

SuZhou-Joe commented Sep 23, 2023

@SuZhou-Joe It looks some tests are failed

Indeed, it is due to:

  1. Permissions is required in route validation rule, is that expected? @wanglam what if there is no authentication?
  2. Management/Public/Personal workspace is created when calling list API.

@SuZhou-Joe SuZhou-Joe force-pushed the backport/backport-194-to-workspace branch from cd670a2 to 838e966 Compare September 29, 2023 01:04
* temp: add unit test

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add function test for workspace CRUD routes

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: use saved objects client instead of internal repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: update CHANGELOG

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: exclude permission check wrapper

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add integration test

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: add configuration

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: enable workspace flag when run workspace related test

Signed-off-by: SuZhou-Joe <[email protected]>

---------

Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
@SuZhou-Joe SuZhou-Joe force-pushed the backport/backport-194-to-workspace branch from 2e2b9fb to 86fcb49 Compare October 8, 2023 03:56
@SuZhou-Joe SuZhou-Joe merged commit cef4ce7 into ruanyl:workspace Oct 9, 2023
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants