Skip to content

Comments

Add API endpoint test cases and implement mock service #4

Merged
Nuvindu merged 8 commits intoballerina-platform:mainfrom
ChathuraIshara:main
Feb 24, 2025
Merged

Add API endpoint test cases and implement mock service #4
Nuvindu merged 8 commits intoballerina-platform:mainfrom
ChathuraIshara:main

Conversation

@ChathuraIshara
Copy link
Contributor

Purpose

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • [ x] Added tests
  • Updated the spec
  • Checked native-image compatibility

Comment on lines 101 to 105
@test:Config {
}
isolated function testTaskBatchUpsert() returns error? {
}

Copy link
Contributor

Choose a reason for hiding this comment

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

We can remove this if you are not going to use the test case

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Comment on lines 264 to 265
}
@test:Config{
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
}
@test:Config{
}
@test:Config{

Copy link
Contributor

Choose a reason for hiding this comment

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

We can format this file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated

@Nuvindu Nuvindu merged commit 3e6e9d6 into ballerina-platform:main Feb 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants