Skip to content
This repository was archived by the owner on Sep 12, 2018. It is now read-only.
This repository was archived by the owner on Sep 12, 2018. It is now read-only.

[tx] Use better keywords in transaction testing #270

@ncalexan

Description

@ncalexan

This ticket tracks replacing :expecteddatoms and :expectedtransactions around

let expected_transaction: Option<&edn::Value> = transaction.get(&edn::Value::NamespacedKeyword(symbols::NamespacedKeyword::new("test", "expectedtransaction")));
with :expected-datoms and :expected-transactions; and updating the Wiki documentation at https://github.com/mozilla/mentat/wiki/Transacting:-EDN-test-format to reflect the changes.

For context, in #214, I landed the first parts of the transaction processor, and I landed the first parts of #188, which starts converting tests from the Clojure implementation to a generic EDN format. While implementing the code in db.rs linked above, I ran into #219, which is now fixed. This ticket tracks patching up the provisional names above.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions