Use improved statement equality check for signed statement comparison.
While doing this I noticed that lrs takes the extra step of removing the attachments key, which isn't mentioned in the comparison instructions but appears to be necessary to run the conformance tests, which omit attachments.
Corrects the issue mentioned in #64
Use improved statement equality check for signed statement comparison.
While doing this I noticed that
lrstakes the extra step of removing theattachmentskey, which isn't mentioned in the comparison instructions but appears to be necessary to run the conformance tests, which omitattachments.See a workflow run proving this here: https://github.com/yetanalytics/lrs/actions/runs/1365198557
Relevant spec on signed statements: https://github.com/adlnet/xAPI-Spec/blob/master/xAPI-Data.md#26-signed-statements