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

Patch/first pr #194

Merged
merged 8 commits into from
Sep 22, 2023
Merged

Conversation

SuZhou-Joe
Copy link
Collaborator

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

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

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2023

Codecov Report

Merging #194 (9d3933c) into workspace-pr-integr (f630277) will not change coverage.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##           workspace-pr-integr     #194   +/-   ##
====================================================
  Coverage                66.44%   66.44%           
====================================================
  Files                     3408     3408           
  Lines                    65149    65149           
  Branches                 10431    10431           
====================================================
  Hits                     43289    43289           
  Misses                   19265    19265           
  Partials                  2595     2595           
Flag Coverage Δ
Linux_ 66.02% <ø> (ø)
_1 34.81% <ø> (ø)
_2 55.35% <ø> (ø)
_3 44.58% <ø> (ø)
_4 34.88% <ø> (ø)

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

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

@SuZhou-Joe SuZhou-Joe merged commit 855c3a8 into ruanyl:workspace-pr-integr Sep 22, 2023
35 checks passed
SuZhou-Joe added a commit that referenced this pull request Oct 9, 2023
* Patch/first pr (#194)

* temp: add unit test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add function test for workspace CRUD routes

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: use saved objects client instead of internal repository

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: update CHANGELOG

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: exclude permission check wrapper

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add integration test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: add configuration

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: enable workspace flag when run workspace related test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: update test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: change to equal 3

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: test

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* feat: make permissions field optional

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* fix: make permissions as optional params

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

* fix: make permissions params optional

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>

---------

Signed-off-by: SuZhou-Joe <suzhou@amazon.com>
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