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

Unit Tests - Deno #735

Merged

Conversation

mvarendorff
Copy link
Contributor

@mvarendorff mvarendorff commented Oct 31, 2023

What does this PR do?

This PR adds unit tests for the Deno SDK.

Test Plan

Generate the SDK, then run deno test in the generated SDK's directory.

Related PRs and Issues

#680

Have you read the Contributing Guidelines on issues?

Yup


Discord username for swag as requested by Tessa: yestheory

@@ -1,17 +1,17 @@
export class {{ spec.title | caseUcfirst}}Exception {
message: String;
code: Number;
message: string;
Copy link
Member

Choose a reason for hiding this comment

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

praise: Good catch! Not sure why we were using these capital Object types

@stnguyen90 stnguyen90 requested review from lohanidamodar and removed request for stnguyen90 December 28, 2023 00:43
@lohanidamodar lohanidamodar merged commit 2244823 into appwrite:master Jan 2, 2024
36 checks passed
@mvarendorff mvarendorff deleted the feat-680-add-unit-tests-deno branch January 2, 2024 08:56
@mvarendorff mvarendorff mentioned this pull request Feb 25, 2024
@gewenyu99
Copy link

Hey there! There were a lot of big PRs during this Hacktoberfest, and we wanted to give everyone ample time to collaborate with our engineering team. If you were able to merge your PRs during October, amazing. If it’s still not merged, don’t worry about it either. Either way, we’ve got your Hacktoberfest swag minted and ready to ship.

Please comment with your Discord username here so we can contact you about your shipping information to deliver your Hacktoberfest swag.

@mvarendorff mvarendorff mentioned this pull request Apr 8, 2024
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.

5 participants