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.

Sign up for GitHub

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

Jump to bottom

Add Test Cases for Twitter Connector APIs and Update Mock Server #112

Merged
merged 16 commits into from
Jul 2, 2024
Merged

Add Test Cases for Twitter Connector APIs and Update Mock Server #112

merged 16 commits into from
Jul 2, 2024

Conversation

vish-mv
Copy link
Contributor

@vish-mv vish-mv commented Jul 2, 2024

Add Test Cases for Twitter Connector APIs and Update Mock Server

Summary

  • 14 New Test Cases: Added comprehensive test cases for 14 REST APIs in the Twitter connector.
  • Mock Server Updates: Enhanced the mock server to support the new test cases.
  • README Update: Updated README with instructions and descriptions for the new tests.

Changes

1. Test Cases

  • Implemented 14 new test cases covering different Twitter API endpoints.

2. Mock Server

  • Updated to handle and simulate responses for the new test cases.

3. README

  • Revised to include details on running the new tests .

Testing

  • All test cases have been validated against the updated mock server.

@vish-mv vish-mv requested a review from abeykoon as a code owner July 2, 2024 09:41
NipunaRanasinghe
NipunaRanasinghe reviewed Jul 2, 2024
View reviewed changes
Copy link
Contributor

@NipunaRanasinghe NipunaRanasinghe left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@vish-mv It seems you've mistakenly commited some examples as well. shall we clean that up?

@vish-mv
Copy link
Contributor Author

vish-mv commented Jul 2, 2024

@vish-mv It seems you've mistakenly commited some examples as well. shall we clean that up?

Sure. Those things are probably came from Loshan's PR to my Repo. I'll Remove Them now.

ayeshLK
ayeshLK reviewed Jul 2, 2024
View reviewed changes
ballerina/tests/test.bal Outdated Show resolved Hide resolved
ayeshLK
ayeshLK reviewed Jul 2, 2024
View reviewed changes
ballerina/tests/test.bal Outdated Show resolved Hide resolved
NipunaRanasinghe
NipunaRanasinghe requested changes Jul 2, 2024
View reviewed changes
example/build.gradle Outdated Show resolved Hide resolved
ballerina/tests/README.md Outdated Show resolved Hide resolved
ballerina/tests/mock_service.bal Outdated Show resolved Hide resolved
NipunaRanasinghe
NipunaRanasinghe reviewed Jul 2, 2024
View reviewed changes
ballerina/tests/mock_service.bal Outdated Show resolved Hide resolved
ballerina/tests/mock_service.bal Outdated Show resolved Hide resolved
ballerina/tests/test.bal Outdated Show resolved Hide resolved
ballerina/tests/test.bal Outdated Show resolved Hide resolved
NipunaRanasinghe
NipunaRanasinghe approved these changes Jul 2, 2024
View reviewed changes
Copy link
Contributor

@NipunaRanasinghe NipunaRanasinghe left a comment

Choose a reason for hiding this comment

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

LGTM

@NipunaRanasinghe NipunaRanasinghe merged commit bbe31a2 into ballerina-platform:main Jul 2, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ayeshLK ayeshLK ayeshLK left review comments

@NipunaRanasinghe NipunaRanasinghe NipunaRanasinghe approved these changes

@abeykoon abeykoon Awaiting requested review from abeykoon

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vish-mv @NipunaRanasinghe @ayeshLK @loshan20011