diff options
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test_rulequery.cpp | 39 |
1 files changed, 39 insertions, 0 deletions
diff --git a/tests/test_rulequery.cpp b/tests/test_rulequery.cpp index 7a34403..54f9477 100644 --- a/tests/test_rulequery.cpp +++ b/tests/test_rulequery.cpp @@ -50,6 +50,7 @@ private slots: void anOrGroupWithExclusionsParses(); void anUnrepresentableQueryRejectsWhole(); void aParenBearingValueIsARowNotAShape(); + void theRuleCorpusRoundTripsByteForByte(); }; void TestRuleQuery::aSingleContainsTermCompiles() @@ -372,5 +373,43 @@ void TestRuleQuery::aParenBearingValueIsARowNotAShape() QCOMPARE(q.compile(), QStringLiteral("from:((((")); } +void TestRuleQuery::theRuleCorpusRoundTripsByteForByte() +{ + // Every query SHAPE present in a real rules file, with placeholder + // values. Verified by hand against the live file when this was written: + // all seventeen rules parsed and compiled back byte for byte, none fell + // to text mode. + // + // A compile that differs by so much as a paren would rewrite the shared + // file on the next save, which the companion tool then sees as a diff + // nobody made. + const QStringList corpus = { + QString(), // a rule with no query yet + QStringLiteral("path:\"account-one/**\""), + QStringLiteral("path:\"account-two/Inbox/topic/**\""), + QStringLiteral("subject:\"[list-name]\""), + QStringLiteral("from:notifications@service.example.org"), + QStringLiteral("from:one@jobs.example.org or " + "from:two@jobs.example.org or " + "from:three@jobs.example.org"), + QStringLiteral("from:mail.vendor.example.org and " + "subject:\"Secure link\""), + QStringLiteral("(from:vendor.example.org or from:vendor.example.net) " + "and not subject:receipt and not subject:refund " + "and not subject:EUR"), + }; + + for (const QString &query : corpus) { + const RuleQuery parsed = RuleQuery::parse(query); + QVERIFY2(parsed.parsed, qPrintable(query)); + QCOMPARE(parsed.compile(), query); + + // And the value itself round-trips, which is what the dialog's + // "was this edited" comparison depends on. + QVERIFY2(RuleQuery::parse(parsed.compile()) == parsed, + qPrintable(query)); + } +} + QTEST_MAIN(TestRuleQuery) #include "test_rulequery.moc" |