diff --git a/tests/ModelValidationTest.cpp b/tests/ModelValidationTest.cpp index 47b7ddf73..5bac11561 100644 --- a/tests/ModelValidationTest.cpp +++ b/tests/ModelValidationTest.cpp @@ -510,12 +510,14 @@ TEST_CASE("ModelValidation_ReindexesTheDataIntoTheModelsIndexing", "[ModelValida CHECK(twin.indexing_op == gemmi::Op::identity()); CHECK(twin.change_of_basis_op == gemmi::Op::identity()); - // Data already in the model's indexing: nothing moves, and there is nothing to arbitrate. + // Data already in the model's indexing: nothing moves. The null is run all the same, as on every + // validation that judges its model (ModelValidationSchedule::fit_test), and the model fits. const auto same = ValidateAgainstModel(own, cell, model, prefix, logger, p4, true, 1, 1.0); REQUIRE(same.ok); CHECK(same.change_of_basis_op == gemmi::Op::identity()); CHECK(same.indexing_op == gemmi::Op::identity()); - CHECK_FALSE(same.fit_tested); + CHECK(same.fit_tested); + CHECK(same.model_fits); CHECK(same.r_work < 0.15); for (const char *suffix : {"_2fofc.ccp4", "_fofc.ccp4", "_anom.ccp4", "_maps.mtz"})