Opened 3 years ago

Closed 13 months ago

Last modified 13 months ago

#4827 closed defect (fixed)

rmgen library tests

Reported by: elexis Owned by: elexis
Priority: Should Have Milestone: Alpha 24
Component: Maps Keywords:
Cc: Itms Patch: Phab:D2085


Once the rmgen library is sorted well (#4804), we could start testing their correctness.

For instance the math.js functions.
Placers could be tested that they have a place function that returns an array of points or undefined.
Centered Placers could be tested that they do have an x and z property.
It could be tested that creating a new Map produces a map of the correct gridsize,
that out-of-bound access to the grid throws an error,
that a Constraint does what it should and so forth.

Change History (6)

comment:1 by Itms, 3 years ago

Cc: Itms added

in reply to:  description comment:2 by FeXoR, 2 years ago

Replying to elexis:

[...] Centered Placers could be tested that they do have an x and z property.

Shouldn't they have a "center" propperty which is a vector2D?

comment:3 by elexis, 2 years ago

As of #4992, yes

comment:4 by elexis, 20 months ago

Milestone: BacklogAlpha 24
Patch: Phab:D2085
Priority: Nice to HaveShould Have

comment:5 by elexis, 13 months ago

Owner: set to elexis
Resolution: fixed
Status: newclosed

In 23455:

Support random map script tests, fixes #4827.

Differential Revision:
Comments By: wraitii

comment:6 by elexis, 13 months ago

In 23460:

Some more random random map script unit tests and ASSERT functions, refs #4827.

Differential Revision:

Note: See TracTickets for help on using tickets.