waku-org
/
nwaku
mirror of https://github.com/waku-org/nwaku.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

chore: Better feedback invalid content topic (#2254) 

* typo correction appplication -> application
* content_topic.nim: better feedback to user when wrong topic is passed
* test_namespaced_topics.nim: updating tests accordingly
This commit is contained in:
Ivan FB 2023-11-30 11:11:33 +01:00 committed by GitHub
parent 958b9bd712
commit 72a1f8c724
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tests/waku_core/test_namespaced_topics.nim
View File

@ -89,7 +89,7 @@ suite "Waku Message - Content topics namespacing":
let err = ns.tryError()
check:
err.kind == ParsingErrorKind.InvalidFormat
err.cause == "invalid topic structure"
err.cause == "Invalid content topic structure. Expected either /<application>/<version>/<topic-name>/<encoding> or /<gen>/<application>/<version>/<topic-name>/<encoding>"
test "Parse content topic string - Invalid string: missing encoding part":
## Given
@ -104,7 +104,7 @@ suite "Waku Message - Content topics namespacing":
let err = ns.tryError()
check:
err.kind == ParsingErrorKind.InvalidFormat
err.cause == "invalid topic structure"
err.cause == "Invalid content topic structure. Expected either /<application>/<version>/<topic-name>/<encoding> or /<gen>/<application>/<version>/<topic-name>/<encoding>"
test "Parse content topic string - Invalid string: wrong extra parts":
## Given

8
waku/waku_core/topics/content_topic.nim
View File

@ -74,7 +74,7 @@ proc parse*(T: type NsContentTopic, topic: ContentTopic|string): ParsingResult[N
of 4:
let app = parts[0]
if app.len == 0:
return err(ParsingError.missingPart("appplication"))
return err(ParsingError.missingPart("application"))
let ver = parts[1]
if ver.len == 0:
@ -100,7 +100,7 @@ proc parse*(T: type NsContentTopic, topic: ContentTopic|string): ParsingResult[N
let app = parts[1]
if app.len == 0:
return err(ParsingError.missingPart("appplication"))
return err(ParsingError.missingPart("application"))
let ver = parts[2]
if ver.len == 0:
@ -116,7 +116,9 @@ proc parse*(T: type NsContentTopic, topic: ContentTopic|string): ParsingResult[N
return ok(NsContentTopic.init(some(gen), app, ver, name, enc))
else:
return err(ParsingError.invalidFormat("invalid topic structure"))
let errMsg = "Invalid content topic structure. Expected either /<application>/<version>/<topic-name>/<encoding> or /<gen>/<application>/<version>/<topic-name>/<encoding>"
return err(ParsingError.invalidFormat(errMsg))
# Content topic compatibility