Skip to content

Conversation

@ngyradi
Copy link
Contributor

@ngyradi ngyradi commented Jul 21, 2023

Added tests for: Dashboard, Image config, Projects, Storage, Teams.

@ngyradi ngyradi self-assigned this Jul 21, 2023
@github-actions github-actions bot added source:web The scope of the issue or pull request is web. lang:typescript pr:test labels Jul 21, 2023
@ngyradi ngyradi marked this pull request as ready for review July 24, 2023 09:35
@ngyradi ngyradi requested a review from a team as a code owner July 24, 2023 09:35
Copy link
Contributor

@robot9706 robot9706 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WS patch matchers need some refactoring, also some minor comments, let's discuss everything in person.

Otherwise good job for writing a ton of tests🎈

Copy link
Contributor

@robot9706 robot9706 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks cool ✨, wait for another review and e2e.

@ngyradi ngyradi requested a review from robot9706 July 25, 2023 11:37
@AmorfEvo
Copy link
Contributor

LGTM (tests run with 4 workers)

@AmorfEvo AmorfEvo self-requested a review July 25, 2023 12:21
@m8vago m8vago merged commit a5dbcb9 into develop Jul 27, 2023
@m8vago m8vago deleted the test/various-tests branch July 27, 2023 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lang:typescript pr:test severity:medium source:web The scope of the issue or pull request is web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants