Skip to content

Commit

Permalink
Don't actually run request() and notification()
Browse files Browse the repository at this point in the history
  • Loading branch information
jspahrsummers committed Nov 4, 2024
1 parent 2026260 commit af9df1f
Showing 1 changed file with 17 additions and 15 deletions.
32 changes: 17 additions & 15 deletions src/client/index.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,21 +54,23 @@ test("should typecheck", () => {
});

// Typecheck that only valid weather requests/notifications/results are allowed
weatherClient.request(
{
method: "weather/get",
params: {
city: "Seattle",
false &&

Check failure on line 57 in src/client/index.test.ts

View workflow job for this annotation

GitHub Actions / build

Expected an assignment or function call and instead saw an expression

Check failure on line 57 in src/client/index.test.ts

View workflow job for this annotation

GitHub Actions / build

Unexpected constant truthiness on the left-hand side of a `&&` expression
weatherClient.request(
{
method: "weather/get",
params: {
city: "Seattle",
},
},
},
WeatherResultSchema,
);
WeatherResultSchema,
);

weatherClient.notification({
method: "weather/alert",
params: {
severity: "warning",
message: "Storm approaching",
},
});
false &&

Check failure on line 68 in src/client/index.test.ts

View workflow job for this annotation

GitHub Actions / build

Expected an assignment or function call and instead saw an expression

Check failure on line 68 in src/client/index.test.ts

View workflow job for this annotation

GitHub Actions / build

Unexpected constant truthiness on the left-hand side of a `&&` expression
weatherClient.notification({
method: "weather/alert",
params: {
severity: "warning",
message: "Storm approaching",
},
});
});

0 comments on commit af9df1f

Please sign in to comment.