Skip to content

Commit f137736

Browse files
committed
Fix tests
1 parent e107838 commit f137736

3 files changed

Lines changed: 19 additions & 5 deletions

File tree

json_rpc/jsonmarshal.nim

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,6 @@ createJsonFlavor JrpcConv,
2626
# JrpcConv is a namespace/flavor for encoding and decoding
2727
# parameters and return value of a rpc method.
2828

29-
when declared(automaticSerialization):
29+
when declared(json_serialization.automaticSerialization):
3030
# Nim 1.6 cannot use this new feature
3131
JrpcConv.automaticSerialization(JsonNode, true)

tests/test_router_rpc.nim

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@ import
1111
unittest2,
1212
../json_rpc/router,
1313
json_serialization/std/options,
14-
json_serialization/stew/results
14+
json_serialization/pkg/results
1515

1616
var server = RpcRouter()
1717

1818
type
1919
OptAlias[T] = results.Opt[T]
20-
20+
2121
server.rpc("std_option") do(A: int, B: Option[int], C: string, D: Option[int], E: Option[string]) -> string:
2222
var res = "A: " & $A
2323
res.add ", B: " & $B.get(99)
@@ -49,7 +49,7 @@ server.rpc("alias_opt") do(A: int, B: OptAlias[int], C: string, D: Option[int],
4949
res.add ", D: " & $D.get(77)
5050
res.add ", E: " & E.get("none")
5151
return res
52-
52+
5353
server.rpc("noParams") do() -> int:
5454
return 123
5555

@@ -93,7 +93,7 @@ template test_optional(meth: static[string]) =
9393
check res == """{"jsonrpc":"2.0","id":0,"error":{"code":-32000,"message":"`mixed_opt` raised an exception","data":"Parameter [D] of type 'Option[system.int]' could not be decoded: number expected"}}"""
9494
else:
9595
check res == """{"jsonrpc":"2.0","id":0,"error":{"code":-32000,"message":"`alias_opt` raised an exception","data":"Parameter [D] of type 'Option[system.int]' could not be decoded: number expected"}}"""
96-
96+
9797
test meth & " D extra, positional":
9898
let n = req(meth, "[44, 567, \"apple\", 999, \"banana\", true]")
9999
let res = waitFor server.route(n)

tests/testrpcmacro.nim

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,20 @@ MyOptional.useDefaultSerializationIn JrpcConv
5050
MyOptionalNotBuiltin.useDefaultSerializationIn JrpcConv
5151
MuscleCar.useDefaultSerializationIn JrpcConv
5252

53+
when declared(json_serialization.automaticSerialization):
54+
# Nim 1.6 cannot use this new feature
55+
JrpcConv.automaticSerialization(int, true)
56+
JrpcConv.automaticSerialization(string, true)
57+
JrpcConv.automaticSerialization(array, true)
58+
JrpcConv.automaticSerialization(byte, true)
59+
JrpcConv.automaticSerialization(seq, true)
60+
JrpcConv.automaticSerialization(float, true)
61+
JrpcConv.automaticSerialization(JsonString, true)
62+
JrpcConv.automaticSerialization(bool, true)
63+
JrpcConv.automaticSerialization(int64, true)
64+
JrpcConv.automaticSerialization(ref, true)
65+
JrpcConv.automaticSerialization(enum, true)
66+
5367
proc readValue*(r: var JsonReader[JrpcConv], val: var MyEnum)
5468
{.gcsafe, raises: [IOError, SerializationError].} =
5569
let intVal = r.parseInt(int)

0 commit comments

Comments
 (0)