-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add test #4
Add test #4
Conversation
873b4c0
to
4c3c1d2
Compare
Hey @Sachindu-Nethmin, Looks good! I will review this and verify by running locally. |
add License and line spaces into test.bal
use camel case in test function names
When I curl it, it also gives an error. I don't know what to do. Can you please help me? |
using the check expression to avoid the error checking scenarios
@Sachindu-Nethmin added a few more suggestions |
35928e5
to
5a1f34d
Compare
use loudness in browser artist test.
19e68fe
to
d73e605
Compare
@Sachindu-Nethmin Were you able to fix the test failures? |
@aashikam curl --request GET response { |
So I guess we can disable this test for now. |
remove test cases that not support in region.
@aashikam, thank you for your assistance. Could you please review it once more? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@NipunaRanasinghe Need your review.
remove the check expression
70c716b
@aashikam, I would greatly appreciate it if you could take another look at this. Thank you! |
Hey @Sachindu-Nethmin |
remove unrelated info about envs.
91b21e2
to
34611d9
Compare
Create test.bal
Purpose
Add test cases to spotify connector
Examples
Checklist