Software /
code /
prosody
Annotate
spec/core_configmanager_spec.lua @ 12462:11765f0605ec
mod_s2s: Store real stanzas in session.sendq, rather than strings
This is the "right" thing to do. Strings were more memory-efficient, but
e.g. bypassed stanza filters at reconnection time. Also not being stanzas
prevents us from potential future work, such as merging sendq with mod_smacks.
Regarding performance: we should counter the probable negative effect of this
change with other positive changes that are desired anyway - e.g. a limit on
the size of the sendq, improved in-memory representation of stanzas, s2s
backoff (e.g. if a remote server is persistently unreachable, cache this
failure for a while and don't just keep forever queuing stanzas for it).
author | Matthew Wild <mwild1@gmail.com> |
---|---|
date | Wed, 23 Mar 2022 15:25:22 +0000 |
parent | 10495:8ea685ec0979 |
rev | line source |
---|---|
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
1 |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
2 local configmanager = require "core.configmanager"; |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
3 |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
4 describe("core.configmanager", function() |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
5 describe("#get()", function() |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
6 it("should work", function() |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
7 configmanager.set("example.com", "testkey", 123); |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
8 assert.are.equal(123, configmanager.get("example.com", "testkey"), "Retrieving a set key"); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
9 |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
10 configmanager.set("*", "testkey1", 321); |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
11 assert.are.equal(321, configmanager.get("*", "testkey1"), "Retrieving a set global key"); |
10495
8ea685ec0979
configmanager tests: Split long line
Matthew Wild <mwild1@gmail.com>
parents:
9168
diff
changeset
|
12 assert.are.equal(321, configmanager.get("example.com", "testkey1"), |
8ea685ec0979
configmanager tests: Split long line
Matthew Wild <mwild1@gmail.com>
parents:
9168
diff
changeset
|
13 "Retrieving a set key of undefined host, of which only a globally set one exists" |
8ea685ec0979
configmanager tests: Split long line
Matthew Wild <mwild1@gmail.com>
parents:
9168
diff
changeset
|
14 ); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
15 |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
16 configmanager.set("example.com", ""); -- Creates example.com host in config |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
17 assert.are.equal(321, configmanager.get("example.com", "testkey1"), "Retrieving a set key, of which only a globally set one exists"); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
18 |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
19 assert.are.equal(nil, configmanager.get(), "No parameters to get()"); |
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
20 assert.are.equal(nil, configmanager.get("undefined host"), "Getting for undefined host"); |
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
21 assert.are.equal(nil, configmanager.get("undefined host", "undefined key"), "Getting for undefined host & key"); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
22 end); |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
23 end); |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
24 |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
25 describe("#set()", function() |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
26 it("should work", function() |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
27 assert.are.equal(false, configmanager.set("*"), "Set with no key"); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
28 |
9168
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
29 assert.are.equal(true, configmanager.set("*", "set_test", "testkey"), "Setting a nil global value"); |
29de7ad20250
spec: Correct order of arguments to asserts in configmanager tests
Kim Alvefur <zash@zash.se>
parents:
8236
diff
changeset
|
30 assert.are.equal(true, configmanager.set("*", "set_test", "testkey", 123), "Setting a global value"); |
8236
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
31 end); |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
32 end); |
4878e4159e12
Port tests to the `busted` test runner
Waqas Hussain <waqas20@gmail.com>
parents:
diff
changeset
|
33 end); |