…asset (#16564)

This change ensures we won't fail the pipeline if the bundle comparison
task fails:

https://learn.microsoft.com/en-us/azure/devops/pipelines/yaml-schema/steps-task?view=azure-pipelines

---------

Co-authored-by: Tony Murphy <anthonm@microsoft.com>

## Description

This PR adds more complex move tests to the file `editing.spec.ts`

Co-authored-by: Yann Achard <achardy@microsoft.com>

…data store (#16578)

## Description

`SharedDirectory` was using `-1` all the time for locally created
subdirectories.
This leads to bugs when considering the logic in
`isMessageForCurrentInstanceOfSubDirectory` when the `SharedDirectory`
is additionally detached: no ack op will ever be received for that
subdir (since an op is never sent), but clients who load using the
attachment summary may start sending ops which refers to the same
subdirectory.

merge-tree has a similar problem which is resolved by using `-1` or `0`
depending on the attach state of the DDS, see `UnassignedSequenceNumber`
vs `UniversalSequenceNumber`. The fix here brings SharedDirectory
handling closer to merge-tree's.