Run the Annex A data tests against a released MobilityDB - #55
Merged
Conversation
The abstract tests that read or write data skip without a database, so continuous integration exercised the routing table and the document assembly and never the lifecycles. A service container gives them one, and the same file that skips by name offline runs there. The image is a released MobilityDB rather than master. A conformance claim names a version somebody can install, and one resting on an unreleased branch names nothing a reader can obtain; the eight test groups pass against 1.3.0. A skip is not a pass. Without MFAPI_DSN every one of these tests skips and the job goes green having exercised nothing, which is the failure the job exists to prevent, so the run is required to report no skip and at least one pass and fails otherwise.
estebanzimanyi
force-pushed
the
ci/conformance-service
branch
from
August 29, 2026 16:30
2769f7b to
58963ee
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The abstract tests that read or write data skip without a database, so continuous integration exercises the routing table and the document assembly and not the lifecycles. A service container gives them one, and the same
ats_live_test.gothat skips by name offline runs there.The image is a released MobilityDB rather than master. A conformance claim names a version somebody can install, and one resting on an unreleased branch names nothing a reader can obtain. The eight groups the job covers against
mobilitydb/mobilitydb:18-3.6-1.3are the collections and collection documents, the features and one feature, the temporal geometry with its distance and velocity queries and its 501 acceleration answer, the temporal properties of all four stored types, both write lifecycles, and the closing read-back of the fixture.A skip is not a pass.
MFAPI_DSNis what makes these tests run at all, and a job that lost it would exercise nothing while still reporting success — the failure this job exists to prevent. The run therefore requires no skip and at least one pass, and fails otherwise.The job triggers on
pull_request, so this pull request exercises it rather than leaving it unproven until a later trigger.Future: with a database in the job, the schema validation can be pointed at real documents rather than scripted ones. That waits on the MF-JSON datetime offset, which
openapi/README.mdrecords — a temporal geometry'sdatetimescarry no format in the schema, so validating them needs the encoder to write the offset minutes RFC 3339 requires.