Commit 9f63f1ee authored by Mohammad Nakhaee's avatar Mohammad Nakhaee
Browse files

Merge branch '963-archivebrowser-hash-test' into 'develop'

Resolve "ArchiveBrowser hash test"

Closes #963

See merge request !765
parents 5193d289 e8ca0b72
Pipeline #138651 passed with stages
in 65 minutes and 49 seconds
......@@ -91,14 +91,16 @@ test('Browse metainfo pseudorandomly', async () => {
const path = ''
const lane = await navigateTo(path)
const laneIndex = path ? path.split('/').length : 0
const {count, hash} = await browseRecursively(lane, laneIndex, join('*MetaInfoBrowser*', path), metainfoItemFilter, 2)
const {count} = await browseRecursively(lane, laneIndex, join('*MetaInfoBrowser*', path), metainfoItemFilter, 2)
// Currently we do not test that the visited items are the same using the hash
// returned by browseRecursively. This is because any change in the metainfo
// schema may modify the traversal order, thus changing the hash. If a fixed
// traversal would be required, the list of traversed items should be given
// explicitly, but that then can break if any of the items is removed/renamed.
// Check that the tested number of paths is enough, but also not too high. Adjust
// visitProbabilityDecayFactor if the number is not in this range.
// Check the "tree hash". If the metainfo tree changes, the hash will change, and we normally don't
// want the tree to change. If you DO expect the tree to change, update the expected hash accordingly.
}, 20 * minutes) // NOTE!!! Do not increase this timeout! Rather, adjust the visitProbabilityDecayFactor
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment